aboutsummaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorJan Edmund Lazo <janedmundlazo@hotmail.com>2018-12-08 14:05:07 -0500
committerJan Edmund Lazo <jan.lazo@mail.utoronto.ca>2019-06-03 00:12:01 -0400
commit8de5d641691a0d7e29759c5f0fe54875acab124b (patch)
tree521a723fd4564407eecc1d8d6e3b25e3c8bd8468
parentf046f3a2396ea23e53e7307e8c143dc646f5273c (diff)
downloadrneovim-8de5d641691a0d7e29759c5f0fe54875acab124b.tar.gz
rneovim-8de5d641691a0d7e29759c5f0fe54875acab124b.tar.bz2
rneovim-8de5d641691a0d7e29759c5f0fe54875acab124b.zip
vim-patch:8.1.0569: execute() always resets display column to zero
Problem: Execute() always resets display column to zero. (Sha Liu) Solution: Don't reset it to zero, restore the previous value. (closes vim/vim#3669) https://github.com/vim/vim/commit/10ccaa17ec8b2be1132fd19059e1cd5fb5c902c4
-rw-r--r--src/nvim/eval.c5
-rw-r--r--src/nvim/testdir/test_execute_func.vim12
2 files changed, 17 insertions, 0 deletions
diff --git a/src/nvim/eval.c b/src/nvim/eval.c
index 1640729c94..f721d398c6 100644
--- a/src/nvim/eval.c
+++ b/src/nvim/eval.c
@@ -8331,6 +8331,7 @@ static void f_execute(typval_T *argvars, typval_T *rettv, FunPtr fptr)
const bool save_emsg_noredir = emsg_noredir;
const bool save_redir_off = redir_off;
garray_T *const save_capture_ga = capture_ga;
+ const int save_msg_col = msg_col;
if (check_secure()) {
return;
@@ -8358,6 +8359,7 @@ static void f_execute(typval_T *argvars, typval_T *rettv, FunPtr fptr)
ga_init(&capture_local, (int)sizeof(char), 80);
capture_ga = &capture_local;
redir_off = false;
+ msg_col = 0; // prevent leading spaces
if (argvars[0].v_type != VAR_LIST) {
do_cmdline_cmd(tv_get_string(&argvars[0]));
@@ -8376,6 +8378,9 @@ static void f_execute(typval_T *argvars, typval_T *rettv, FunPtr fptr)
emsg_silent = save_emsg_silent;
emsg_noredir = save_emsg_noredir;
redir_off = save_redir_off;
+ // "silent reg" or "silent echo x" leaves msg_col somewhere in the line.
+ // Put it back where it was, since nothing should have been written.
+ msg_col = save_msg_col;
ga_append(capture_ga, NUL);
rettv->v_type = VAR_STRING;
diff --git a/src/nvim/testdir/test_execute_func.vim b/src/nvim/testdir/test_execute_func.vim
index 6f61bede93..e474e0ce36 100644
--- a/src/nvim/testdir/test_execute_func.vim
+++ b/src/nvim/testdir/test_execute_func.vim
@@ -53,3 +53,15 @@ func Test_execute_list()
call assert_equal("", execute([]))
call assert_equal("", execute(v:_null_list))
endfunc
+
+func Test_execute_does_not_change_col()
+ echo ''
+ echon 'abcd'
+ let x = execute('silent echo 234343')
+ echon 'xyz'
+ let text = ''
+ for col in range(1, 7)
+ let text .= nr2char(screenchar(&lines, col))
+ endfor
+ call assert_equal('abcdxyz', text)
+endfunc