patch 8.2.2857: Vim9: exception in ISN_INSTR caught at wrong level

Problem:    Vim9: exception in ISN_INSTR caught at wrong level.
Solution:   Set the starting trylevel in exec_instructions(). (closes #8214)

diff --git a/src/globals.h b/src/globals.h
index 0a6b87a7807280950faba0b9518ba3740b1b05d1..74a7c3967f5395108510d0a4c51f7d2495ba3a6d 100644 (file)
--- a/src/globals.h
+++ b/src/globals.h
@@ -1616,7 +1616,7 @@ EXTERN char e_duparg2[]           INIT(= N_("E983: Duplicate argument: %s"));
 EXTERN char e_invargval[]      INIT(= N_("E475: Invalid value for argument %s"));
 EXTERN char e_invargNval[]     INIT(= N_("E475: Invalid value for argument %s: %s"));
 #ifdef FEAT_EVAL
-EXTERN char e_invexpr2[]       INIT(= N_("E15: Invalid expression: %s"));
+EXTERN char e_invexpr2[]       INIT(= N_("E15: Invalid expression: \"%s\""));
 #endif
 EXTERN char e_invrange[]       INIT(= N_("E16: Invalid range"));
 #if defined(UNIX) || defined(FEAT_SYN_HL) || defined(FEAT_SPELL)

diff --git a/src/testdir/test_vim9_builtin.vim b/src/testdir/test_vim9_builtin.vim
index 41063c544446a089e22f3e7571afcb1be9c7a587..2e9e0566c16bfa5bac429236bc82fca79afc635d 100644 (file)
--- a/src/testdir/test_vim9_builtin.vim
+++ b/src/testdir/test_vim9_builtin.vim
@@ -1006,6 +1006,20 @@ def Test_searchpair()
   normal 0f{
   assert_equal([0, 0], searchpairpos('{', '', '}', '', 'col(".") > col'))
 
+  var lines =<< trim END
+      vim9script
+      setline(1, '()')
+      normal gg
+      def Fail()
+        try
+          searchpairpos('(', '', ')', 'nW', '[0]->map("")')
+        catch
+        endtry
+      enddef
+      Fail()
+  END
+  CheckScriptFailure(lines, 'E15:')
+
   bwipe!
 enddef
 

diff --git a/src/version.c b/src/version.c
index 70e02ed8edf8c167374dde0b38d26d2949f60713..e45d3be709a9063667058eaf51b7d390f92e5428 100644 (file)
--- a/src/version.c
+++ b/src/version.c
@@ -750,6 +750,8 @@ static char *(features[]) =
 
 static int included_patches[] =
 {   /* Add new patch number below this line */
+/**/
+    2857,
 /**/
     2856,
 /**/

diff --git a/src/vim9compile.c b/src/vim9compile.c
index 39c36dfb24fdd55a44dd8ca26e82f9a8387ed2dd..9aa93de7b21459a21b0c3f7646c8c76286640da2 100644 (file)
--- a/src/vim9compile.c
+++ b/src/vim9compile.c
@@ -3131,7 +3131,8 @@ compile_string(isn_T *isn, cctx_T *cctx)
     s = skipwhite(s);
     trailing_error = *s != NUL;
 
-    if (expr_res == FAIL || trailing_error)
+    if (expr_res == FAIL || trailing_error
+                                      || ga_grow(&cctx->ctx_instr, 1) == FAIL)
     {
        if (trailing_error)
            semsg(_(e_trailing_arg), s);
@@ -3185,7 +3186,7 @@ compile_arguments(char_u **arg, cctx_T *cctx, int *argcount, int is_searchpair)
            return FAIL;
        ++*argcount;
 
-       if (is_searchpair && *argcount == 5
+       if (is_searchpair && *argcount >= 5
                && cctx->ctx_instr.ga_len == instr_count + 1)
        {
            isn_T *isn = ((isn_T *)cctx->ctx_instr.ga_data) + instr_count;

diff --git a/src/vim9execute.c b/src/vim9execute.c
index ae4142b0cedffef65f832a56df91977ad5850367..425cdb9bb24bb0c5d0a60aeef9bf68ff95176cd9 100644 (file)
--- a/src/vim9execute.c
+++ b/src/vim9execute.c
@@ -1295,6 +1295,9 @@ exec_instructions(ectx_T *ectx)
     // Start execution at the first instruction.
     ectx->ec_iidx = 0;
 
+    // Only catch exceptions in this instruction list.
+    ectx->ec_trylevel_at_start = trylevel;
+
     for (;;)
     {
        isn_T       *iptr;
@@ -4158,7 +4161,6 @@ call_def_function(
     ga_init2(&ectx.ec_trystack, sizeof(trycmd_T), 10);
     ga_init2(&ectx.ec_funcrefs, sizeof(partial_T *), 10);
     ectx.ec_did_emsg_before = did_emsg_before;
-    ectx.ec_trylevel_at_start = trylevel;
 
     idx = argc - ufunc->uf_args.ga_len;
     if (idx > 0 && ufunc->uf_va_name == NULL)