changeset 24760:ca0f983f08cf v8.2.2918

patch 8.2.2918: builtin function can be shadowed by global variable Commit: https://github.com/vim/vim/commit/3d9c4eefe656ee8bf58c0496a48bd56bac180056 Author: Bram Moolenaar <Bram@vim.org> 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)
author Bram Moolenaar <Bram@vim.org>
date Mon, 31 May 2021 22:30:02 +0200
parents 7603f2a9f1aa
children 156525f3592b
files src/eval.c src/testdir/test_functions.vim src/version.c
diffstat 3 files changed, 17 insertions(+), 0 deletions(-) [+]
line wrap: on
line diff
--- 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);
--- 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
--- 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,