# HG changeset patch # User Jim Blandy # Date 734064172 0 # Node ID d713b2c4f84f9d96176801817311f6cba3a1102e # Parent 63606015cd94778ab30c2a9617b1e491a58873ac *** empty log message *** diff --git a/regex.c b/regex.c --- a/regex.c +++ b/regex.c @@ -127,7 +127,18 @@ /* isalpha etc. are used for the character classes. */ #include -#ifndef isascii +/* Jim Meyering writes: + + "... Some ctype macros are valid only for character codes that + isascii says are ASCII (SGI's IRIX-4.0.5 is one such system --when + using /bin/cc or gcc but without giving an ansi option). So, all + ctype uses should be through macros like ISPRINT... If + STDC_HEADERS is defined, then autoconf has verified that the ctype + macros don't need to be guarded with references to isascii. ... + Defining isascii to 1 should let any compiler worth its salt + eliminate the && through constant folding." */ +#if ! defined (isascii) || defined (STDC_HEADERS) +#undef isascii #define isascii(c) 1 #endif @@ -3463,7 +3474,11 @@ } } else - assert (bufp->regs_allocated == REGS_FIXED); + { + /* These braces fend off a "empty body in an else-statement" + warning under GCC when assert expands to nothing. */ + assert (bufp->regs_allocated == REGS_FIXED); + } /* Convert the pointer data in `regstart' and `regend' to indices. Register zero has to be set differently,