[Python-Dev] Re: sre warnings

[Martin v. Loewis]

Gustavo Niemeyer wrote:

> Ohh.. I wasn't aware about this case, since I get no errors at all at
> this line. Do you see any others? 

As Tim explains, this is likely due to me using a more recent gcc (3.3).

I'll attach my current list of warnings below.

The signed/unsigned warnings are actually harmless: they cause a problem
only when comparing negative and unsigned numbers (where the negative
number gets converted to unsigned and may actually compare larger than
the positive one). This should not be an issue for SRE - I believe
all the numbers are always non-negative, unless they are the -1 marker
(for which I'm uncertain whether it is ever relationally compared).

Regards,
Martin

Modules/_sre.c: In function `sre_lower':
Modules/_sre.c:149: warning: signed and unsigned type in conditional 
expression
Modules/_sre.c: In function `sre_lower_locale':
Modules/_sre.c:162: warning: signed and unsigned type in conditional 
expression
In file included from Modules/_sre.c:296:
Modules/_sre.c: In function `sre_at':
Modules/_sre.c:381: warning: comparison is always true due to limited 
range of data type
Modules/_sre.c:383: warning: comparison is always true due to limited 
range of data type
Modules/_sre.c:390: warning: comparison is always true due to limited 
range of data type
Modules/_sre.c:392: warning: comparison is always true due to limited 
range of data type
Modules/_sre.c: In function `sre_match':
Modules/_sre.c:811: warning: comparison between signed and unsigned
Modules/_sre.c:824: warning: comparison between signed and unsigned
Modules/_sre.c:980: warning: comparison between signed and unsigned
Modules/_sre.c:991: warning: comparison between signed and unsigned
Modules/_sre.c:1059: warning: comparison between signed and unsigned
Modules/_sre.c:1076: warning: comparison between signed and unsigned
Modules/_sre.c:1132: warning: comparison between signed and unsigned
Modules/_sre.c:1166: warning: comparison between signed and unsigned
Modules/_sre.c:1194: warning: comparison between signed and unsigned
Modules/_sre.c:1198: warning: comparison between signed and unsigned
Modules/_sre.c:1208: warning: comparison between signed and unsigned
Modules/_sre.c:1215: warning: comparison between signed and unsigned
Modules/_sre.c:1217: warning: comparison between signed and unsigned
Modules/_sre.c:1220: warning: comparison between signed and unsigned
Modules/_sre.c:1236: warning: comparison between signed and unsigned
Modules/_sre.c:1257: warning: comparison between signed and unsigned
Modules/_sre.c:1261: warning: comparison between signed and unsigned
Modules/_sre.c:1275: warning: comparison between signed and unsigned
Modules/_sre.c:1287: warning: comparison between signed and unsigned
Modules/_sre.c:1292: warning: comparison between signed and unsigned
Modules/_sre.c:1380: warning: comparison between signed and unsigned
Modules/_sre.c:1392: warning: comparison between signed and unsigned
Modules/_sre.c: In function `sre_umatch':
Modules/_sre.c:811: warning: comparison between signed and unsigned
Modules/_sre.c:824: warning: comparison between signed and unsigned
Modules/_sre.c:980: warning: comparison between signed and unsigned
Modules/_sre.c:991: warning: comparison between signed and unsigned
Modules/_sre.c:1051: warning: comparison between signed and unsigned
Modules/_sre.c:1059: warning: comparison between signed and unsigned
Modules/_sre.c:1076: warning: comparison between signed and unsigned
Modules/_sre.c:1132: warning: comparison between signed and unsigned
Modules/_sre.c:1166: warning: comparison between signed and unsigned
Modules/_sre.c:1194: warning: comparison between signed and unsigned
Modules/_sre.c:1198: warning: comparison between signed and unsigned
Modules/_sre.c:1208: warning: comparison between signed and unsigned
Modules/_sre.c:1215: warning: comparison between signed and unsigned
Modules/_sre.c:1217: warning: comparison between signed and unsigned
Modules/_sre.c:1220: warning: comparison between signed and unsigned
Modules/_sre.c:1236: warning: comparison between signed and unsigned
Modules/_sre.c:1257: warning: comparison between signed and unsigned
Modules/_sre.c:1261: warning: comparison between signed and unsigned
Modules/_sre.c:1275: warning: comparison between signed and unsigned
Modules/_sre.c:1287: warning: comparison between signed and unsigned
Modules/_sre.c:1292: warning: comparison between signed and unsigned
Modules/_sre.c:1380: warning: comparison between signed and unsigned
Modules/_sre.c:1392: warning: comparison between signed and unsigned