From 4403c674c462e5abcbff99545f5680e847cb35ea Mon Sep 17 00:00:00 2001 From: Christian Brabandt Date: Tue, 30 Sep 2025 20:01:00 +0000 Subject: [PATCH] patch 9.1.1814: Patch v9.1.1812 causes crashes Problem: Patch v9.1.1812 causes crashes Solution: Revert v9.1.1812 Revert "patch 9.1.1812: completion: flicker with slow LSPs" This reverts commit 71b97f29054135151d618433a11ce611ba25844d. fixes: #18450 Signed-off-by: Christian Brabandt --- src/globals.h | 3 --- src/insexpand.c | 13 ++----------- src/term.c | 2 +- src/testdir/test_ins_complete.vim | 4 ++-- src/version.c | 2 ++ 5 files changed, 7 insertions(+), 17 deletions(-) diff --git a/src/globals.h b/src/globals.h index b105ce5154..5e2f7b0f46 100644 --- a/src/globals.h +++ b/src/globals.h @@ -1133,9 +1133,6 @@ EXTERN int secure INIT(= FALSE); // allowed, e.g. when sourcing .exrc or .vimrc // in current directory -EXTERN int no_flush INIT(= 0); - // non-zero to prevent flushing output buffer - EXTERN int textlock INIT(= 0); // non-zero when changing text and jumping to // another window or editing another buffer is diff --git a/src/insexpand.c b/src/insexpand.c index 17d0965ffe..286fc0b472 100644 --- a/src/insexpand.c +++ b/src/insexpand.c @@ -3601,18 +3601,8 @@ expand_by_function(int type, char_u *base, callback_T *cb) // Insert mode in another buffer. ++textlock; - // Suppress flushing of the output buffer. Without this, text removed - // temporarily by ins_compl_delete() is flushed to the terminal and shown - // as deleted, only to be redrawn later. This causes visible flicker (typed - // chars disappear and reappear) when a user func (e.g. an LSP server) - // responds slowly. Such funcs may call sleep(), which indirectly triggers - // out_flush(). We want deleted text to remain visible. - ++no_flush; - retval = call_callback(cb, 0, &rettv, 2, args); - --no_flush; - // Call a function, which returns a list or dict. if (retval == OK) { @@ -6103,6 +6093,7 @@ find_next_completion_match( int compl_fuzzy_match = (cur_cot_flags & COT_FUZZY) != 0; string_T *leader; + while (--todo >= 0) { if (compl_shows_dir_forward() && compl_shown_match->cp_next != NULL) @@ -6212,7 +6203,7 @@ find_next_completion_match( * * Note that this function may be called recursively once only. First with * "allow_get_expansion" TRUE, which calls ins_compl_get_exp(), which in turn - * calls this with "allow_get_expansion" FALSE (via ins_compl_check_keys()). + * calls this function with "allow_get_expansion" FALSE. */ static int ins_compl_next( diff --git a/src/term.c b/src/term.c index 0fa5652b3c..e230858373 100644 --- a/src/term.c +++ b/src/term.c @@ -2772,7 +2772,7 @@ out_flush(void) { int len; - if (no_flush > 0 || out_pos == 0) + if (out_pos == 0) return; // set out_pos to 0 before ui_write, to avoid recursiveness diff --git a/src/testdir/test_ins_complete.vim b/src/testdir/test_ins_complete.vim index 5f5fb8d971..e7346b8b58 100644 --- a/src/testdir/test_ins_complete.vim +++ b/src/testdir/test_ins_complete.vim @@ -420,7 +420,7 @@ func Test_CompleteDoneDict() au CompleteDone * :call CompleteDone_CheckCompletedItemDict(0) set complete=.,FCompleteDone_CompleteFuncDict - execute "normal dda\\" + execute "normal a\\" set complete& call assert_equal(['one', 'two'], v:completed_item[ 'user_data' ]) @@ -473,7 +473,7 @@ func Test_CompleteDoneDictNoUserData() let s:called_completedone = 0 set complete=.,FCompleteDone_CompleteFuncDictNoUserData - execute "normal dda\\" + execute "normal a\\" set complete& call assert_equal('', v:completed_item[ 'user_data' ]) diff --git a/src/version.c b/src/version.c index 3cc88a1064..281038c99e 100644 --- a/src/version.c +++ b/src/version.c @@ -729,6 +729,8 @@ static char *(features[]) = static int included_patches[] = { /* Add new patch number below this line */ +/**/ + 1814, /**/ 1813, /**/