cvs commit: src/lib/libc/locale utf8.c

Andrey Chernov ache at nagual.pp.ru
Sat Oct 27 02:33:50 PDT 2007


On Sat, Oct 27, 2007 at 01:22:13PM +0400, Andrey Chernov wrote:
> > After some tests, the patch is at
> > 
> > http://people.freebsd.org/~rafan/ctype-abi-fix-6.diff
> > 
> > Let me describe this patch in more details. Fist, we actually
> > compile those inline functions as real functions in
> > lib/libc/locale/nomacro.c if _EXTERNAL_CTYPE_INLINES_ is defined.
> > I make 4 inline functions that use the __mb_sb_limit symbol,
> > which break the compatibility, always compiled as real functions.
> > I also considered that moving those inlined function in another
> > C file, but I think it would make the source codes diverse
> > among 6 and 7/8.
> 
> As a small improvement, we can skip __isctype() from this list too and 
> just hardcode 128 there, i.e.: return (_c < 0 || _c >= 128) ...
> since default locale never define anything above 128.

See _ctype.h patch just committed in -current

-- 
http://ache.pp.ru/


More information about the cvs-src mailing list