Joris Huizer : winedump: sign-compare fixes.

Alexandre Julliard julliard at wine.codeweavers.com
Mon Mar 5 13:10:22 CST 2007


Module: wine
Branch: master
Commit: 4bfd7059f2c2094f8f7fab0d4c7b079b9fbd09be
URL:    http://source.winehq.org/git/wine.git/?a=commit;h=4bfd7059f2c2094f8f7fab0d4c7b079b9fbd09be

Author: Joris Huizer <joris_huizer at yahoo.com>
Date:   Sat Mar  3 05:30:32 2007 -0800

winedump: sign-compare fixes.

---

 tools/winedump/minidump.c |    2 +-
 tools/winedump/msc.c      |   16 +++++++++-------
 2 files changed, 10 insertions(+), 8 deletions(-)

diff --git a/tools/winedump/minidump.c b/tools/winedump/minidump.c
index 2c61a7c..07108b8 100644
--- a/tools/winedump/minidump.c
+++ b/tools/winedump/minidump.c
@@ -44,7 +44,7 @@ static void dump_mdmp_string(DWORD rva)
         printf("<<?>>");
 }
 
-static const MINIDUMP_DIRECTORY* get_mdmp_dir(const MINIDUMP_HEADER* hdr, int str_idx)
+static const MINIDUMP_DIRECTORY* get_mdmp_dir(const MINIDUMP_HEADER* hdr, unsigned int str_idx)
 {
     const MINIDUMP_DIRECTORY*   dir;
     unsigned int                i;
diff --git a/tools/winedump/msc.c b/tools/winedump/msc.c
index 4ef3387..e7e1100 100644
--- a/tools/winedump/msc.c
+++ b/tools/winedump/msc.c
@@ -511,6 +511,7 @@ static void codeview_dump_one_type(unsigned curr_type, const union codeview_type
 {
     const union codeview_reftype* reftype = (const union codeview_reftype*)type;
     int                 i, leaf_len, value;
+    unsigned int        j;
     const char*         str;
 
     switch (type->generic.id)
@@ -663,9 +664,9 @@ static void codeview_dump_one_type(unsigned curr_type, const union codeview_type
 
     case LF_ARGLIST_V2:
         printf("\t%x => Arglist V2(#%u):", curr_type, reftype->arglist_v2.num);
-        for (i = 0; i < reftype->arglist_v2.num; i++)
+        for (j = 0; j < reftype->arglist_v2.num; j++)
         {
-            printf("\t %x", reftype->arglist_v2.args[i]);
+            printf("\t %x", reftype->arglist_v2.args[j]);
         }
         printf("\t\n");
         break;
@@ -786,9 +787,9 @@ static void codeview_dump_one_type(unsigned curr_type, const union codeview_type
 
     case LF_DERIVED_V2:
         printf("\t%x => Derived V2(#%u):", curr_type, reftype->derived_v2.num);
-        for (i = 0; i < reftype->derived_v2.num; i++)
+        for (j = 0; j < reftype->derived_v2.num; j++)
         {
-            printf(" %x", reftype->derived_v2.drvdcls[i]);
+            printf(" %x", reftype->derived_v2.drvdcls[j]);
         }
         printf("\n");
         break;
@@ -832,9 +833,10 @@ int codeview_dump_types_from_block(const void* table, unsigned long len)
 
 int codeview_dump_symbols(const void* root, unsigned long size)
 {
-    int     i, length;
-    char*   curr_func = NULL;
-    int     nest_block = 0;
+    unsigned int i;
+    int          length;
+    char*        curr_func = NULL;
+    int          nest_block = 0;
     /*
      * Loop over the different types of records and whenever we
      * find something we are interested in, record it and move on.




More information about the wine-cvs mailing list