commit 8c446da34998f6350911e07fbfd7932412c83185
author Mohamed Akram <mohd.akram@outlook.com> Sat Jul 13 18:49:55 2024 +0200
committer Christian Brabandt <cb@256bit.org> Sat Jul 13 18:49:55 2024 +0200
tree 7663c3ba00d13b238aac9060fcfab4774e9a75b0
parent c5e24ee24bc00981509170104dd0700fbc4669c0

patch 9.1.0574: ex: wrong handling of commands after bar

Problem:  ex: wrong handling of commands after bar
Solution: for :append, :insert and :change use the text after the bar
          as input for those commands. This is what POSIX requests.
          (Mohamed Akram)

See the POSIX Spec:
https://pubs.opengroup.org/onlinepubs/9699919799/utilities/ex.html#tag_20_40_13_03
Section 12.c

closes: #15229

Signed-off-by: Mohamed Akram <mohd.akram@outlook.com>
Signed-off-by: Christian Brabandt <cb@256bit.org>

runtime/doc/insert.txt

diff --git a/runtime/doc/insert.txt b/runtime/doc/insert.txt
index 08bf4f3..d48018f 100644
--- a/runtime/doc/insert.txt
+++ b/runtime/doc/insert.txt
@@ -1,4 +1,4 @@
-*insert.txt*    For Vim version 9.1.  Last change: 2024 Jul 12
+*insert.txt*    For Vim version 9.1.  Last change: 2024 Jul 13
 
 
 		  VIM REFERENCE MANUAL    by Bram Moolenaar
@@ -1995,6 +1995,16 @@
 containing only a ".".  Watch out for lines starting with a backslash, see
 |line-continuation|.
 
+Text typed after a "|" command separator is used first. So the following
+command in ex mode: >
+	:a|one
+	two
+	.
+	:visual
+<appends the following text, after the cursor line: >
+	one
+	two
+<
 NOTE: These commands cannot be used with |:global| or |:vglobal|.
 ":append" and ":insert" don't work properly in between ":if" and
 ":endif", ":for" and ":endfor", ":while" and ":endwhile".

src/ex_cmds.c

diff --git a/src/ex_cmds.c b/src/ex_cmds.c
index 8143c24..c365937 100644
--- a/src/ex_cmds.c
+++ b/src/ex_cmds.c
@@ -3360,7 +3360,13 @@
 		indent = get_indent_lnum(lnum);
 	}
 	ex_keep_indent = FALSE;
-	if (eap->ea_getline == NULL)
+	if (*eap->arg == '|')
+	{
+	    // Get the text after the trailing bar.
+	    theline = vim_strsave(eap->arg + 1);
+	    *eap->arg = NUL;
+	}
+	else if (eap->ea_getline == NULL)
 	{
 	    // No getline() function, use the lines that follow. This ends
 	    // when there is no more.

src/ex_docmd.c

diff --git a/src/ex_docmd.c b/src/ex_docmd.c
index 7ab1c69..614ba20 100644
--- a/src/ex_docmd.c
+++ b/src/ex_docmd.c
@@ -5401,7 +5401,11 @@
 		    && in_vim9script()
 		    && !(eap->argt & EX_NOTRLCOM)
 		    && p > eap->cmd && VIM_ISWHITE(p[-1]))
-		|| *p == '|' || *p == '\n')
+		|| (*p == '|'
+		    && eap->cmdidx != CMD_append
+		    && eap->cmdidx != CMD_change
+		    && eap->cmdidx != CMD_insert)
+		|| *p == '\n')
 	{
 	    /*
 	     * We remove the '\' before the '|', unless EX_CTRLV is used

src/testdir/test_ex_mode.vim

diff --git a/src/testdir/test_ex_mode.vim b/src/testdir/test_ex_mode.vim
index b0e5d81..c4f82d3 100644
--- a/src/testdir/test_ex_mode.vim
+++ b/src/testdir/test_ex_mode.vim
Binary files differ

src/version.c

diff --git a/src/version.c b/src/version.c
index a4ae10f..25e38ca 100644
--- a/src/version.c
+++ b/src/version.c
@@ -705,6 +705,8 @@
 static int included_patches[] =
 {   /* Add new patch number below this line */
 /**/
+    574,
+/**/
     573,
 /**/
     572,