On 07/16/2014 01:29 PM, Eric Blake wrote:
On 07/15/2014 11:41 PM, Bernhard Voelker wrote:
That looks good to include in the release.
No need to respin a snapshot for this though I think.
I didn't follow closely - you know, the FIFA world cup ;-) -
... but what about the other 17 uses of isblank() in src/*.c?
Most of them are already 'isblank (to_uchar (expr))', which is safe even
in the presence of a unibyte locale where byte 255 is considered space.
Right, but I guess Bernhard was wondering about the consistency aspect.
This would be significant in unibyte locales where isblank(\xA0)
would return non zero. For consistency with parsing in other utils,
I would use c_isdigit() and isblank(to_uchar (...)).
Assaf I'll adjust your patch accordingly and apply later.