* libc/locale/nl_langinfo.c (nl_langinfo): Return "CP932" when using

	SJIS on Cygwin.  Explain why.

diff --git a/newlib/ChangeLog b/newlib/ChangeLog
index 187978c..5b21f2b 100644 (file)
--- a/newlib/ChangeLog
+++ b/newlib/ChangeLog
@@ -1,3 +1,8 @@
+2010-01-24  Corinna Vinschen  <corinna@vinschen.de>
+
+       * libc/locale/nl_langinfo.c (nl_langinfo): Return "CP932" when using
+       SJIS on Cygwin.  Explain why.
+
 2010-01-23  Corinna Vinschen  <corinna@vinschen.de>
 
        * libc/locale/locale.c (loadlocale): Allow CP932 as alias for SJIS.

diff --git a/newlib/libc/locale/nl_langinfo.c b/newlib/libc/locale/nl_langinfo.c
index ae7228c..cd86c15 100644 (file)
--- a/newlib/libc/locale/nl_langinfo.c
+++ b/newlib/libc/locale/nl_langinfo.c
@@ -79,6 +79,20 @@ _DEFUN(nl_langinfo, (item),
                    else if (strcmp (ret + 2, "21866") == 0)
                      ret = "KOI8-U";
                  }
+               else if (ret[0] == 'S'/*JIS*/)
+                 {
+                   /* Cygwin uses MSFT's implementation of SJIS, which differs
+                      in some codepoints from the real thing, especially
+                      0x5c: yen sign instead of backslash,
+                      0x7e: overline instead of tilde.
+                      We can't use the real SJIS since otherwise Win32
+                      pathnames would become invalid.  OTOH, if we return
+                      "SJIS" here, then libiconv will do mb<->wc conversion
+                      differently to our internal functions.  Therefore we
+                      return what we really implement, CP932.  This is handled
+                      fine by libiconv. */
+                   ret = "CP932";
+                 }
 #else
                ret = "";
                if ((s = setlocale(LC_CTYPE, NULL)) != NULL) {