# HG changeset patch # User Bram Moolenaar # Date 1584881108 -3600 # Node ID 4174c4da6ff7ecbaa5b3755503196761def1b9e4 # Parent d16efe1407c178ad536cbac4dd614914b2fcfe7d patch 8.2.0424: checking for wrong return value Commit: https://github.com/vim/vim/commit/97acfc781bdb7fa2838dc6e0e7f9952ea61bb2fd Author: Bram Moolenaar Date: Sun Mar 22 13:44:28 2020 +0100 patch 8.2.0424: checking for wrong return value Problem: Checking for wrong return value. (Tom) Solution: Invert the check and fix the test. diff --git a/src/testdir/test_vim9_script.vim b/src/testdir/test_vim9_script.vim --- a/src/testdir/test_vim9_script.vim +++ b/src/testdir/test_vim9_script.vim @@ -943,13 +943,20 @@ def Test_while_loop() enddef def Test_interrupt_loop() + let caught = false let x = 0 - while 1 - x += 1 - if x == 100 - feedkeys("\", 'Lt') - endif - endwhile + try + while 1 + x += 1 + if x == 100 + feedkeys("\", 'Lt') + endif + endwhile + catch + caught = true + assert_equal(100, x) + endtry + assert_true(caught, 'should have caught an exception') enddef def Test_substitute_cmd() diff --git a/src/version.c b/src/version.c --- a/src/version.c +++ b/src/version.c @@ -739,6 +739,8 @@ static char *(features[]) = static int included_patches[] = { /* Add new patch number below this line */ /**/ + 424, +/**/ 423, /**/ 422, diff --git a/src/vim9execute.c b/src/vim9execute.c --- a/src/vim9execute.c +++ b/src/vim9execute.c @@ -487,7 +487,7 @@ call_def_function( { // Turn CTRL-C into an exception. got_int = FALSE; - if (throw_exception("Vim:Interrupt", ET_INTERRUPT, NULL) != FAIL) + if (throw_exception("Vim:Interrupt", ET_INTERRUPT, NULL) == FAIL) goto failed; did_throw = TRUE; }