Skip to content

Commit 0732932

Browse files
committed
patch 8.2.3242: Vim9: valgrind reports leaks in builtin function test
Problem: Vim9: valgrind reports leaks in builtin function test. Solution: Do not start a job.
1 parent 6e850a6 commit 0732932

2 files changed

Lines changed: 4 additions & 2 deletions

File tree

src/testdir/test_vim9_builtin.vim

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -3239,8 +3239,8 @@ def Test_substitute()
32393239
assert_equal('AX234', res)
32403240

32413241
if has('job')
3242-
assert_fails('"text"->substitute(".*", () => job_start(":"), "")', 'E908: using an invalid value as a String: job')
3243-
assert_fails('"text"->substitute(".*", () => job_start(":")->job_getchannel(), "")', 'E908: using an invalid value as a String: channel')
3242+
assert_fails('"text"->substitute(".*", () => test_null_job(), "")', 'E908: using an invalid value as a String: job')
3243+
assert_fails('"text"->substitute(".*", () => test_null_channel(), "")', 'E908: using an invalid value as a String: channel')
32443244
endif
32453245
CheckDefAndScriptFailure2(['substitute(1, "b", "1", "d")'], 'E1013: Argument 1: type mismatch, expected string but got number', 'E1174: String required for argument 1')
32463246
CheckDefAndScriptFailure2(['substitute("a", 2, "1", "d")'], 'E1013: Argument 2: type mismatch, expected string but got number', 'E1174: String required for argument 2')

src/version.c

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -755,6 +755,8 @@ static char *(features[]) =
755755

756756
static int included_patches[] =
757757
{ /* Add new patch number below this line */
758+
/**/
759+
3242,
758760
/**/
759761
3241,
760762
/**/

0 commit comments

Comments
 (0)