diff options
author | Christian Brabandt <cb@256bit.org> | 2023-11-04 09:58:14 +0100 |
---|---|---|
committer | Christian Brabandt <cb@256bit.org> | 2023-11-04 09:58:14 +0100 |
commit | 57f9ce1a0977da13e5923214086795ffa2d28ce1 (patch) | |
tree | d0bae6e2534822005e208ba858cb6b054f2c47a2 /src/insexpand.c | |
parent | b23c1fc596501a8dfc0355ed8084dcbf018f7907 (diff) |
patch 9.0.2090: complete_info() skips entries with 'noselect'v9.0.2090
Problem: complete_info() skips entries with 'noselect'
Solution: Check, if first entry is at original text state
Unfortunately, Commit daef8c74375141974d61b85199b383017644978c
introduced a regression, that when ':set completeopt+=noselect' is set
and no completion item has been selected yet, it did not fill the
complete_info['items'] list.
This happened, because the current match item did not have the
CP_ORIGINAL_TEXT flag set and then the cp->prev pointer did point to the
original flag item, which caused the following while loop to not being
run but being skipped instead.
So when the 'noselect' is set, only start with to the previous selection
item, if the initial completion item has the CP_ORIGINAL_TEXT flag set,
else use the 2nd previous item instead.
fixes: #13451
closes: #13452
Signed-off-by: Christian Brabandt <cb@256bit.org>
Diffstat (limited to 'src/insexpand.c')
-rw-r--r-- | src/insexpand.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/src/insexpand.c b/src/insexpand.c index 43fb8ca505..7ede6bd91f 100644 --- a/src/insexpand.c +++ b/src/insexpand.c @@ -3080,7 +3080,7 @@ info_add_completion_info(list_T *li) // Skip the element with the CP_ORIGINAL_TEXT flag at the beginning, in case of // forward completion, or at the end, in case of backward completion. - match = forward ? match->cp_next : (compl_no_select ? match->cp_prev : match->cp_prev->cp_prev); + match = forward ? match->cp_next : (compl_no_select && match_at_original_text(match) ? match->cp_prev : match->cp_prev->cp_prev); while (match != NULL && !match_at_original_text(match)) { |