Patch 9.0.1534

3 views
Skip to first unread message

Bram Moolenaar

unread,
May 9, 2023, 4:46:25 PM5/9/23
to vim...@googlegroups.com

Patch 9.0.1534 (after 9.0.1532)
Problem: Test for expanding "~" in substitute takes too long.
Solution: Disable the test for now.
Files: src/testdir/test_substitute.vim


*** ../vim-9.0.1533/src/testdir/test_substitute.vim 2023-05-09 21:15:26.397845247 +0100
--- src/testdir/test_substitute.vim 2023-05-09 21:42:24.650770714 +0100
***************
*** 1415,1432 ****
endfunc

" Check handling expanding "~" resulting in extremely long text.
! func Test_substitute_tilde_too_long()
! enew!
!
! s/.*/ixxx
! s//~~~~~~~~~AAAAAAA@(
!
! " Either fails with "out of memory" or "text too long".
! " This can take a long time.
! call assert_fails('sil! norm &&&&&&&&&', ['E1240:\|E342:'])
!
! bwipe!
! endfunc

" This should be done last to reveal a memory leak when vim_regsub_both() is
" called to evaluate an expression but it is not used in a second call.
--- 1415,1433 ----
endfunc

" Check handling expanding "~" resulting in extremely long text.
! " FIXME: disabled, it takes too long to run on CI
! #func Test_substitute_tilde_too_long()
! # enew!
! #
! # s/.*/ixxx
! # s//~~~~~~~~~AAAAAAA@(
! #
! # " Either fails with "out of memory" or "text too long".
! # " This can take a long time.
! # call assert_fails('sil! norm &&&&&&&&&', ['E1240:\|E342:'])
! #
! # bwipe!
! #endfunc

" This should be done last to reveal a memory leak when vim_regsub_both() is
" called to evaluate an expression but it is not used in a second call.
*** ../vim-9.0.1533/src/version.c 2023-05-09 21:23:48.762914107 +0100
--- src/version.c 2023-05-09 21:43:17.226656248 +0100
***************
*** 697,698 ****
--- 697,700 ----
{ /* Add new patch number below this line */
+ /**/
+ 1534,
/**/

--
10E21 Picolos = 1 Gigolo

/// 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