summaryrefslogtreecommitdiffstats
path: root/include/wctype.h
diff options
context:
space:
mode:
authorache <ache@FreeBSD.org>2007-10-13 16:28:22 +0000
committerache <ache@FreeBSD.org>2007-10-13 16:28:22 +0000
commita5038f060de9f1cc50cf532f78541dfd901f10b8 (patch)
tree364de71872fe91708dda5fd7ffeb957967a6f749 /include/wctype.h
parent5b067f00c53dea274158508c1867eedfa02afea8 (diff)
downloadFreeBSD-src-a5038f060de9f1cc50cf532f78541dfd901f10b8.zip
FreeBSD-src-a5038f060de9f1cc50cf532f78541dfd901f10b8.tar.gz
The problem is: currently our single byte ctype(3) functions are broken
for wide characters locales in the argument range >= 0x80 - they may return false positives. Example 1: 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 iswspace(0xA0)==1 and isspace(0xA0)==0 (because there is no such character and all others in the range 0x80..0xff for the UTF-8 locale, it keeps ASCII only in the single byte range because our internal wchar_t representation for UTF-8 is UCS-4). Example 2: for all wide character locales isalpha(arg) when arg > 0xFF may return false positives (must be 0). (because iswalpha() and isalpha() are the same code) This change address this issue separating single byte and wide ctype and also fix iswascii() (currently iswascii() is broken for arguments > 0xFF). This change is 100% binary compatible with old binaries. Reviewied by: i18n@
Diffstat (limited to 'include/wctype.h')
-rw-r--r--include/wctype.h2
1 files changed, 1 insertions, 1 deletions
diff --git a/include/wctype.h b/include/wctype.h
index 098045f..31f401f 100644
--- a/include/wctype.h
+++ b/include/wctype.h
@@ -106,7 +106,7 @@ __END_DECLS
#define towupper(wc) __toupper(wc)
#if __BSD_VISIBLE
-#define iswascii(wc) (((wc) & ~0x7F) == 0)
+#define iswascii(wc) ((wc) < 0x80)
#define iswhexnumber(wc) __istype((wc), _CTYPE_X)
#define iswideogram(wc) __istype((wc), _CTYPE_I)
#define iswnumber(wc) __istype((wc), _CTYPE_D)
OpenPOWER on IntegriCloud