Patch 8.2.3242

4 views
Skip to first unread message

Bram Moolenaar

unread,
Jul 28, 2021, 4:44:46 PM7/28/21
to vim...@googlegroups.com

Patch 8.2.3242
Problem: Vim9: valgrind reports leaks in builtin function test.
Solution: Do not start a job.
Files: src/testdir/test_vim9_builtin.vim


*** ../vim-8.2.3241/src/testdir/test_vim9_builtin.vim 2021-07-28 11:51:44.321061719 +0200
--- src/testdir/test_vim9_builtin.vim 2021-07-28 22:41:26.357944332 +0200
***************
*** 3239,3246 ****
assert_equal('AX234', res)

if has('job')
! assert_fails('"text"->substitute(".*", () => job_start(":"), "")', 'E908: using an invalid value as a String: job')
! assert_fails('"text"->substitute(".*", () => job_start(":")->job_getchannel(), "")', 'E908: using an invalid value as a String: channel')
endif
CheckDefAndScriptFailure2(['substitute(1, "b", "1", "d")'], 'E1013: Argument 1: type mismatch, expected string but got number', 'E1174: String required for argument 1')
CheckDefAndScriptFailure2(['substitute("a", 2, "1", "d")'], 'E1013: Argument 2: type mismatch, expected string but got number', 'E1174: String required for argument 2')
--- 3239,3246 ----
assert_equal('AX234', res)

if has('job')
! assert_fails('"text"->substitute(".*", () => test_null_job(), "")', 'E908: using an invalid value as a String: job')
! assert_fails('"text"->substitute(".*", () => test_null_channel(), "")', 'E908: using an invalid value as a String: channel')
endif
CheckDefAndScriptFailure2(['substitute(1, "b", "1", "d")'], 'E1013: Argument 1: type mismatch, expected string but got number', 'E1174: String required for argument 1')
CheckDefAndScriptFailure2(['substitute("a", 2, "1", "d")'], 'E1013: Argument 2: type mismatch, expected string but got number', 'E1174: String required for argument 2')
*** ../vim-8.2.3241/src/version.c 2021-07-28 22:21:20.372610767 +0200
--- src/version.c 2021-07-28 22:43:27.413669780 +0200
***************
*** 757,758 ****
--- 757,760 ----
{ /* Add new patch number below this line */
+ /**/
+ 3242,
/**/

--
GALAHAD: Camelot ...
LAUNCELOT: Camelot ...
GAWAIN: It's only a model.
"Monty Python and the Holy Grail" PYTHON (MONTY) PICTURES LTD

/// Bram Moolenaar -- Br...@Moolenaar.net -- http://www.Moolenaar.net \\\
/// \\\
\\\ sponsor Vim, vote for features -- http://www.Vim.org/sponsor/ ///
\\\ help me help AIDS victims -- http://ICCF-Holland.org ///
Reply all
Reply to author
Forward
0 new messages