From 3d79f0a4309995956bd8889940cca22f7a15881d Mon Sep 17 00:00:00 2001 From: Bram Moolenaar Date: Sun, 22 Jan 2023 20:14:26 +0000 Subject: patch 9.0.1233: search() loops forever if "skip" is TRUE for all matches Problem: search() loops forever if "skip" is TRUE for all matches. Solution: Keep the position of the first match. --- src/evalfunc.c | 3 ++- src/testdir/test_search.vim | 16 ++++++++++++++++ src/version.c | 2 ++ 3 files changed, 20 insertions(+), 1 deletion(-) (limited to 'src') diff --git a/src/evalfunc.c b/src/evalfunc.c index 5b25e97cbe..7b34e3925b 100644 --- a/src/evalfunc.c +++ b/src/evalfunc.c @@ -8794,7 +8794,8 @@ search_cmn(typval_T *argvars, pos_T *match_pos, int *flagsp) if (subpatnum == FAIL || !use_skip) // didn't find it or no skip argument break; - firstpos = pos; + if (firstpos.lnum == 0) + firstpos = pos; // If the skip expression matches, ignore this match. { diff --git a/src/testdir/test_search.vim b/src/testdir/test_search.vim index 074ab15467..ef44751659 100644 --- a/src/testdir/test_search.vim +++ b/src/testdir/test_search.vim @@ -1411,6 +1411,22 @@ func Test_subst_word_under_cursor() set noincsearch endfunc +func Test_search_skip_all_matches() + enew + call setline(1, ['no match here', + \ 'match this line', + \ 'nope', + \ 'match in this line', + \ 'last line', + \ ]) + call cursor(1, 1) + let lnum = search('this', '', 0, 0, 'getline(".") =~ "this line"') + " Only check that no match is found. Previously it searched forever. + call assert_equal(0, lnum) + + bwipe! +endfunc + func Test_search_undefined_behaviour() CheckFeature terminal diff --git a/src/version.c b/src/version.c index 6604e01648..293e176263 100644 --- a/src/version.c +++ b/src/version.c @@ -695,6 +695,8 @@ static char *(features[]) = static int included_patches[] = { /* Add new patch number below this line */ +/**/ + 1233, /**/ 1232, /**/ -- cgit v1.2.3