shlwapi: Removed superflous NULL check (Coverity).

e9d4ca06 · Marcus Meissner · Alexandre Julliard · d27478b4 · e9d4ca06
Commit e9d4ca06 authored Jun 15, 2011 by Marcus Meissner Committed by Alexandre Julliard Jun 15, 2011
Hide whitespace changes
Inline Side-by-side

Showing with 4 additions and 14 deletions

string.c dlls/shlwapi/string.c +4 -14

No files found.
--- a/dlls/shlwapi/string.c
+++ b/dlls/shlwapi/string.c
@@ -2473,17 +2473,12 @@ char WINAPI SHStripMneumonicA(LPCSTR lpszStr)
  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 @@ WCHAR WINAPI SHStripMneumonicW(LPCWSTR lpszStr)
  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) );
    }
  }