diff options
author | John Mark Bell <jmb@netsurf-browser.org> | 2008-05-13 14:37:44 +0000 |
---|---|---|
committer | John Mark Bell <jmb@netsurf-browser.org> | 2008-05-13 14:37:44 +0000 |
commit | 23fb72ea6b7324d78df4b89a37566bfed6d77fbb (patch) | |
tree | 52b575661394b487e813fcd11bce3a5cdbff00a0 /riscos | |
parent | 74a1095cd6cf6c8f3b42db14e1d3352eb019d335 (diff) | |
download | netsurf-23fb72ea6b7324d78df4b89a37566bfed6d77fbb.tar.gz netsurf-23fb72ea6b7324d78df4b89a37566bfed6d77fbb.tar.bz2 |
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 <ctype.h> 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_<whatever>, "");
<your operation(s) here>
setlocale(LC_<whatever>, "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
Diffstat (limited to 'riscos')
-rw-r--r-- | riscos/401login.c | 1 | ||||
-rw-r--r-- | riscos/download.c | 1 | ||||
-rw-r--r-- | riscos/plugin.c | 1 | ||||
-rw-r--r-- | riscos/wimp.c | 20 |
4 files changed, 18 insertions, 5 deletions
diff --git a/riscos/401login.c b/riscos/401login.c index e65c0097b..cfc256ad8 100644 --- a/riscos/401login.c +++ b/riscos/401login.c @@ -17,7 +17,6 @@ */ #include <assert.h> -#include <ctype.h> #include <stdbool.h> #include <string.h> #include "oslib/wimp.h" diff --git a/riscos/download.c b/riscos/download.c index 4affe26dd..9d0625f4a 100644 --- a/riscos/download.c +++ b/riscos/download.c @@ -29,7 +29,6 @@ */ #include <assert.h> -#include <ctype.h> #include <string.h> #include <sys/time.h> #include <time.h> diff --git a/riscos/plugin.c b/riscos/plugin.c index c08397de4..dd10b3a2d 100644 --- a/riscos/plugin.c +++ b/riscos/plugin.c @@ -38,7 +38,6 @@ */ #include <assert.h> -#include <ctype.h> #include <stdbool.h> #include <stdio.h> #include <stdlib.h> diff --git a/riscos/wimp.c b/riscos/wimp.c index cc22c1091..9463df56b 100644 --- a/riscos/wimp.c +++ b/riscos/wimp.c @@ -21,7 +21,7 @@ */ #include <assert.h> -#include <ctype.h> +#include <locale.h> #include <stdbool.h> #include <stdlib.h> #include <string.h> @@ -51,7 +51,7 @@ static wimp_w furniture_window = NULL; unsigned char last_sprite_found[16]; /** - * Gets the horzontal scrollbar height + * Gets the horizontal scrollbar height * * \param w the window to read (or NULL to read a cached value) */ @@ -398,7 +398,13 @@ void ro_gui_set_icon_string_le(wimp_w w, wimp_i i, const char *text) { */ void ro_gui_set_icon_integer(wimp_w w, wimp_i i, int value) { char buffer[20]; // Big enough for 64-bit int + + setlocale(LC_NUMERIC, ""); + sprintf(buffer, "%d", value); + + setlocale(LC_NUMERIC, "C"); + ro_gui_set_icon_string(w, i, buffer); } @@ -413,6 +419,8 @@ void ro_gui_set_icon_integer(wimp_w w, wimp_i i, int value) { void ro_gui_set_icon_decimal(wimp_w w, wimp_i i, int value, int decimal_places) { char buffer[20]; // Big enough for 64-bit int + setlocale(LC_NUMERIC, ""); + switch (decimal_places) { case 0: sprintf(buffer, "%d", value); @@ -427,6 +435,9 @@ void ro_gui_set_icon_decimal(wimp_w w, wimp_i i, int value, int decimal_places) assert(!"Unsupported decimal format"); break; } + + setlocale(LC_NUMERIC, "C"); + ro_gui_set_icon_string(w, i, buffer); } @@ -445,7 +456,12 @@ int ro_gui_get_icon_decimal(wimp_w w, wimp_i i, int decimal_places) { for (; decimal_places > 0; decimal_places--) multiple *= 10; + setlocale(LC_NUMERIC, ""); + value = atof(ro_gui_get_icon_string(w, i)) * multiple; + + setlocale(LC_NUMERIC, "C"); + return (int)value; } |