I believe this one causes problems -- please do not commit yet<br><br><b><i>Joris Huizer <joris_huizer@yahoo.com></i></b> wrote:<blockquote class="replbq" style="border-left: 2px solid rgb(16, 16, 255); margin-left: 5px; padding-left: 5px;"> <br><div> </div><hr size="1">Need Mail bonding?<br>Go to the <a href="http://answers.yahoo.com/dir/index;_ylc=X3oDMTFvbGNhMGE3BF9TAzM5NjU0NTEwOARfcwMzOTY1NDUxMDMEc2VjA21haWxfdGFnbGluZQRzbGsDbWFpbF90YWcx?link=ask&sid=396546091">Yahoo! Mail Q&A</a> for <a href="http://answers.yahoo.com/dir/index;_ylc=X3oDMTFvbGNhMGE3BF9TAzM5NjU0NTEwOARfcwMzOTY1NDUxMDMEc2VjA21haWxfdGFnbGluZQRzbGsDbWFpbF90YWcx?link=ask&sid=396546091">great tips from Yahoo! Answers</a> users.>From 8686a200bd1f1cbd934d761013c487a9ddc195c9 Mon Sep 17 00:00:00 2001<br>From: Joris Huizer <jorishuizer@debian.(none)><br>Date: Wed, 7 Feb 2007 15:19:25 +0100<br>Subject: [PATCH] winegcc sign-compare fixes<br>To: wine-patches
<wine-patches@winehq.org><br><br>---<br> tools/winegcc/utils.c | 13 +++++++------<br> tools/winegcc/winegcc.c | 11 ++++++-----<br> 2 files changed, 13 insertions(+), 11 deletions(-)<br><br>diff --git a/tools/winegcc/utils.c b/tools/winegcc/utils.c<br>index 1af91e9..c144077 100644<br>--- a/tools/winegcc/utils.c<br>+++ b/tools/winegcc/utils.c<br>@@ -120,14 +120,14 @@ void strarray_add(strarray* arr, const c<br> <br> void strarray_del(strarray* arr, int i)<br> {<br>- if (i < 0 || i >= arr->size) error("Invalid index i=%d", i);<br>+ if (i < 0 || (unsigned int)i >= arr->size) error("Invalid index i=%d", i);<br> memmove(&arr->base[i], &arr->base[i + 1], (arr->size - i - 1) * sizeof(arr->base[0]));<br> arr->size--;<br> }<br> <br> void strarray_addall(strarray* arr, const strarray* from)<br> {<br>- int i;<br>+ unsigned int i;<br> <br> for (i = 0; i < from->size; i++)<br> strarray_add(arr,
from->base[i]);<br>@@ -136,7 +136,7 @@ void strarray_addall(strarray* arr, cons<br> strarray* strarray_dup(const strarray* arr)<br> {<br> strarray* dup = strarray_alloc();<br>- int i;<br>+ unsigned int i;<br> <br> for (i = 0; i < arr->size; i++)<br> strarray_add(dup, arr->base[i]);<br>@@ -160,7 +160,7 @@ strarray* strarray_fromstring(const char<br> char* strarray_tostring(const strarray* arr, const char* sep)<br> {<br> char *str, *newstr;<br>- int i;<br>+ unsigned int i;<br> <br> str = strmake("%s", arr->base[0]);<br> for (i = 1; i < arr->size; i++)<br>@@ -277,7 +277,7 @@ static file_type guess_lib_type(const ch<br> <br> file_type get_lib_type(strarray* path, const char* library, char** file)<br> {<br>- int i;<br>+ unsigned int i;<br> <br> for (i = 0; i < path->size; i++)<br> {<br>@@ -289,7 +289,8 @@ file_type get_lib_type(strarray* path, c<br> <br> void spawn(const strarray* prefix, const strarray*
args, int ignore_errors)<br> {<br>- int i, status;<br>+ unsigned int i;<br>+ int status;<br> strarray* arr = strarray_dup(args);<br> const char** argv;<br> char* prog = 0;<br>diff --git a/tools/winegcc/winegcc.c b/tools/winegcc/winegcc.c<br>index f1aa87e..f1a7bb0 100644<br>--- a/tools/winegcc/winegcc.c<br>+++ b/tools/winegcc/winegcc.c<br>@@ -170,7 +170,7 @@ struct options<br> <br> static void clean_temp_files(void)<br> {<br>- int i;<br>+ unsigned int i;<br> <br> if (keep_generated) return;<br> <br>@@ -239,7 +239,8 @@ static const strarray* get_translator(en<br> static void compile(struct options* opts, const char* lang)<br> {<br> strarray* comp_args = strarray_alloc();<br>- int j, gcc_defs = 0;<br>+ unsigned int j;<br>+ int gcc_defs = 0;<br> <br> switch(opts->processor)<br> {<br>@@ -434,7 +435,7 @@ static void build(struct options* opts)<br> const char *output_name, *spec_file, *lang;<br> const char* winebuild
= getenv("WINEBUILD");<br> int generate_app_loader = 1;<br>- int j;<br>+ unsigned int j;<br> <br> /* NOTE: for the files array we'll use the following convention:<br> * -axxx: xxx is an archive (.a)<br>@@ -719,7 +720,7 @@ static int is_linker_arg(const char* arg<br> "-static", "-static-libgcc", "-shared", "-shared-libgcc", "-symbolic",<br> "-framework"<br> };<br>- int j;<br>+ unsigned int j;<br> <br> switch (arg[1]) <br> {<br>@@ -769,7 +770,7 @@ static int is_mingw_arg(const char* arg)<br> {<br> "-mno-cygwin", "-mwindows", "-mconsole", "-mthreads", "-municode"<br> };<br>- int j;<br>+ unsigned int j;<br> <br> for (j = 0; j < sizeof(mingw_switches)/sizeof(mingw_switches[0]); j++)<br> if (strcmp(mingw_switches[j], arg) == 0) return 1;<br>-- <br>1.4.4<br><br><br></wine-patches@winehq.org></jorishuizer@debian.(none)></blockquote><br><p> 
<hr size=1>Need a quick answer? Get one in minutes from people who know. Ask your question on
<a href="http://answers.yahoo.com/;_ylc=X3oDMTFvbGNhMGE3BF9TAzM5NjU0NTEwOARfcwMzOTY1NDUxMDMEc2VjA21haWxfdGFnbGluZQRzbGsDbWFpbF90YWcx">Yahoo! Answers</a>.