diff options
author | Bram Moolenaar <Bram@vim.org> | 2018-12-08 13:57:42 +0100 |
---|---|---|
committer | Bram Moolenaar <Bram@vim.org> | 2018-12-08 13:57:42 +0100 |
commit | 446e7a3cd36b2de7d559f167eb5795d1e1cd3ddb (patch) | |
tree | f520c88e0b036da263e02eca6fc60e1ccaa41e9c /src | |
parent | 539328197c51c1586cbbb6b6be3db3c412249b49 (diff) | |
download | vim-git-446e7a3cd36b2de7d559f167eb5795d1e1cd3ddb.tar.gz |
patch 8.1.0571: non-silent execute() resets display column to zerov8.1.0571
Problem: Non-silent execute() resets display column to zero.
Solution: Keep the display column as-is.
Diffstat (limited to 'src')
-rw-r--r-- | src/evalfunc.c | 16 | ||||
-rw-r--r-- | src/testdir/test_execute_func.vim | 17 | ||||
-rw-r--r-- | src/version.c | 2 |
3 files changed, 32 insertions, 3 deletions
diff --git a/src/evalfunc.c b/src/evalfunc.c index 51e069981..d3b2da7c4 100644 --- a/src/evalfunc.c +++ b/src/evalfunc.c @@ -3263,6 +3263,7 @@ f_execute(typval_T *argvars, typval_T *rettv) int save_redir_off = redir_off; garray_T save_ga; int save_msg_col = msg_col; + int echo_output = FALSE; rettv->vval.v_string = NULL; rettv->v_type = VAR_STRING; @@ -3289,6 +3290,8 @@ f_execute(typval_T *argvars, typval_T *rettv) if (s == NULL) return; + if (*s == NUL) + echo_output = TRUE; if (STRNCMP(s, "silent", 6) == 0) ++msg_silent; if (STRCMP(s, "silent!") == 0) @@ -3305,7 +3308,8 @@ f_execute(typval_T *argvars, typval_T *rettv) ga_init2(&redir_execute_ga, (int)sizeof(char), 500); redir_execute = TRUE; redir_off = FALSE; - msg_col = 0; // prevent leading spaces + if (!echo_output) + msg_col = 0; // prevent leading spaces if (cmd != NULL) do_cmdline_cmd(cmd); @@ -3339,8 +3343,14 @@ f_execute(typval_T *argvars, typval_T *rettv) 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; + if (echo_output) + // When not working silently: put it in column zero. A following + // "echon" will overwrite the message, unavoidably. + msg_col = 0; + else + // When working silently: Put it back where it was, since nothing + // should have been written. + msg_col = save_msg_col; } /* diff --git a/src/testdir/test_execute_func.vim b/src/testdir/test_execute_func.vim index 7b3cf21a5..b84fbebe5 100644 --- a/src/testdir/test_execute_func.vim +++ b/src/testdir/test_execute_func.vim @@ -61,3 +61,20 @@ func Test_execute_does_not_change_col() endfor call assert_equal('abcdxyz', text) endfunc + +func Test_execute_not_silent() + echo '' + echon 'abcd' + let x = execute('echon 234', '') + echo 'xyz' + let text1 = '' + for col in range(1, 8) + let text1 .= nr2char(screenchar(&lines - 1, col)) + endfor + call assert_equal('abcd234 ', text1) + let text2 = '' + for col in range(1, 4) + let text2 .= nr2char(screenchar(&lines, col)) + endfor + call assert_equal('xyz ', text2) +endfunc diff --git a/src/version.c b/src/version.c index 8df3ac733..049f34b81 100644 --- a/src/version.c +++ b/src/version.c @@ -793,6 +793,8 @@ static char *(features[]) = static int included_patches[] = { /* Add new patch number below this line */ /**/ + 571, +/**/ 570, /**/ 569, |