[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