]> granicus.if.org Git - vim/commitdiff
patch 8.2.1757: Mac: default locale is lacking the encoding v8.2.1757
authorBram Moolenaar <Bram@vim.org>
Sun, 27 Sep 2020 14:03:15 +0000 (16:03 +0200)
committerBram Moolenaar <Bram@vim.org>
Sun, 27 Sep 2020 14:03:15 +0000 (16:03 +0200)
Problem:    Mac: default locale is lacking the encoding.
Solution:   Add ".UTF-8 to the locale. (Yee Cheng Chin, closes #7022)

src/os_mac_conv.c
src/testdir/test_environ.vim
src/version.c

index fd62124644c7b5601b539f0432c25abff0cf8c24..734f8f6c9f0d15007d62998b3885923dfad1786b 100644 (file)
@@ -570,12 +570,17 @@ mac_lang_init(void)
 {
     if (mch_getenv((char_u *)"LANG") == NULL)
     {
-       char    buf[20];
+       char    buf[50];
+
+       // $LANG is not set, either because it was unset or Vim was started
+       // from the Dock.  Query the system locale.
        if (LocaleRefGetPartString(NULL,
                    kLocaleLanguageMask | kLocaleLanguageVariantMask |
                    kLocaleRegionMask | kLocaleRegionVariantMask,
-                   sizeof buf, buf) == noErr && *buf)
+                   sizeof(buf) - 10, buf) == noErr && *buf)
        {
+           if (strcasestr(buf, "utf-8") == NULL)
+               strcat(buf, ".UTF-8");
            vim_setenv((char_u *)"LANG", (char_u *)buf);
 #   ifdef HAVE_LOCALE_H
            setlocale(LC_ALL, "");
index d9f99f057f1adcc234faeb919d1ffc375b7a0f16..dd34983ee5a8a7fa37e4aa4b1eb9ee41dd215f37 100644 (file)
@@ -2,6 +2,8 @@
 
 scriptencoding utf-8
 
+source check.vim
+
 func Test_environ()
   unlet! $TESTENV
   call assert_equal(0, has_key(environ(), 'TESTENV'))
@@ -45,4 +47,23 @@ func Test_external_env()
   call assert_equal('', result)
 endfunc
 
+func Test_mac_locale()
+  CheckFeature osxdarwin
+
+  " If $LANG is not set then the system locale will be used.
+  " Run Vim after unsetting all the locale environmental vars, and capture the
+  " output of :lang.
+  let lang_results = system("unset LANG; unset LC_MESSAGES; " ..
+            \ shellescape(v:progpath) ..
+            \ " --clean -esX -c 'redir @a' -c 'lang' -c 'put a' -c 'print' -c 'qa!' ")
+
+  " Check that:
+  " 1. The locale is the form of <locale>.UTF-8.
+  " 2. Check that fourth item (LC_NUMERIC) is properly set to "C".
+  " Example match: "en_US.UTF-8/en_US.UTF-8/en_US.UTF-8/C/en_US.UTF-8/en_US.UTF-8"
+  call assert_match('"\([a-zA-Z_]\+\.UTF-8/\)\{3}C\(/[a-zA-Z_]\+\.UTF-8\)\{2}"',
+        \ lang_results,
+        \ "Default locale should have UTF-8 encoding set, and LC_NUMERIC set to 'C'")
+endfunc
+
 " vim: shiftwidth=2 sts=2 expandtab
index a0bea6ff5511afaff61335494f2e8498871904a3..9ef77d87a0d5eb854b3e90fa54d481fd572d888b 100644 (file)
@@ -750,6 +750,8 @@ static char *(features[]) =
 
 static int included_patches[] =
 {   /* Add new patch number below this line */
+/**/
+    1757,
 /**/
     1756,
 /**/