commit 2017d6f3b1d523204e5471e941cfa687b4da0058
author Bram Moolenaar <Bram@vim.org> Thu Jan 20 19:38:46 2022 +0000
committer Bram Moolenaar <Bram@vim.org> Thu Jan 20 19:38:46 2022 +0000
tree 7c95ca32695664d17459de50b81bd49f5fa47d58
parent 9c7cae66bc21a3dc5c6e60cd64216ce7c9073003

patch 8.2.4163: no error for omitting function name after autoload prefix

Problem:    No error for omitting function name after autoload prefix.
Solution:   Check for missing function name. (issue #9577)

src/testdir/test_vim9_import.vim

diff --git a/src/testdir/test_vim9_import.vim b/src/testdir/test_vim9_import.vim
index c26ccf8..b15d411 100644
--- a/src/testdir/test_vim9_import.vim
+++ b/src/testdir/test_vim9_import.vim
@@ -1514,6 +1514,21 @@
   delete('Xdir', 'rf')
 enddef
 
+def Test_autoload_missing_function_name()
+  mkdir('Xdir/autoload', 'p')
+
+  var lines =<< trim END
+     vim9script
+
+     def loadme#()
+     enddef
+  END
+  writefile(lines, 'Xdir/autoload/loadme.vim')
+  assert_fails('source Xdir/autoload/loadme.vim', 'E129:')
+
+  delete('Xdir', 'rf')
+enddef
+
 def Test_import_autoload_postponed()
   mkdir('Xdir/autoload', 'p')
   var save_rtp = &rtp

src/userfunc.c

diff --git a/src/userfunc.c b/src/userfunc.c
index 014ec55..680017c 100644
--- a/src/userfunc.c
+++ b/src/userfunc.c
@@ -3693,7 +3693,8 @@
     // Note that TFN_ flags use the same values as GLV_ flags.
     end = get_lval(start, NULL, &lv, FALSE, skip, flags | GLV_READ_ONLY,
 					      lead > 2 ? 0 : FNE_CHECK_START);
-    if (end == start)
+    if (end == start
+		   || (end != NULL && end[-1] == AUTOLOAD_CHAR && *end == '('))
     {
 	if (!skip)
 	    emsg(_(e_function_name_required));

src/version.c

diff --git a/src/version.c b/src/version.c
index 7fd24f5..2dd0194 100644
--- a/src/version.c
+++ b/src/version.c
@@ -751,6 +751,8 @@
 static int included_patches[] =
 {   /* Add new patch number below this line */
 /**/
+    4163,
+/**/
     4162,
 /**/
     4161,