[PATCH] winemenubuilder: Avoid warning about uninitialized value (clang).

Lauri Kenttä lauri.kentta at gmail.com
Tue May 17 15:33:03 CDT 2011


---
 programs/winemenubuilder/winemenubuilder.c |    8 +++-----
 1 files changed, 3 insertions(+), 5 deletions(-)

diff --git a/programs/winemenubuilder/winemenubuilder.c b/programs/winemenubuilder/winemenubuilder.c
index 5908e4e..56c0f80 100644
--- a/programs/winemenubuilder/winemenubuilder.c
+++ b/programs/winemenubuilder/winemenubuilder.c
@@ -1012,8 +1012,7 @@ static HRESULT platform_write_icon(IStream *icoStream, int exeIndex, LPCWSTR ico
 
     for (i = 0; i < numEntries; i++)
     {
-        int bestIndex;
-        int maxBits = -1;
+        int bestIndex = i;
         int j;
         BOOLEAN duplicate = FALSE;
         int w, h;
@@ -1034,14 +1033,13 @@ static HRESULT platform_write_icon(IStream *icoStream, int exeIndex, LPCWSTR ico
         }
         if (duplicate)
             continue;
-        for (j = i; j < numEntries; j++)
+        for (j = i + 1; j < numEntries; j++)
         {
             if (iconDirEntries[j].bWidth == iconDirEntries[i].bWidth &&
                 iconDirEntries[j].bHeight == iconDirEntries[i].bHeight &&
-                iconDirEntries[j].wBitCount >= maxBits)
+                iconDirEntries[j].wBitCount >= iconDirEntries[bestIndex].wBitCount)
             {
                 bestIndex = j;
-                maxBits = iconDirEntries[j].wBitCount;
             }
         }
         WINE_TRACE("Selected: %d\n", bestIndex);
-- 
1.7.5.1




More information about the wine-patches mailing list