# HG changeset patch # User Bram Moolenaar # Date 1617299102 -7200 # Node ID bbf4b31855549118c63d53a0fbdf0e7c87a8bde2 # Parent 09302e0fb4a5d129ad9160c6bbb6d23bf9df113e patch 8.2.2687: Vim9: cannot use "const" for global variable in :def function Commit: https://github.com/vim/vim/commit/d877a5700fe9d5b92562514924549316382384aa Author: Bram Moolenaar Date: Thu Apr 1 19:42:48 2021 +0200 patch 8.2.2687: Vim9: cannot use "const" for global variable in :def function Problem: Vim9: cannot use "const" for global variable in :def function. Solution: Do allow using :const for a global variable. (closes https://github.com/vim/vim/issues/8030) diff --git a/src/testdir/test_vim9_assign.vim b/src/testdir/test_vim9_assign.vim --- a/src/testdir/test_vim9_assign.vim +++ b/src/testdir/test_vim9_assign.vim @@ -1277,6 +1277,13 @@ def Test_var_declaration() g:FLIST[0] = 22 assert_equal([22], g:FLIST) + def SetGlobalConst() + const g:globConst = 123 + enddef + SetGlobalConst() + assert_equal(123, g:globConst) + assert_true(islocked('g:globConst')) + const w:FOO: number = 46 assert_equal(46, w:FOO) const w:FOOS = 'wfoos' @@ -1341,6 +1348,17 @@ def Test_var_declaration_fails() lines =<< trim END vim9script + def SetGlobalConst() + const g:globConst = 123 + enddef + SetGlobalConst() + g:globConst = 234 + END + CheckScriptFailure(lines, 'E741: Value is locked: globConst') + unlet g:globConst + + lines =<< trim END + vim9script const cdict: dict = {} def Change() cdict.foo = 'foo' 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 */ /**/ + 2687, +/**/ 2686, /**/ 2685, diff --git a/src/vim9compile.c b/src/vim9compile.c --- a/src/vim9compile.c +++ b/src/vim9compile.c @@ -5752,7 +5752,8 @@ compile_lhs( &lhs->lhs_opt_flags, &lhs->lhs_vimvaridx, &lhs->lhs_type, cctx) == FAIL) return FAIL; - if (lhs->lhs_dest != dest_local) + if (lhs->lhs_dest != dest_local + && cmdidx != CMD_const && cmdidx != CMD_final) { // Specific kind of variable recognized. declare_error = is_decl; @@ -6519,6 +6520,7 @@ compile_assignment(char_u *arg, exarg_T else { if (is_decl && cmdidx == CMD_const && (lhs.lhs_dest == dest_script + || lhs.lhs_dest == dest_global || lhs.lhs_dest == dest_local)) // ":const var": lock the value, but not referenced variables generate_LOCKCONST(cctx); diff --git a/src/vim9execute.c b/src/vim9execute.c --- a/src/vim9execute.c +++ b/src/vim9execute.c @@ -963,9 +963,12 @@ check_for_number(typval_T *tv) store_var(char_u *name, typval_T *tv) { funccal_entry_T entry; - + int flags = ASSIGN_DECL; + + if (tv->v_lock) + flags |= ASSIGN_CONST; save_funccal(&entry); - set_var_const(name, NULL, tv, FALSE, ASSIGN_DECL, 0); + set_var_const(name, NULL, tv, FALSE, flags, 0); restore_funccal(); }