From: Bram Moolenaar <Bram@vim.org>
Date: Mon, 5 Nov 2018 19:25:52 +0000 (+0100)
Subject: patch 8.1.0512: 'helplang' default is inconsistent for C and C.UTF-8
X-Git-Tag: v8.1.0512
X-Git-Url: https://granicus.if.org/sourcecode?a=commitdiff_plain;h=389ab7122bec99c11ad4ce6d87cc6f38a21e4e40;p=vim

patch 8.1.0512: 'helplang' default is inconsistent for C and C.UTF-8

Problem:    'helplang' default is inconsistent for C and C.UTF-8.
Solution:   Don't accept a value unless it starts with two letters.
---

diff --git a/src/ex_cmds2.c b/src/ex_cmds2.c
index 9a23f8205..3a4bce8a2 100644
--- a/src/ex_cmds2.c
+++ b/src/ex_cmds2.c
@@ -5358,6 +5358,16 @@ gettext_lang(char_u *name)
 #endif
 
 #if defined(FEAT_MULTI_LANG) || defined(PROTO)
+/*
+ * Return TRUE when "lang" starts with a valid language name.
+ * Rejects NULL, empty string, "C", "C.UTF-8" and others.
+ */
+    static int
+is_valid_mess_lang(char_u *lang)
+{
+    return lang != NULL && ASCII_ISALPHA(lang[0]) && ASCII_ISALPHA(lang[1]);
+}
+
 /*
  * Obtain the current messages language.  Used to set the default for
  * 'helplang'.  May return NULL or an empty string.
@@ -5379,17 +5389,17 @@ get_mess_lang(void)
 #  endif
 # else
     p = mch_getenv((char_u *)"LC_ALL");
-    if (p == NULL || *p == NUL)
+    if (!is_valid_mess_lang(p))
     {
 	p = mch_getenv((char_u *)"LC_MESSAGES");
-	if (p == NULL || *p == NUL)
+	if (!is_valid_mess_lang(p))
 	    p = mch_getenv((char_u *)"LANG");
     }
 # endif
 # ifdef WIN32
     p = gettext_lang(p);
 # endif
-    return p;
+    return is_valid_mess_lang(p) ? p : NULL;
 }
 #endif
 
diff --git a/src/version.c b/src/version.c
index 8440ce940..363970231 100644
--- a/src/version.c
+++ b/src/version.c
@@ -792,6 +792,8 @@ static char *(features[]) =
 
 static int included_patches[] =
 {   /* Add new patch number below this line */
+/**/
+    512,
 /**/
     511,
 /**/