mirror of
https://github.com/vim/vim.git
synced 2025-09-25 03:54:15 -04:00
patch 8.2.0287: Vim9: return in try block not tested; catch not tested
Problem: Vim9: return in try block not tested; catch with pattern not tested. Solution: Add tests. Make it work.
This commit is contained in:
@@ -236,12 +236,40 @@ def CatchInDef()
|
|||||||
endtry
|
endtry
|
||||||
enddef
|
enddef
|
||||||
|
|
||||||
|
def ReturnFinally(): string
|
||||||
|
try
|
||||||
|
return 'intry'
|
||||||
|
finally
|
||||||
|
g:in_finally = 'finally'
|
||||||
|
endtry
|
||||||
|
return 'end'
|
||||||
|
enddef
|
||||||
|
|
||||||
def Test_try_catch_nested()
|
def Test_try_catch_nested()
|
||||||
CatchInFunc()
|
CatchInFunc()
|
||||||
assert_equal('getout', g:thrown_func)
|
assert_equal('getout', g:thrown_func)
|
||||||
|
|
||||||
CatchInDef()
|
CatchInDef()
|
||||||
assert_equal('getout', g:thrown_def)
|
assert_equal('getout', g:thrown_def)
|
||||||
|
|
||||||
|
assert_equal('intry', ReturnFinally())
|
||||||
|
assert_equal('finally', g:in_finally)
|
||||||
|
enddef
|
||||||
|
|
||||||
|
def Test_try_catch_match()
|
||||||
|
let seq = 'a'
|
||||||
|
try
|
||||||
|
throw 'something'
|
||||||
|
catch /nothing/
|
||||||
|
seq ..= 'x'
|
||||||
|
catch /some/
|
||||||
|
seq ..= 'b'
|
||||||
|
catch /asdf/
|
||||||
|
seq ..= 'x'
|
||||||
|
finally
|
||||||
|
seq ..= 'c'
|
||||||
|
endtry
|
||||||
|
assert_equal('abc', seq)
|
||||||
enddef
|
enddef
|
||||||
|
|
||||||
let s:export_script_lines =<< trim END
|
let s:export_script_lines =<< trim END
|
||||||
|
@@ -738,6 +738,8 @@ static char *(features[]) =
|
|||||||
|
|
||||||
static int included_patches[] =
|
static int included_patches[] =
|
||||||
{ /* Add new patch number below this line */
|
{ /* Add new patch number below this line */
|
||||||
|
/**/
|
||||||
|
287,
|
||||||
/**/
|
/**/
|
||||||
286,
|
286,
|
||||||
/**/
|
/**/
|
||||||
|
@@ -1065,6 +1065,7 @@ call_def_function(
|
|||||||
trycmd->tcd_frame = ectx.ec_frame;
|
trycmd->tcd_frame = ectx.ec_frame;
|
||||||
trycmd->tcd_catch_idx = iptr->isn_arg.try.try_catch;
|
trycmd->tcd_catch_idx = iptr->isn_arg.try.try_catch;
|
||||||
trycmd->tcd_finally_idx = iptr->isn_arg.try.try_finally;
|
trycmd->tcd_finally_idx = iptr->isn_arg.try.try_finally;
|
||||||
|
trycmd->tcd_caught = FALSE;
|
||||||
}
|
}
|
||||||
break;
|
break;
|
||||||
|
|
||||||
@@ -1109,7 +1110,7 @@ call_def_function(
|
|||||||
--trylevel;
|
--trylevel;
|
||||||
trycmd = ((trycmd_T *)trystack->ga_data)
|
trycmd = ((trycmd_T *)trystack->ga_data)
|
||||||
+ trystack->ga_len;
|
+ trystack->ga_len;
|
||||||
if (trycmd->tcd_caught)
|
if (trycmd->tcd_caught && current_exception != NULL)
|
||||||
{
|
{
|
||||||
// discard the exception
|
// discard the exception
|
||||||
if (caught_stack == current_exception)
|
if (caught_stack == current_exception)
|
||||||
|
Reference in New Issue
Block a user