]> granicus.if.org Git - vim/commitdiff
patch 8.2.2758: Vim9: wrong line number for autoload function with wrong name v8.2.2758
authorBram Moolenaar <Bram@vim.org>
Mon, 12 Apr 2021 20:02:36 +0000 (22:02 +0200)
committerBram Moolenaar <Bram@vim.org>
Mon, 12 Apr 2021 20:02:36 +0000 (22:02 +0200)
Problem:    Vim9: wrong line number for autoload function with wrong name.
Solution:   Set and restore SOURCING_LNUM. (closes #8100)

src/testdir/test_vim9_func.vim
src/userfunc.c
src/version.c

index a97146d45b85a494407ed0b571b5e4bd492d2b6e..a941a6102232629f6b051e7cd604f4123fe592ea 100644 (file)
@@ -74,6 +74,30 @@ def TestCompilingErrorInTry()
   delete('Xdir', 'rf')
 enddef
 
+def Test_autoload_name_mismatch()
+  var dir = 'Xdir/autoload'
+  mkdir(dir, 'p')
+
+  var lines =<< trim END
+      vim9script
+      def scriptX#Function()
+        # comment
+        g:runtime = 'yes'
+      enddef
+  END
+  writefile(lines, dir .. '/script.vim')
+
+  var save_rtp = &rtp
+  exe 'set rtp=' .. getcwd() .. '/Xdir'
+  lines =<< trim END
+      call script#Function()
+  END
+  CheckScriptFailure(lines, 'E746:', 2)
+
+  &rtp = save_rtp
+  delete(dir, 'rf')
+enddef
+
 def CallRecursive(n: number): number
   return CallRecursive(n + 1)
 enddef
index 54335d41f9b66cc43739ed035785aa0fc8382512..772d45d12631f25bb7552412c5e6daeb0f6dd78a 100644 (file)
@@ -4058,7 +4058,11 @@ define_function(exarg_T *eap, char_u *name_arg)
            }
            if (j == FAIL)
            {
+               linenr_T save_lnum = SOURCING_LNUM;
+
+               SOURCING_LNUM = sourcing_lnum_top;
                semsg(_("E746: Function name does not match script file name: %s"), name);
+               SOURCING_LNUM = save_lnum;
                goto erret;
            }
        }
index 3728f320e6081c9b4c465c9ada552fba11c56569..4e61210536102b6342a3295bc35569e9618e4dda 100644 (file)
@@ -750,6 +750,8 @@ static char *(features[]) =
 
 static int included_patches[] =
 {   /* Add new patch number below this line */
+/**/
+    2758,
 /**/
     2757,
 /**/