patch 8.2.4509: Vim9: can declare a variable with ":va"

Problem:    Vim9: can declare a variable with ":va".
Solution:   Disallow using ":va", require using ":var".

diff --git a/src/errors.h b/src/errors.h
index 6f25af6afc38e826a2f1d2c40e3684a4dd74a99d..82708d3bfa9af11fa15b34dbaa04f3774597869c 100644 (file)
--- a/src/errors.h
+++ b/src/errors.h
@@ -2762,7 +2762,8 @@ EXTERN char e_type_mismatch_for_v_variable[]
 EXTERN char e_yank_register_changed_while_using_it[]
        INIT(= N_("E1064: Yank register changed while using it"));
 #ifdef FEAT_EVAL
-// E1065 unused
+EXTERN char e_must_use_var_instead_of_va[]
+       INIT(= N_("E1065: Must use :var instead of :va"));
 EXTERN char e_cannot_declare_a_register_str[]
        INIT(= N_("E1066: Cannot declare a register: %s"));
 EXTERN char e_separator_mismatch_str[]

diff --git a/src/evalvars.c b/src/evalvars.c
index 40e643b9f37e20f8e5dd4fbfb1b5a181bc68340b..56ac3daa7e7abf66c61c4468b2c0d83e2a8f71d5 100644 (file)
--- a/src/evalvars.c
+++ b/src/evalvars.c
@@ -755,17 +755,24 @@ heredoc_get(exarg_T *eap, char_u *cmd, int script_get)
 ex_var(exarg_T *eap)
 {
     char_u *p = eap->cmd;
+    int            has_var;
 
     if (!in_vim9script())
     {
        semsg(_(e_str_cannot_be_used_in_legacy_vim_script), ":var");
        return;
     }
-    if (current_sctx.sc_sid == 0 && checkforcmd_noparen(&p, "var", 3))
+    has_var = checkforcmd_noparen(&p, "var", 3);
+    if (current_sctx.sc_sid == 0 && has_var)
     {
        emsg(_(e_cannot_declare_variable_on_command_line));
        return;
     }
+    if (eap->arg > eap->cmd && !has_var)
+    {
+       emsg(_(e_must_use_var_instead_of_va));
+       return;
+    }
     ex_let(eap);
 }
 

diff --git a/src/testdir/test_vim9_assign.vim b/src/testdir/test_vim9_assign.vim
index ccf2a01a0d7a6286d7830bdc8fb413132f16a5ab..6f6433493b0d533b3aa11df875c38f6a4d5dd948 100644 (file)
--- a/src/testdir/test_vim9_assign.vim
+++ b/src/testdir/test_vim9_assign.vim
@@ -1965,6 +1965,11 @@ def Test_var_declaration_fails()
   v9.CheckDefFailure(['var foo.bar = 2'], 'E1087:')
   v9.CheckDefFailure(['var foo[3] = 2'], 'E1087:')
   v9.CheckDefFailure(['const foo: number'], 'E1021:')
+
+  lines =<< trim END
+      va foo = 123
+  END
+  v9.CheckDefAndScriptFailure(lines, 'E1065:', 1)
 enddef
 
 def Test_var_declaration_inferred()

diff --git a/src/version.c b/src/version.c
index 5bbd307c75d17dfb679bec65979a253f1dc2a7e5..999f8ac431b832f9f2134f813e31548dbde938b6 100644 (file)
--- a/src/version.c
+++ b/src/version.c
@@ -754,6 +754,8 @@ static char *(features[]) =
 
 static int included_patches[] =
 {   /* Add new patch number below this line */
+/**/
+    4509,
 /**/
     4508,
 /**/

diff --git a/src/vim9compile.c b/src/vim9compile.c
index 898712396e1073394ab41b557d110c95f2e03193..52c60fb9cace7f9d3e43b136274239ab3d830218 100644 (file)
--- a/src/vim9compile.c
+++ b/src/vim9compile.c
@@ -1883,7 +1883,14 @@ compile_assignment(char_u *arg, exarg_T *eap, cmdidx_T cmdidx, cctx_T *cctx)
     lhs_T      lhs;
     long       start_lnum = SOURCING_LNUM;
 
-    // Skip over the "var" or "[var, var]" to get to any "=".
+    p = eap->cmd;
+    if (eap->cmdidx == CMD_var && arg > p && !checkforcmd_noparen(&p, "var", 3))
+    {
+       emsg(_(e_must_use_var_instead_of_va));
+       return NULL;
+    }
+
+    // Skip over the "varname" or "[varname, varname]" to get to any "=".
     p = skip_var_list(arg, TRUE, &var_count, &semicolon, TRUE);
     if (p == NULL)
        return *arg == '[' ? arg : NULL;