[ruby-talk:5709] build warning...

My primary platform is powerpc, and I'm really used to seeing:
'Comparison always true/false due to limited range in data type'

spit out by gcc due to the fact that on x86 (and other LE archs) char
defaults to signed char, but on the ppc (and ARM, don't know about
sparc) char =3D=3D unsigned char.

for example:
void main(void) {
        char t;
        int i =3D -1;
        t =3D i;
        printf("val: %d\n", (int)t);
}

prints
val =3D 255
on powerpc, but
val =3D -1
on intel.

normally this can cause major problems, and I saw this while building
ruby.  the warning was actually in the parser...

line 3520 of parse.y contains is_identchar(tok()[0]).  tok()
(tokenbuf) is a char *, and is_identchar does this:
((int)(c))!=3D-1 ...

of course, this will always be true on ppc systems.

now, I don't think there's really a problem, because I _think_
tokenbuf characters are never set to a negative value. (could be
wrong, but I didn't see any cases).

in any case, a fix is attached.  this would fix future (obscure) bugs,
and at least get rid of that warning.  (that warning is flagged on the
debian buildd for powerpc because it is usually an indication that
something is broken).

Thanks,

--=20
Josh
| [e-mail] |
1024D/6B21489A 61F0 6138 BE7B FEBF A223  E9D1 BFE1 2065 6B21 489A