commit 52c124d3303d3cc3e4504708881906d17d6d6c55
author Bram Moolenaar <Bram@vim.org> Sun Dec 20 21:43:35 2020 +0100
committer Bram Moolenaar <Bram@vim.org> Sun Dec 20 21:43:35 2020 +0100
tree 0c93cd9c0e54856fa52c7631d483fd49cdc627b3
parent 5082471f91dd42ed8c35e0f649d0a6572e6fe3fc

patch 8.2.2173: Vim9: get internal error when assigning to undefined variable

Problem:    Vim9: get internal error when assigning to undefined variable.
Solution:   Add error message. (closes #7475)

src/errors.h

diff --git a/src/errors.h b/src/errors.h
index e761a4e..023c1d1 100644
--- a/src/errors.h
+++ b/src/errors.h
@@ -321,3 +321,5 @@
 	INIT(= N_("E1144: Command is not followed by white space: %s"));
 EXTERN char e_missing_heredoc_end_marker_str[]
 	INIT(= N_("E1145: Missing heredoc end marker: %s"));
+EXTERN char e_command_not_recognized_str[]
+	INIT(= N_("E1146: Command not recognized: %s"));

src/testdir/test_vim9_cmd.vim

diff --git a/src/testdir/test_vim9_cmd.vim b/src/testdir/test_vim9_cmd.vim
index bdc9ad0..d297931 100644
--- a/src/testdir/test_vim9_cmd.vim
+++ b/src/testdir/test_vim9_cmd.vim
@@ -710,5 +710,16 @@
   CheckScriptFailure(lines, 'E464:')
 enddef
 
+def Test_command_not_recognized()
+  var lines =<< trim END
+    d.key = 'asdf'
+  END
+  CheckDefFailure(lines, 'E1146:', 1)
+
+  lines =<< trim END
+    d['key'] = 'asdf'
+  END
+  CheckDefFailure(lines, 'E1146:', 1)
+enddef
 
 " vim: ts=8 sw=2 sts=2 expandtab tw=80 fdm=marker

src/version.c

diff --git a/src/version.c b/src/version.c
index f374d43..1a45c95 100644
--- a/src/version.c
+++ b/src/version.c
@@ -751,6 +751,8 @@
 static int included_patches[] =
 {   /* Add new patch number below this line */
 /**/
+    2173,
+/**/
     2172,
 /**/
     2171,

src/vim9compile.c

diff --git a/src/vim9compile.c b/src/vim9compile.c
index 9bd3698..133a09c 100644
--- a/src/vim9compile.c
+++ b/src/vim9compile.c
@@ -7681,8 +7681,9 @@
 	    // Expression or function call.
 	    if (ea.cmdidx != CMD_eval)
 	    {
-		// CMD_var cannot happen, compile_assignment() above is used
-		iemsg("Command from find_ex_command() not handled");
+		// CMD_var cannot happen, compile_assignment() above would be
+		// used.  Most likely an assignment to a non-existing variable.
+		semsg(_(e_command_not_recognized_str), ea.cmd);
 		goto erret;
 	    }
 	}

src/vim9execute.c

diff --git a/src/vim9execute.c b/src/vim9execute.c
index 606ce0c..db15cea 100644
--- a/src/vim9execute.c
+++ b/src/vim9execute.c
@@ -606,8 +606,7 @@
 	return FAIL;
     if (ufunc->uf_def_status == UF_COMPILED)
     {
-	int error = check_user_func_argcount(ufunc, argcount);
-
+	error = check_user_func_argcount(ufunc, argcount);
 	if (error != FCERR_UNKNOWN)
 	{
 	    if (error == FCERR_TOOMANY)