commit e9d4ca069c92de1c9b567fb294d6fc50ab7d9418
author Marcus Meissner <marcus@jet.franken.de> Wed Jun 15 10:07:55 2011 +0200
committer Alexandre Julliard <julliard@winehq.org> Wed Jun 15 20:04:38 2011 +0200
tree 226ed8848cd2c9b50de619f4c5d61d8eb3a5991c
parent d27478b4769daa79c882fbe70230e5a6d55a796e

shlwapi: Removed superflous NULL check (Coverity).

dlls/shlwapi/string.c

diff --git a/dlls/shlwapi/string.c b/dlls/shlwapi/string.c
index 6102f3c..d0f0f0e 100644
--- a/dlls/shlwapi/string.c
+++ b/dlls/shlwapi/string.c
@@ -2473,17 +2473,12 @@
   if ((lpszIter = StrChrA(lpszStr, '&')))
   {
     lpszTmp = CharNextA(lpszIter);
-    if (lpszTmp && *lpszTmp)
+    if (*lpszTmp)
     {
       if (*lpszTmp != '&')
         ch =  *lpszTmp;
 
-      while (lpszIter && *lpszIter)
-      {
-        lpszTmp = CharNextA(lpszIter);
-        *lpszIter = *lpszTmp;
-        lpszIter = lpszTmp;
-      }
+      memmove( lpszIter, lpszTmp, strlen(lpszTmp) + 1 );
     }
   }
 
@@ -2507,17 +2502,12 @@
   if ((lpszIter = StrChrW(lpszStr, '&')))
   {
     lpszTmp = lpszIter + 1;
-    if (lpszTmp && *lpszTmp)
+    if (*lpszTmp)
     {
       if (*lpszTmp != '&')
         ch =  *lpszTmp;
 
-      while (lpszIter && *lpszIter)
-      {
-        lpszTmp = lpszIter + 1;
-        *lpszIter = *lpszTmp;
-        lpszIter = lpszTmp;
-      }
+      memmove( lpszIter, lpszTmp, (strlenW(lpszTmp) + 1) * sizeof(WCHAR) );
     }
   }