mirror of
https://github.com/neovim/neovim.git
synced 2025-02-25 18:55:25 -06:00
vim-patch:8.1.0018: using "gn" may select wrong text when wrapping
Problem: Using "gn" may select wrong text when wrapping.
Solution: Avoid wrapping when searching forward. (Christian Brabandt)
bdb657924d
This commit is contained in:
parent
ff1d111120
commit
c553109932
@ -3953,7 +3953,9 @@ current_search(
|
||||
dec_cursor();
|
||||
|
||||
pos_T orig_pos; /* position of the cursor at beginning */
|
||||
pos_T first_match; // position of first match
|
||||
pos_T pos; /* position after the pattern */
|
||||
int result; // result of various function calls
|
||||
|
||||
if (VIsual_active) {
|
||||
orig_pos = pos = curwin->w_cursor;
|
||||
@ -3988,7 +3990,7 @@ current_search(
|
||||
if (!dir && !one_char)
|
||||
flags = SEARCH_END;
|
||||
|
||||
int result = searchit(curwin, curbuf, &pos, (dir ? FORWARD : BACKWARD),
|
||||
result = searchit(curwin, curbuf, &pos, (dir ? FORWARD : BACKWARD),
|
||||
spats[last_idx].pat, i ? count : 1,
|
||||
SEARCH_KEEP | flags, RE_SEARCH, 0, NULL);
|
||||
|
||||
@ -4012,6 +4014,9 @@ current_search(
|
||||
ml_get(curwin->w_buffer->b_ml.ml_line_count));
|
||||
}
|
||||
}
|
||||
if (i == 0) {
|
||||
first_match = pos;
|
||||
}
|
||||
p_ws = old_p_ws;
|
||||
}
|
||||
|
||||
@ -4029,9 +4034,26 @@ current_search(
|
||||
|
||||
/* move to match, except for zero-width matches, in which case, we are
|
||||
* already on the next match */
|
||||
if (!one_char)
|
||||
searchit(curwin, curbuf, &pos, direction,
|
||||
spats[last_idx].pat, 0L, flags | SEARCH_KEEP, RE_SEARCH, 0, NULL);
|
||||
if (!one_char) {
|
||||
p_ws = false;
|
||||
for (int i = 0; i < 2; i++) {
|
||||
result = searchit(curwin, curbuf, &pos, direction,
|
||||
spats[last_idx].pat, 0L, flags | SEARCH_KEEP, RE_SEARCH,
|
||||
0, NULL);
|
||||
// Search successfull, break out from the loop
|
||||
if (result) {
|
||||
break;
|
||||
}
|
||||
// search failed, try again from the last search position match
|
||||
pos = first_match;
|
||||
}
|
||||
}
|
||||
|
||||
p_ws = old_p_ws;
|
||||
// not found
|
||||
if (!result) {
|
||||
return FAIL;
|
||||
}
|
||||
|
||||
if (!VIsual_active)
|
||||
VIsual = start_pos;
|
||||
|
@ -120,6 +120,15 @@ func Test_gn_command()
|
||||
call assert_equal(['foo', 'bar', '', 'baz'], getline(1,'$'))
|
||||
sil! %d_
|
||||
|
||||
" search using the \zs atom
|
||||
call setline(1, [' nnoremap', '' , 'nnoremap'])
|
||||
set wrapscan&vim
|
||||
let @/ = '\_s\zsnnoremap'
|
||||
$
|
||||
norm! cgnmatch
|
||||
call assert_equal([' nnoremap', '', 'match'], getline(1,'$'))
|
||||
sil! %d_
|
||||
|
||||
set wrapscan&vim
|
||||
set belloff&vim
|
||||
endfu
|
||||
|
Loading…
Reference in New Issue
Block a user