Re: Patch for isdigit/isalpha/etc. macro arguments

Steve Ellcey
2007-04-06T23:59:48Z

> Bleach! So that's not a viable option. I think it's time to find the
> magic gcc option to turn the warning off. I mean
>
> char *s = "...";
> isalpha(*s);
>
> is perfectly reasonable code and should not result in compiler
> warnings. And I really don't like adding the casts you propose. To
> me, a cast is an indication that something naughty is being done, and
> that certainly isn't the case here.
>
> Mark

That would be -Wno-char-subscripts. I'll look into adding it.

Steve Ellcey
sje@cup.hp.com