Make sure monetary, numeric, and time locale categories are set to C and
authorPeter Eisentraut
Fri, 9 Aug 2002 22:52:04 +0000 (22:52 +0000)
committerPeter Eisentraut
Fri, 9 Aug 2002 22:52:04 +0000 (22:52 +0000)
are only activated temporarily to read out formatting information.

doc/src/sgml/runtime.sgml
src/backend/main/main.c
src/backend/utils/adt/pg_locale.c

index 8733725c2e32af444df5d3c9278d3b8995e71bd1..8061f74ecc2d383bb2984686e222d1736fe9b694 100644 (file)
@@ -1,5 +1,5 @@
 
 
 
@@ -1451,8 +1451,9 @@ dynamic_library_path = '/usr/local/lib/postgresql:/home/my_project/lib:$libdir'
       LC_MONETARY (string)
       
        
-        Sets the locale to use for formatting monetary amounts.
-        Acceptable values are system-dependent; see 
+        Sets the locale to use for formatting monetary amounts, for
+        example with the to_char() family of
+        functions.  Acceptable values are system-dependent; see 
         linkend="locale"> for more information.  If this variable is
         set to the empty string (which is the default) then the value
         is inherited from the execution environment of the server in a
@@ -1480,9 +1481,9 @@ dynamic_library_path = '/usr/local/lib/postgresql:/home/my_project/lib:$libdir'
       LC_TIME (string)
       
        
-        Sets the locale to use for formatting date and time values,
-        for example with the to_char() family of
-        functions. Acceptable values are system-dependent; see 
+        Sets the locale to use for formatting date and time values.
+        (Currently, this setting does nothing, but it may in the
+        future.)  Acceptable values are system-dependent; see 
         linkend="locale"> for more information.  If this variable is
         set to the empty string (which is the default) then the value
         is inherited from the execution environment of the server in a
index 9e1d44739ad62a0988799b7a192a6c8cc2b7e66a..595052a91ca2ad6866e91e284875fda1dd4f1bb5 100644 (file)
@@ -13,7 +13,7 @@
  *
  *
  * IDENTIFICATION
- *   $Header: /cvsroot/pgsql/src/backend/main/main.c,v 1.52 2002/06/20 20:29:29 momjian Exp $
+ *   $Header: /cvsroot/pgsql/src/backend/main/main.c,v 1.53 2002/08/09 22:52:04 petere Exp $
  *
  *-------------------------------------------------------------------------
  */
@@ -134,9 +134,11 @@ main(int argc, char *argv[])
 #ifdef LC_MESSAGES
    setlocale(LC_MESSAGES, "");
 #endif
-   setlocale(LC_MONETARY, "");
-   setlocale(LC_NUMERIC, "");
-   setlocale(LC_TIME, "");
+   /* We don't use these during startup.  See also pg_locale.c about
+    * why these are set to "C". */
+   setlocale(LC_MONETARY, "C");
+   setlocale(LC_NUMERIC, "C");
+   setlocale(LC_TIME, "C");
 
 #ifdef ENABLE_NLS
    bindtextdomain("postgres", LOCALEDIR);
index ba962ac8b152d177185db983fc94b872ee63fd45..33a7a105eda731e103cd9108b5407b4438bc3820 100644 (file)
@@ -2,13 +2,35 @@
  *
  * PostgreSQL locale utilities
  *
- * $Header: /cvsroot/pgsql/src/backend/utils/adt/pg_locale.c,v 1.17 2002/05/17 01:19:18 tgl Exp $
+ * $Header: /cvsroot/pgsql/src/backend/utils/adt/pg_locale.c,v 1.18 2002/08/09 22:52:04 petere Exp $
  *
  * Portions Copyright (c) 2002, PostgreSQL Global Development Group
  *
  *-----------------------------------------------------------------------
  */
 
+/*
+ * Here is how the locale stuff is handled: LC_COLLATE and LC_CTYPE
+ * are fixed by initdb, stored in pg_control, and cannot be changed.
+ * Thus, the effects of strcoll(), strxfrm(), isupper(), toupper(),
+ * etc. are always in the same fixed locale.
+ *
+ * LC_MESSAGES is settable at run time and will take effect
+ * immediately.
+ *
+ * The other categories, LC_MONETARY, LC_NUMERIC, and LC_TIME are also
+ * settable at run-time.  However, we don't actually set those locale
+ * categories permanently.  This would have bizzare effects like no
+ * longer accepting standard floating-point literals in some locales.
+ * Instead, we only set the locales briefly when needed, cache the
+ * required information obtained from localeconv(), and set them back.
+ * The information is only used by the formatting functions (to_char,
+ * etc.) and the money type.  For the user, this should all be
+ * transparent.  (Actually, LC_TIME doesn't do anything at all right
+ * now.)
+ */
+
+
 #include "postgres.h"
 
 #include 
 #include "utils/pg_locale.h"
 
 
+/* indicated whether locale information cache is valid */
+static bool CurrentLocaleConvValid = false;
+
+
 /* GUC storage area */
 
 char *locale_messages;
@@ -26,41 +52,33 @@ char *locale_time;
 
 /* GUC assign hooks */
 
+/*
+ * This is common code for several locale categories.  This doesn't
+ * actually set the locale permanently, it only tests if the locale is
+ * valid.  (See explanation at the top of this file.)
+ */
 static const char *
 locale_xxx_assign(int category, const char *value, bool doit, bool interactive)
 {
-   if (doit)
-   {
-       if (!setlocale(category, value))
-           return NULL;
-   }
-   else
-   {
-       char *save;
+   char *save;
 
-       save = setlocale(category, NULL);
-       if (!save)
-           return NULL;
+   save = setlocale(category, NULL);
+   if (!save)
+       return NULL;
 
-       if (!setlocale(category, value))
-           return NULL;
+   if (!setlocale(category, value))
+       return NULL;
 
-       setlocale(category, save);
-   }
-   return value;
-}
+   setlocale(category, save);
+
+   /* need to reload cache next time */
+   if (doit)
+       CurrentLocaleConvValid = false;
 
-const char *
-locale_messages_assign(const char *value, bool doit, bool interactive)
-{
-   /* LC_MESSAGES category does not exist everywhere, but accept it anyway */
-#ifdef LC_MESSAGES
-   return locale_xxx_assign(LC_MESSAGES, value, doit, interactive);
-#else
    return value;
-#endif
 }
 
+
 const char *
 locale_monetary_assign(const char *value, bool doit, bool interactive)
 {
@@ -80,6 +98,37 @@ locale_time_assign(const char *value, bool doit, bool interactive)
 }
 
 
+/*
+ * lc_messages takes effect immediately
+ */
+const char *
+locale_messages_assign(const char *value, bool doit, bool interactive)
+{
+   /* LC_MESSAGES category does not exist everywhere, but accept it anyway */
+#ifdef LC_MESSAGES
+   if (doit)
+   {
+       if (!setlocale(LC_MESSAGES, value))
+           return NULL;
+   }
+   else
+   {
+       char *save;
+
+       save = setlocale(LC_MESSAGES, NULL);
+       if (!save)
+           return NULL;
+       
+       if (!setlocale(LC_MESSAGES, value))
+           return NULL;
+       
+       setlocale(LC_MESSAGES, save);
+   }
+#endif
+   return value;
+}
+
+
 /*
  * We'd like to cache whether LC_COLLATE is C (or POSIX), so we can
  * optimize a few code paths in various places.
@@ -107,6 +156,39 @@ lc_collate_is_c(void)
 }
 
 
+/*
+ * Frees the malloced content of a struct lconv.  (But not the struct
+ * itself.)
+ */
+static void
+free_struct_lconv(struct lconv *s)
+{
+   if (s == NULL)
+       return;
+
+   if (s->currency_symbol)
+       free(s->currency_symbol);
+   if (s->decimal_point)
+       free(s->decimal_point);
+   if (s->grouping)
+       free(s->grouping);
+   if (s->thousands_sep)
+       free(s->thousands_sep);
+   if (s->int_curr_symbol)
+       free(s->int_curr_symbol);
+   if (s->mon_decimal_point)
+       free(s->mon_decimal_point);
+   if (s->mon_grouping)
+       free(s->mon_grouping);
+   if (s->mon_thousands_sep)
+       free(s->mon_thousands_sep);
+   if (s->negative_sign)
+       free(s->negative_sign);
+   if (s->positive_sign)
+       free(s->positive_sign);
+}
+
+
 /*
  * Return the POSIX lconv struct (contains number/money formatting
  * information) with locale information for all categories.
@@ -114,16 +196,24 @@ lc_collate_is_c(void)
 struct lconv *
 PGLC_localeconv(void)
 {
-   static bool CurrentLocaleConvValid = false;
    static struct lconv CurrentLocaleConv;
-
    struct lconv *extlconv;
+   char       *save_lc_monetary;
+   char       *save_lc_numeric;
 
    /* Did we do it already? */
    if (CurrentLocaleConvValid)
        return &CurrentLocaleConv;
 
-   /* Get formatting information for the external environment */
+   free_struct_lconv(&CurrentLocaleConv);
+
+   save_lc_monetary = setlocale(LC_MONETARY, NULL);
+   save_lc_numeric = setlocale(LC_NUMERIC, NULL);
+
+   setlocale(LC_MONETARY, locale_monetary);
+   setlocale(LC_NUMERIC, locale_numeric);
+
+   /* Get formatting information */
    extlconv = localeconv();
 
    /*
@@ -141,6 +231,10 @@ PGLC_localeconv(void)
    CurrentLocaleConv.mon_thousands_sep = strdup(extlconv->mon_thousands_sep);
    CurrentLocaleConv.negative_sign = strdup(extlconv->negative_sign);
    CurrentLocaleConv.positive_sign = strdup(extlconv->positive_sign);
+   CurrentLocaleConv.n_sign_posn = extlconv->n_sign_posn;
+
+   setlocale(LC_MONETARY, save_lc_monetary);
+   setlocale(LC_NUMERIC, save_lc_numeric);
 
    CurrentLocaleConvValid = true;
    return &CurrentLocaleConv;