summaryrefslogtreecommitdiffstats
path: root/src
diff options
context:
space:
mode:
authorBram Moolenaar <Bram@vim.org>2019-07-10 22:04:48 +0200
committerBram Moolenaar <Bram@vim.org>2019-07-10 22:04:48 +0200
commit7780e5c1c5485a979ab6e3914a43e17912e213a8 (patch)
treea3aa3d74492df6bd48ae4dec09ca3469d2e95977 /src
parentb05caa782dbab51db8de60940eff7992f8cfd882 (diff)
patch 8.1.1660: assert_fails() does not fail inside try/catchv8.1.1660
Problem: Assert_fails() does not fail inside try/catch. Solution: Set trylevel to zero. (Ozaki Kiichi, closes #4639)
Diffstat (limited to 'src')
-rw-r--r--src/eval.c7
-rw-r--r--src/testdir/test_assert.vim6
-rw-r--r--src/version.c2
3 files changed, 14 insertions, 1 deletions
diff --git a/src/eval.c b/src/eval.c
index 26fad0c5fa..d89093cb33 100644
--- a/src/eval.c
+++ b/src/eval.c
@@ -9910,10 +9910,14 @@ assert_fails(typval_T *argvars)
char_u *cmd = tv_get_string_chk(&argvars[0]);
garray_T ga;
int ret = 0;
+ int save_trylevel = trylevel;
+ // trylevel must be zero for a ":throw" command to be considered failed
+ trylevel = 0;
called_emsg = FALSE;
suppress_errthrow = TRUE;
emsg_silent = TRUE;
+
do_cmdline_cmd(cmd);
if (!called_emsg)
{
@@ -9939,10 +9943,11 @@ assert_fails(typval_T *argvars)
assert_append_cmd_or_arg(&ga, argvars, cmd);
assert_error(&ga);
ga_clear(&ga);
- ret = 1;
+ ret = 1;
}
}
+ trylevel = save_trylevel;
called_emsg = FALSE;
suppress_errthrow = FALSE;
emsg_silent = FALSE;
diff --git a/src/testdir/test_assert.vim b/src/testdir/test_assert.vim
index 0d59a43a2d..900710b893 100644
--- a/src/testdir/test_assert.vim
+++ b/src/testdir/test_assert.vim
@@ -166,6 +166,12 @@ func Test_assert_fail_fails()
call remove(v:errors, 0)
endfunc
+func Test_assert_fails_in_try_block()
+ try
+ call assert_equal(0, assert_fails('throw "error"'))
+ endtry
+endfunc
+
func Test_assert_beeps()
new
call assert_equal(0, assert_beeps('normal h'))
diff --git a/src/version.c b/src/version.c
index 5e83159edf..6af9abaaab 100644
--- a/src/version.c
+++ b/src/version.c
@@ -778,6 +778,8 @@ static char *(features[]) =
static int included_patches[] =
{ /* Add new patch number below this line */
/**/
+ 1660,
+/**/
1659,
/**/
1658,