changeset 23254:4b7e996354e0 v8.2.2173

patch 8.2.2173: Vim9: get internal error when assigning to undefined variable Commit: https://github.com/vim/vim/commit/52c124d3303d3cc3e4504708881906d17d6d6c55 Author: Bram Moolenaar <Bram@vim.org> Date: Sun Dec 20 21:43:35 2020 +0100 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 https://github.com/vim/vim/issues/7475)
author Bram Moolenaar <Bram@vim.org>
date Sun, 20 Dec 2020 21:45:04 +0100
parents fe74b64d34e4
children a1523cdab8ac
files src/errors.h src/testdir/test_vim9_cmd.vim src/version.c src/vim9compile.c src/vim9execute.c
diffstat 5 files changed, 19 insertions(+), 4 deletions(-) [+]
line wrap: on
line diff
--- a/src/errors.h
+++ b/src/errors.h
@@ -321,3 +321,5 @@ EXTERN char e_command_not_followed_by_wh
 	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"));
--- a/src/testdir/test_vim9_cmd.vim
+++ b/src/testdir/test_vim9_cmd.vim
@@ -710,5 +710,16 @@ def Test_ambiguous_user_cmd()
   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
--- 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 */
 /**/
+    2173,
+/**/
     2172,
 /**/
     2171,
--- a/src/vim9compile.c
+++ b/src/vim9compile.c
@@ -7681,8 +7681,9 @@ compile_def_function(ufunc_T *ufunc, int
 	    // 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;
 	    }
 	}
--- a/src/vim9execute.c
+++ b/src/vim9execute.c
@@ -606,8 +606,7 @@ call_ufunc(ufunc_T *ufunc, int argcount,
 	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)