On Windows, call bind_textdomain_codeset on domains other than the default one,
authorAlvaro Herrera
Sun, 8 Mar 2009 16:07:12 +0000 (16:07 +0000)
committerAlvaro Herrera
Sun, 8 Mar 2009 16:07:12 +0000 (16:07 +0000)
too, so that the codeset is properly mapped on the newly added PL domains.

src/backend/utils/init/miscinit.c
src/backend/utils/mb/mbutils.c
src/include/mb/pg_wchar.h

index b4b87b6f10afabfb9e8015ce69f0ff89be4021b8..30ecbd3779ef923bd76c548a63c8715586472aac 100644 (file)
@@ -8,7 +8,7 @@
  *
  *
  * IDENTIFICATION
- *   $PostgreSQL: pgsql/src/backend/utils/init/miscinit.c,v 1.172 2009/01/05 13:57:12 tgl Exp $
+ *   $PostgreSQL: pgsql/src/backend/utils/init/miscinit.c,v 1.173 2009/03/08 16:07:12 alvherre Exp $
  *
  *-------------------------------------------------------------------------
  */
@@ -30,6 +30,7 @@
 #endif
 
 #include "catalog/pg_authid.h"
+#include "mb/pg_wchar.h"
 #include "miscadmin.h"
 #include "postmaster/autovacuum.h"
 #include "storage/fd.h"
@@ -1241,6 +1242,7 @@ pg_bindtextdomain(const char *domain)
 
        get_locale_path(my_exec_path, locale_path);
        bindtextdomain(domain, locale_path);
+       pg_bind_textdomain_codeset(domain, GetDatabaseEncoding());
    }
 #endif
 }
index f5ba80d101d5e6f40f4cc9ff772a0261c04f4d39..cf6e5d0824f1955d8331a40f2cb1381502fdb105 100644 (file)
@@ -4,7 +4,7 @@
  * (currently mule internal code (mic) is used)
  * Tatsuo Ishii
  *
- * $PostgreSQL: pgsql/src/backend/utils/mb/mbutils.c,v 1.79 2009/03/02 15:10:09 teodor Exp $
+ * $PostgreSQL: pgsql/src/backend/utils/mb/mbutils.c,v 1.80 2009/03/08 16:07:12 alvherre Exp $
  */
 #include "postgres.h"
 
@@ -891,22 +891,27 @@ SetDatabaseEncoding(int encoding)
    DatabaseEncoding = &pg_enc2name_tbl[encoding];
    Assert(DatabaseEncoding->encoding == encoding);
 
-   /*
-    * On Windows, we need to explicitly bind gettext to the correct
-    * encoding, because gettext() tends to get confused.
-    */
+   pg_bind_textdomain_codeset(textdomain(NULL), encoding);
+}
+
+/*
+ * On Windows, we need to explicitly bind gettext to the correct
+ * encoding, because gettext() tends to get confused.
+ */
+void
+pg_bind_textdomain_codeset(const char *domainname, int encoding)
+{
 #if defined(ENABLE_NLS) && defined(WIN32)
-   {
-       int i;
+   int     i;
 
-       for (i = 0; i < sizeof(codeset_map_array) / sizeof(codeset_map_array[0]); i++)
+   for (i = 0; i < lengthof(codeset_map_array); i++)
+   {
+       if (codeset_map_array[i].encoding == encoding)
        {
-           if (codeset_map_array[i].encoding == encoding)
-           {
-               if (bind_textdomain_codeset(textdomain(NULL), codeset_map_array[i].codeset) == NULL)
-                   elog(LOG, "bind_textdomain_codeset failed");
-               break;
-           }
+           if (bind_textdomain_codeset(domainname,
+                                       codeset_map_array[i].codeset) == NULL)
+               elog(LOG, "bind_textdomain_codeset failed");
+           break;
        }
    }
 #endif
index 12820d4e20b9f553b5d27f7ef766c84727a2cb99..9d37191eabaeb72928c3b3562d3657cd13f28940 100644 (file)
@@ -6,7 +6,7 @@
  * Portions Copyright (c) 1996-2009, PostgreSQL Global Development Group
  * Portions Copyright (c) 1994, Regents of the University of California
  *
- * $PostgreSQL: pgsql/src/include/mb/pg_wchar.h,v 1.84 2009/02/10 19:29:39 petere Exp $
+ * $PostgreSQL: pgsql/src/include/mb/pg_wchar.h,v 1.85 2009/03/08 16:07:12 alvherre Exp $
  *
  * NOTES
  *     This is used both by the backend and by libpq, but should not be
@@ -392,6 +392,7 @@ extern const char *pg_get_client_encoding_name(void);
 extern void SetDatabaseEncoding(int encoding);
 extern int GetDatabaseEncoding(void);
 extern const char *GetDatabaseEncodingName(void);
+extern void pg_bind_textdomain_codeset(const char *domainname, int encoding);
 
 extern int pg_valid_client_encoding(const char *name);
 extern int pg_valid_server_encoding(const char *name);