From 23fb72ea6b7324d78df4b89a37566bfed6d77fbb Mon Sep 17 00:00:00 2001 From: John Mark Bell Date: Tue, 13 May 2008 14:37:44 +0000 Subject: The core code has always assumed a locale of "C". Do not change the locale globally, else things will break in weird and wonderful ways. Introduce utils/locale.[ch], which provide locale-specific wrappers for various functions (currently just the ones). Fix up the few places I can see that actually require that the underlying locale is paid attention to. Some notes: 1) The GTK frontend code has not been touched. It is possible that reading of numeric values (e.g. from the preferences dialogue) may break with this change, particularly in locales that use something other than '.' as their decimal separator. 2) The search code is left unchanged (i.e. assuming a locale of "C"). This may break case insensitive matching of non-ASCII characters. I doubt that ever actually worked, anyway. In future, it should use Unicode case conversion to achieve the same effect. 3) The text input handling in the core makes use of isspace() to detect word boundaries. This is fine for western languages (even in the C locale, which it's currently assuming). It will, however, break for CJK et. al. (this has always been the case, rather than being a new issue) 4) text-transform uses locale-specific variants of to{lower,upper}. In future this should probably be performing Unicode case conversion. This is the only part of the core code that makes use of locale information. In future, if you require locale-specific behaviour, do the following: setlocale(LC_, ""); setlocale(LC_, "C"); The first setlocale will change the current locale to the native environment. The second setlocale will reset the current locale to "C". Any value other than "" or "C" is probably a bug, unless there's a really good reason for it. In the long term, it is expected that all locale-dependent code will reside in platform frontends -- the core being wholly locale agnostic (though assuming "C" for things like decimal separators). svn path=/trunk/netsurf/; revision=4153 --- desktop/netsurf.c | 2 -- 1 file changed, 2 deletions(-) (limited to 'desktop/netsurf.c') diff --git a/desktop/netsurf.c b/desktop/netsurf.c index 1a2eb5f64..cc823d4e9 100644 --- a/desktop/netsurf.c +++ b/desktop/netsurf.c @@ -18,7 +18,6 @@ * along with this program. If not, see . */ -#include #include #include #include @@ -111,7 +110,6 @@ void netsurf_init(int argc, char** argv) lib_init(); url_init(); gui_init(argc, argv); - setlocale(LC_ALL, ""); fetch_init(); fetchcache_init(); gui_init2(argc, argv); -- cgit v1.2.3