From 419a40ac9657e39646b2e0f3f71d7736b0c459d1 Mon Sep 17 00:00:00 2001
From: Bram Moolenaar <Bram@vim.org>
Date: Mon, 21 Jun 2021 21:55:18 +0200
Subject: patch 8.2.3033: no error when using alpha delimiter with :global

Problem:    No error when using alpha delimiter with :global.
Solution:   Check the delimiter like with :substitute. (closes #8415)
---
 src/ex_cmds.c               | 21 +++++++++++++++++----
 src/testdir/test_global.vim |  4 ++++
 src/version.c               |  2 ++
 3 files changed, 23 insertions(+), 4 deletions(-)

(limited to 'src')

diff --git a/src/ex_cmds.c b/src/ex_cmds.c
index 614d41d69..d8c5a12c7 100644
--- a/src/ex_cmds.c
+++ b/src/ex_cmds.c
@@ -3643,6 +3643,17 @@ skip_substitute(char_u *start, int delimiter)
     return p;
 }
 
+    static int
+check_regexp_delim(int c)
+{
+    if (isalpha(c))
+    {
+	emsg(_("E146: Regular expressions can't be delimited by letters"));
+	return FAIL;
+    }
+    return OK;
+}
+
 /*
  * Perform a substitution from line eap->line1 to line eap->line2 using the
  * command pointed to by eap->arg which should be of the form:
@@ -3705,11 +3716,9 @@ ex_substitute(exarg_T *eap)
 		&& vim_strchr((char_u *)"0123456789cegriIp|\"", *cmd) == NULL)
     {
 				// don't accept alphanumeric for separator
-	if (isalpha(*cmd))
-	{
-	    emsg(_("E146: Regular expressions can't be delimited by letters"));
+	if (check_regexp_delim(*cmd) == FAIL)
 	    return;
-	}
+
 	/*
 	 * undocumented vi feature:
 	 *  "\/sub/" and "\?sub?" use last used search pattern (almost like
@@ -4909,6 +4918,10 @@ ex_global(exarg_T *eap)
 	emsg(_("E148: Regular expression missing from global"));
 	return;
     }
+    else if (check_regexp_delim(*cmd) == FAIL)
+    {
+	return;
+    }
     else
     {
 	delim = *cmd;		// get the delimiter
diff --git a/src/testdir/test_global.vim b/src/testdir/test_global.vim
index 4a01be419..48753cfcc 100644
--- a/src/testdir/test_global.vim
+++ b/src/testdir/test_global.vim
@@ -83,4 +83,8 @@ func Test_global_newline()
   close!
 endfunc
 
+func Test_wrong_delimiter()
+  call assert_fails('g x^bxd', 'E146:')
+endfunc
+
 " vim: shiftwidth=2 sts=2 expandtab
diff --git a/src/version.c b/src/version.c
index a6e483e05..d8d033a2e 100644
--- a/src/version.c
+++ b/src/version.c
@@ -755,6 +755,8 @@ static char *(features[]) =
 
 static int included_patches[] =
 {   /* Add new patch number below this line */
+/**/
+    3033,
 /**/
     3032,
 /**/
-- 
cgit v1.2.1