Thanks for the report.  Please note that your patch sets the thousands
separator to "C", which is probably not what you had in mind.  I'm
about to apply a slightly different patch to deal with the problem you
describe:

2007-02-11  Hrvoje Niksic  <[EMAIL PROTECTED]>

        * utils.c (get_grouping_data): Cope with systems where
        localeconv() doesn't initialize lconv->thousand_sep and/or
        lconv->grouping.  Based on report by Dirk Vanhaute.

Index: src/utils.c
===================================================================
--- src/utils.c (revision 2205)
+++ src/utils.c (working copy)
@@ -1215,8 +1215,8 @@
       /* Get the grouping info from the locale. */
       struct lconv *lconv = localeconv ();
       cached_sep = lconv->thousands_sep;
-      cached_grouping = lconv->grouping;
-      if (!*cached_sep)
+      cached_grouping = lconv->grouping ? lconv->grouping : "\x03";
+      if (!cached_sep || !*cached_sep)
        {
          /* Many locales (such as "C" or "hr_HR") don't specify
             grouping, which we still want to use it for legibility.

Reply via email to