summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorBram Moolenaar <Bram@vim.org>2020-08-16 18:29:35 +0200
committerBram Moolenaar <Bram@vim.org>2020-08-16 18:29:35 +0200
commite5abf7af08ff69e7e038c067497f080d4e44332c (patch)
tree7560953e5984d2fcffa9d32a401e0d6b491e9184
parentcc673e746ab98566556ff964d7a76f2fb46d7f84 (diff)
downloadvim-git-e5abf7af08ff69e7e038c067497f080d4e44332c.tar.gz
patch 8.2.1467: Vim9: :echomsg doesn't like a dict argumentv8.2.1467
Problem: Vim9: :echomsg doesn't like a dict argument. Solution: Convert arguments like in legacy script. (closes #6717)
-rw-r--r--src/testdir/test_vim9_script.vim3
-rw-r--r--src/version.c2
-rw-r--r--src/vim9compile.c7
-rw-r--r--src/vim9execute.c20
4 files changed, 22 insertions, 10 deletions
diff --git a/src/testdir/test_vim9_script.vim b/src/testdir/test_vim9_script.vim
index 3b8264fba..905deb6de 100644
--- a/src/testdir/test_vim9_script.vim
+++ b/src/testdir/test_vim9_script.vim
@@ -2102,6 +2102,9 @@ def Test_execute_cmd()
execute 'echomsg' (n ? '"true"' : '"no"')
assert_match('^true$', Screenline(&lines))
+ echomsg [1, 2, 3] #{a: 1, b: 2}
+ assert_match('^\[1, 2, 3\] {''a'': 1, ''b'': 2}$', Screenline(&lines))
+
call CheckDefFailure(['execute xxx'], 'E1001:')
call CheckDefFailure(['execute "cmd"# comment'], 'E488:')
enddef
diff --git a/src/version.c b/src/version.c
index 66506b22c..c8794ce92 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 */
/**/
+ 1467,
+/**/
1466,
/**/
1465,
diff --git a/src/vim9compile.c b/src/vim9compile.c
index 579ccbc78..3e0ad9a3e 100644
--- a/src/vim9compile.c
+++ b/src/vim9compile.c
@@ -2422,9 +2422,6 @@ compile_list(char_u **arg, cctx_T *cctx)
if (*p == ']')
{
++p;
- // Allow for following comment, after at least one space.
- if (VIM_ISWHITE(*p) && *skipwhite(p) == '#')
- p += STRLEN(p);
break;
}
if (compile_expr0(&p, cctx) == FAIL)
@@ -6206,6 +6203,7 @@ compile_throw(char_u *arg, cctx_T *cctx UNUSED)
compile_mult_expr(char_u *arg, int cmdidx, cctx_T *cctx)
{
char_u *p = arg;
+ char_u *prev;
int count = 0;
for (;;)
@@ -6213,8 +6211,9 @@ compile_mult_expr(char_u *arg, int cmdidx, cctx_T *cctx)
if (compile_expr0(&p, cctx) == FAIL)
return NULL;
++count;
+ prev = p;
p = skipwhite(p);
- if (ends_excmd(*p))
+ if (ends_excmd2(prev, p))
break;
}
diff --git a/src/vim9execute.c b/src/vim9execute.c
index f8156c8a1..059e768e8 100644
--- a/src/vim9execute.c
+++ b/src/vim9execute.c
@@ -1028,14 +1028,20 @@ call_def_function(
for (idx = 0; idx < count; ++idx)
{
tv = STACK_TV_BOT(idx - count);
- if (tv->v_type == VAR_CHANNEL || tv->v_type == VAR_JOB)
+ if (iptr->isn_type == ISN_EXECUTE)
{
- SOURCING_LNUM = iptr->isn_lnum;
- emsg(_(e_inval_string));
- break;
+ if (tv->v_type == VAR_CHANNEL
+ || tv->v_type == VAR_JOB)
+ {
+ SOURCING_LNUM = iptr->isn_lnum;
+ emsg(_(e_inval_string));
+ break;
+ }
+ else
+ p = tv_get_string_buf(tv, buf);
}
else
- p = tv_get_string_buf(tv, buf);
+ p = tv_stringify(tv, buf);
len = (int)STRLEN(p);
if (ga_grow(&ga, len + 2) == FAIL)
@@ -1050,8 +1056,10 @@ call_def_function(
clear_tv(tv);
}
ectx.ec_stack.ga_len -= count;
+ if (failed)
+ goto on_error;
- if (!failed && ga.ga_data != NULL)
+ if (ga.ga_data != NULL)
{
if (iptr->isn_type == ISN_EXECUTE)
do_cmdline_cmd((char_u *)ga.ga_data);