forked from aniani/vim
patch 8.2.1839: Vim9: memory leaks reported in assign test
Problem: Vim9: memory leaks reported in assign test.
Solution: Move the failing job_start() call to separate test files, it
causes false leak reports.
This commit is contained in:
@@ -34,6 +34,7 @@ TEST_VIM9 = \
|
|||||||
test_vim9_cmd \
|
test_vim9_cmd \
|
||||||
test_vim9_disassemble \
|
test_vim9_disassemble \
|
||||||
test_vim9_expr \
|
test_vim9_expr \
|
||||||
|
test_vim9_fails \
|
||||||
test_vim9_func \
|
test_vim9_func \
|
||||||
test_vim9_script
|
test_vim9_script
|
||||||
|
|
||||||
@@ -42,6 +43,7 @@ TEST_VIM9_RES = \
|
|||||||
test_vim9_cmd.res \
|
test_vim9_cmd.res \
|
||||||
test_vim9_disassemble.res \
|
test_vim9_disassemble.res \
|
||||||
test_vim9_expr.res \
|
test_vim9_expr.res \
|
||||||
|
test_vim9_fails.res \
|
||||||
test_vim9_func.res \
|
test_vim9_func.res \
|
||||||
test_vim9_script.res
|
test_vim9_script.res
|
||||||
|
|
||||||
|
|||||||
@@ -75,7 +75,7 @@ def Test_assignment()
|
|||||||
if has('channel')
|
if has('channel')
|
||||||
var chan1: channel
|
var chan1: channel
|
||||||
var job1: job
|
var job1: job
|
||||||
var job2: job = job_start('willfail')
|
# calling job_start() is in test_vim9_fails.vim, it causes leak reports
|
||||||
endif
|
endif
|
||||||
if has('float')
|
if has('float')
|
||||||
var float1: float = 3.4
|
var float1: float = 3.4
|
||||||
@@ -216,12 +216,9 @@ def Test_assignment()
|
|||||||
CheckDefFailure(['v:errmsg += "more"'], 'E1051:')
|
CheckDefFailure(['v:errmsg += "more"'], 'E1051:')
|
||||||
CheckDefFailure(['v:errmsg += 123'], 'E1012:')
|
CheckDefFailure(['v:errmsg += 123'], 'E1012:')
|
||||||
|
|
||||||
# this should not leak
|
|
||||||
if 0
|
|
||||||
var text =<< trim END
|
var text =<< trim END
|
||||||
some text
|
some text
|
||||||
END
|
END
|
||||||
endif
|
|
||||||
enddef
|
enddef
|
||||||
|
|
||||||
def Test_extend_list()
|
def Test_extend_list()
|
||||||
|
|||||||
10
src/testdir/test_vim9_fails.vim
Normal file
10
src/testdir/test_vim9_fails.vim
Normal file
@@ -0,0 +1,10 @@
|
|||||||
|
" Test for Vim9 script with failures, causing memory leaks to be reported.
|
||||||
|
" The leaks happen after a fork() and can be ignored.
|
||||||
|
|
||||||
|
def Test_assignment()
|
||||||
|
if has('channel')
|
||||||
|
var chan1: channel
|
||||||
|
var job1: job
|
||||||
|
var job2: job = job_start('willfail')
|
||||||
|
endif
|
||||||
|
enddef
|
||||||
@@ -750,6 +750,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 */
|
||||||
|
/**/
|
||||||
|
1839,
|
||||||
/**/
|
/**/
|
||||||
1838,
|
1838,
|
||||||
/**/
|
/**/
|
||||||
|
|||||||
Reference in New Issue
Block a user