X11DRV: Fix signed/unsigned comparison warnings

Rein Klazes rklazes at xs4all.nl
Thu Aug 12 02:06:43 CDT 2004


On Wed, 11 Aug 2004 23:10:29 +0200, you wrote:

>  static int LookupEntry(LFANDSIZE *plfsz)
>  {
> -  int i, prev_i = -1;
> +  unsigned int i;
> +  int prev_i = -1;
>  
>    for(i = mru; i >= 0; i = glyphsetCache[i].next) {
                  ^^^^^^^

Hey, this comparison is now always true, does gcc does not warn about
this?

I am afraid that this exercition is in many cases changing perfectly
valid C code, but with warning, to possibly buggy code without warnings.
I do not like this idea at all.

Rein.
-- 
Rein Klazes
rklazes at xs4all.nl



More information about the wine-devel mailing list