On Fri, 2 Dec 2011, Joerg Sonnenberger wrote:

> On Fri, Dec 02, 2011 at 09:51:12PM +0900, Carsten Haitzler wrote:
>>>> 2. all the chasting and changing to unsigned char for passing into isspace
>>>> () - isspace() actually takes an int, not unsigned char, so this just
>>>> doesn't make sense. :( (well it does according to the manual page i have)
>>>
>>> Please read the manual again. ctype.h stuff takes an integer argument,
>>> but the value must be -1..255 (for 8bit char and EOF==-1). It is
>>> undefined behavior to pass anything else. Especially in programs using
>>
>> no such mention in the manual page at all. no limits on range. since an 
>> integer
>> is the possible input, then it must handle all int values sensibly in some 
>> way
>> as the docs don't mention any limitations. :)
>>
>>> setlocale(), the result for 0xff as input byte can make a real
>>> difference for platforms with signed char.
>>
>> can  you point me to these docs, as the manual page i have in front of me for
>> linux disagrees :)
>
> http://pubs.opengroup.org/onlinepubs/9699919799/functions/isspace.html
>
> Note the last paragraph in DESCRIPTION.

so why not just do a & 0xff instead of casting ?

Vincent

>
> Joerg
>
> ------------------------------------------------------------------------------
> All the data continuously generated in your IT infrastructure
> contains a definitive record of customers, application performance,
> security threats, fraudulent activity, and more. Splunk takes this
> data and makes sense of it. IT sense. And common sense.
> http://p.sf.net/sfu/splunk-novd2d
> _______________________________________________
> enlightenment-devel mailing list
> [email protected]
> https://lists.sourceforge.net/lists/listinfo/enlightenment-devel
>
>

------------------------------------------------------------------------------
All the data continuously generated in your IT infrastructure 
contains a definitive record of customers, application performance, 
security threats, fraudulent activity, and more. Splunk takes this 
data and makes sense of it. IT sense. And common sense.
http://p.sf.net/sfu/splunk-novd2d
_______________________________________________
enlightenment-devel mailing list
[email protected]
https://lists.sourceforge.net/lists/listinfo/enlightenment-devel

Reply via email to