# HG changeset patch # User Bram Moolenaar # Date 1617982204 -7200 # Node ID bed5ec739304726ed0103625081138ef93286ffa # Parent 9d63465bc5073ef565e157c5a484506c4d82a091 patch 8.2.2739: Vim9: a lambda accepts too many arguments at the script level Commit: https://github.com/vim/vim/commit/767034c5b82ba8999d9fed2f997436e6e3e99419 Author: Bram Moolenaar Date: Fri Apr 9 17:24:52 2021 +0200 patch 8.2.2739: Vim9: a lambda accepts too many arguments at the script level Problem: Vim9: a lambda accepts too many arguments at the script level. Solution: Do not set uf_varargs in Vim9 script. diff --git a/src/testdir/test_vim9_func.vim b/src/testdir/test_vim9_func.vim --- a/src/testdir/test_vim9_func.vim +++ b/src/testdir/test_vim9_func.vim @@ -836,6 +836,11 @@ def Test_call_lambda_args() enddef END CheckDefFailure(lines, 'E1167:') + + lines =<< trim END + echo ((a) => a)('aa', 'bb') + END + CheckDefAndScriptFailure(lines, 'E118:', 1) enddef def FilterWithCond(x: string, Cond: func(string): bool): bool @@ -2114,7 +2119,7 @@ def Test_list_lambda() ->substitute("('", ' ', '') ->substitute("')", '', '') ->substitute('function\zs', ' ', '')) - assert_match('def \d\+(_: any, ...): number\n1 return 0\n enddef', body) + assert_match('def \d\+(_: any): number\n1 return 0\n enddef', body) enddef def DoFilterThis(a: string): list @@ -2363,7 +2368,7 @@ def Test_did_emsg_reset() vim9script au BufWinLeave * # def Func() - popup_menu('', {callback: () => popup_create('', {})->popup_close()}) + popup_menu('', {callback: (a, b) => popup_create('', {})->popup_close()}) eval [][0] enddef nno call Func() diff --git a/src/testdir/test_vim9_script.vim b/src/testdir/test_vim9_script.vim --- a/src/testdir/test_vim9_script.vim +++ b/src/testdir/test_vim9_script.vim @@ -3518,7 +3518,7 @@ func Test_no_redraw_when_restoring_cpo() vim9script set cpo+=M exe 'set rtp^=' .. getcwd() .. '/Xdir' - au CmdlineEnter : ++once timer_start(0, () => script#func()) + au CmdlineEnter : ++once timer_start(0, (_) => script#func()) setline(1, 'some text') END call writefile(lines, 'XTest_redraw_cpo') diff --git a/src/userfunc.c b/src/userfunc.c --- a/src/userfunc.c +++ b/src/userfunc.c @@ -1263,8 +1263,9 @@ get_lambda_tv( #endif if (sandbox) flags |= FC_SANDBOX; - // can be called with more args than uf_args.ga_len - fp->uf_varargs = TRUE; + // In legacy script a lambda can be called with more args than + // uf_args.ga_len. + fp->uf_varargs = !in_vim9script(); fp->uf_flags = flags; fp->uf_calls = 0; fp->uf_script_ctx = current_sctx; diff --git a/src/version.c b/src/version.c --- a/src/version.c +++ b/src/version.c @@ -751,6 +751,8 @@ static char *(features[]) = static int included_patches[] = { /* Add new patch number below this line */ /**/ + 2739, +/**/ 2738, /**/ 2737,