Re: Ctype patch for review

From: Andrey Chernov <ache_at_nagual.pp.ru>
Date: Mon, 17 Sep 2007 13:21:30 +0400
On Mon, Sep 17, 2007 at 10:29:21AM +0200, Petr Hroudn?? wrote:
> 2007/9/16, Andrey Chernov <ache_at_nagual.pp.ru>:
> > The problem is: currently our single byte ctype functions are broken for
> > wide characters locales in the argument range >= 0x80 - they may return
> > false positives.
> >
> > For example, for UTF-8 locale we currently have:
> > iswspace(0xA0)==1 and isspace(0xA0)==1
> > (because iswspace() and isspace() are the same code)
> > but must have
> > isspace(0xA0)==0
> 
> This is exactly what happens on other OSes and I agree this is the
> right behaviour
> for UTF-8. However, we must ensure, that:
> 
> for C locale:  isspace(0xA0)==0
> for ISO8859-* locales: isspace(0xA0)==1
> for UTF-8 locales: isspace(0xA0)==0

The patch test for wide char locale presence first (__mb_cur_max > 1), so 
does not affect single byte locales like ISO8859-*

-- 
http://ache.pp.ru/
Received on Mon Sep 17 2007 - 07:21:32 UTC

This archive was generated by hypermail 2.4.0 : Wed May 19 2021 - 11:39:17 UTC