# HG changeset patch # User Bram Moolenaar # Date 1622493002 -7200 # Node ID ca0f983f08cf5217e9eef4e90099c5732e783841 # Parent 7603f2a9f1aa057273d20243f633c3c25fb0b0e9 patch 8.2.2918: builtin function can be shadowed by global variable Commit: https://github.com/vim/vim/commit/3d9c4eefe656ee8bf58c0496a48bd56bac180056 Author: Bram Moolenaar Date: Mon May 31 22:15:26 2021 +0200 patch 8.2.2918: builtin function can be shadowed by global variable Problem: Builtin function can be shadowed by global variable. Solution: Check for builtin function before variable. (Yasuhiro Matsumoto, closes #8302) diff --git a/src/eval.c b/src/eval.c --- a/src/eval.c +++ b/src/eval.c @@ -1462,6 +1462,13 @@ set_var_lval( semsg(_(e_dictkey), lp->ll_newkey); return; } + if ((lp->ll_tv->vval.v_dict == get_globvar_dict() + || lp->ll_tv->vval.v_dict == + &SCRIPT_ITEM(current_sctx.sc_sid)->sn_vars->sv_dict) + && (rettv->v_type == VAR_FUNC + || rettv->v_type == VAR_PARTIAL) + && var_wrong_func_name(lp->ll_newkey, TRUE)) + return; // Need to add an item to the Dictionary. di = dictitem_alloc(lp->ll_newkey); diff --git a/src/testdir/test_functions.vim b/src/testdir/test_functions.vim --- a/src/testdir/test_functions.vim +++ b/src/testdir/test_functions.vim @@ -2681,4 +2681,12 @@ func Test_gettext() call assert_fails('call gettext(1)', 'E475:') endfunc +func Test_builtin_check() + call assert_fails('let g:["trim"] = {x -> " " .. x}', 'E704:') + call assert_fails('let g:.trim = {x -> " " .. x}', 'E704:') + call assert_fails('let s:["trim"] = {x -> " " .. x}', 'E704:') + call assert_fails('let s:.trim = {x -> " " .. x}', 'E704:') +endfunc + + " vim: shiftwidth=2 sts=2 expandtab 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 */ /**/ + 2918, +/**/ 2917, /**/ 2916,