[PATCH/RFA] Fix ctype table and isblank
Jeff Johnston
jjohnstn@redhat.com
Wed Mar 25 19:14:00 GMT 2009
Corinna Vinschen wrote:
> On Mar 24 19:21, Howland Craig D (Craig) wrote:
>
>> Index: libc/include/ctype.h
>> ===================================================================
>> RCS file: /cvs/src/src/newlib/libc/include/ctype.h,v
>> retrieving revision 1.13
>> diff -u -p -r1.13 ctype.h
>> --- libc/include/ctype.h 15 Oct 2008 20:49:56 -0000 1.13
>> +++ libc/include/ctype.h 24 Mar 2009 10:49:14 -0000
>> @@ -54,6 +54,10 @@ extern __IMPORT _CONST char *__ctype_ptr
>> #define isgraph(c)
>> ((__ctype_ptr__)[(unsigned)((c)+1)]&(_P|_U|_L|_N))
>> #define iscntrl(c) ((__ctype_ptr__)[(unsigned)((c)+1)]&_C)
>>
>> +#if !defined(__STRICT_ANSI__) || defined(__cplusplus) ||
>> __STDC_VERSION__ >= 199901L
>> +#define isblank(c) ((__ctype_ptr__)[(unsigned)((c)+1)]&_B)
>> +#endif
>> +
>>
>> /* Non-gcc versions will get the library versions, and will be
>> slightly slower */
>>
>> -------------------------------------------------------------------
>> Looking at this in the larger context, the "|| defined(__cplusplus)"
>> is useless, as where it is positioned in the file puts it inside of
>> #ifndef __cplusplus [line 44]
>> #endif /* !__cplusplus */ [line 66]
>>
>
> Right. I removed the "|| defined(__cplusplus)" locally.
>
>
> Thanks for the heads up,
> Corinna
>
>
Feel free to go ahead and commit.
-- Jeff J.
More information about the Newlib
mailing list