Patch 8.2.1293

[Bram Moolenaar]

Patch 8.2.1293
Problem: Vim9: :execute mixes up () expression and function call.
Solution: Do not skip white space when looking for the "(". (closes #6531)
Files: src/vim9compile.c, src/testdir/test_vim9_script.vim


*** ../vim-8.2.1292/src/vim9compile.c 2020-07-23 22:41:39.779937421 +0200
--- src/vim9compile.c 2020-07-25 19:18:14.995776840 +0200
***************
*** 3755,3761 ****
}
}

! if (*p == '(')
{
garray_T *stack = &cctx->ctx_type_stack;
type_T *type;
--- 3755,3763 ----
}
}

! // Do not skip over white space to find the "(", "exeucte 'x' ()" is
! // not a function call.
! if (**arg == '(')
{
garray_T *stack = &cctx->ctx_type_stack;
type_T *type;
*** ../vim-8.2.1292/src/testdir/test_vim9_script.vim 2020-07-25 15:41:04.408638663 +0200
--- src/testdir/test_vim9_script.vim 2020-07-25 19:27:17.304934434 +0200
***************
*** 1708,1713 ****
--- 1708,1717 ----
assert_equal('execute-var-var', getline(1))
bwipe!

+ let n = true
+ execute 'echomsg' (n ? '"true"' : '"no"')
+ assert_match('^true$', Screenline(&lines))
+
call CheckDefFailure(['execute xxx'], 'E1001:')
call CheckDefFailure(['execute "cmd"# comment'], 'E488:')
enddef
***************
*** 2634,2639 ****
--- 2638,2669 ----
quit
enddef

+ " test using a vim9script that is auto-loaded from an autocmd
+ def Test_vim9_autoload()
+ let lines =<< trim END
+ vim9script
+ def foo#test()
+ echomsg getreg('"')
+ enddef
+ END
+
+ mkdir('Xdir/autoload', 'p')
+ writefile(lines, 'Xdir/autoload/foo.vim')
+ let save_rtp = &rtp
+ exe 'set rtp^=' .. getcwd() .. '/Xdir'
+ augroup test
+ autocmd TextYankPost * call foo#test()
+ augroup END
+
+ normal Y
+
+ augroup test
+ autocmd!
+ augroup END
+ delete('Xdir', 'rf')
+ &rtp = save_rtp
+ enddef
+
" Keep this last, it messes up highlighting.
def Test_substitute_cmd()
new
*** ../vim-8.2.1292/src/version.c 2020-07-25 16:53:07.938737495 +0200
--- src/version.c 2020-07-25 17:35:40.996621160 +0200
***************
*** 756,757 ****
--- 756,759 ----
{ /* Add new patch number below this line */
+ /**/
+ 1293,
/**/

--
hundred-and-one symptoms of being an internet addict:
85. Choice between paying Compuserve bill and paying for kids education
is a no brainer -- although a bit painful for your kids.

/// Bram Moolenaar -- Br...@Moolenaar.net -- http://www.Moolenaar.net \\\
/// sponsor Vim, vote for features -- http://www.Vim.org/sponsor/ \\\
\\\ an exciting new programming language -- http://www.Zimbu.org ///
\\\ help me help AIDS victims -- http://ICCF-Holland.org ///

[Bram Moolenaar]

I wrote:

> Patch 8.2.1293
> Problem: Vim9: :execute mixes up () expression and function call.
> Solution: Do not skip white space when looking for the "(". (closes #6531)
> Files: src/vim9compile.c, src/testdir/test_vim9_script.vim

Patch 8.2.1293 and 8.2.1294 were mixed up a bit, sorry about that.


--
Ed's Radiator Shop: The Best Place in Town to Take a Leak.