patch 9.1.0503: cannot use fuzzy keyword completion

Commit: 
https://github.com/vim/vim/commit/43eef882ff42e673af1e753892801ba20c5d002a
Author: glepnir <glephun...@gmail.com>
Date:   Wed Jun 19 20:20:48 2024 +0200

    patch 9.1.0503: cannot use fuzzy keyword completion
    
    Problem:  cannot use fuzzy keyword completion
              (Maxim Kim)
    Solution: add the "fuzzycollect" value for the 'completeopt'
              setting, to gather matches using fuzzy logic (glepnir)
    
    fixes: #14912
    closes: #14976
    
    Signed-off-by: glepnir <glephun...@gmail.com>
    Signed-off-by: Christian Brabandt <c...@256bit.org>

diff --git a/runtime/doc/options.txt b/runtime/doc/options.txt
index 6ec403117..62d6ded11 100644
--- a/runtime/doc/options.txt
+++ b/runtime/doc/options.txt
@@ -1,4 +1,4 @@
-*options.txt*  For Vim version 9.1.  Last change: 2024 Jun 17
+*options.txt*  For Vim version 9.1.  Last change: 2024 Jun 19
 
 
                  VIM REFERENCE MANUAL    by Bram Moolenaar
@@ -2151,6 +2151,11 @@ A jump table for the options with a short description 
can be found at |Q_op|.
                    difference how completion candidates are reduced from the
                    list of alternatives, but not how the candidates are
                    collected (using different completion types).
+         fuzzycollect
+                   Enable fuzzy collection for default keyword completion.
+                   This allows the collection of matches using fuzzy matching
+                   criteria, providing more comprehensive and flexible
+                   results. Works in combination with other fuzzy options.
 
                                        *'completepopup'* *'cpp'*
 'completepopup' 'cpp'  string (default empty)
diff --git a/runtime/doc/version9.txt b/runtime/doc/version9.txt
index 86dfe6da4..e33b8fa70 100644
--- a/runtime/doc/version9.txt
+++ b/runtime/doc/version9.txt
@@ -1,4 +1,4 @@
-*version9.txt*  For Vim version 9.1.  Last change: 2024 Jun 18
+*version9.txt*  For Vim version 9.1.  Last change: 2024 Jun 19
 
 
                  VIM REFERENCE MANUAL    by Bram Moolenaar
@@ -41554,7 +41554,7 @@ Enum support for Vim9 script |:enum|
                                                        *new-other-9.2*
 Other new features ~
 ------------------
-The comment plugin |comment-install| is included.
+The new plugins |comment-install| and |nohlsearch-install| are included.
 
 Support for Wayland UI.
 
@@ -41634,7 +41634,8 @@ Improved visual highlighting.
 
 Python3 support in OpenVMS.
 
-Support |fuzzy-matching| during |ins-completion| with "fuzzy" item for 
'completeopt'
+Support for |fuzzy-matching| during |ins-completion| with the "fuzzy" and
+"fuzzycollect" values of the 'completeopt' setting
 
 ==============================================================================
 COMPILE TIME CHANGES                                   *compile-changes-9.2*
@@ -41643,6 +41644,8 @@ Support for building with Ruby 3.3.
 
 Support for building Vim 9 in z/OS (MVS).
 
+Clean-up old MS-Windows code.
+
 ==============================================================================
 PATCHES                                                *patches-9.2* 
*bug-fixes-9.2*
                                                *patches-after-9.1*
diff --git a/src/insexpand.c b/src/insexpand.c
index c673df927..78fea5127 100644
--- a/src/insexpand.c
+++ b/src/insexpand.c
@@ -4100,8 +4100,7 @@ find_comp_when_fuzzy(void)
     int                is_backward = compl_shows_dir_backward();
     compl_T    *comp = NULL;
 
-    if (compl_match_array == NULL ||
-           (is_forward && compl_selected_item == compl_match_arraysize - 1)
+    if ((is_forward && compl_selected_item == compl_match_arraysize - 1)
            || (is_backward && compl_selected_item == 0))
        return compl_first_match;
 
@@ -4508,6 +4507,11 @@ ins_compl_use_match(int c)
     static int
 get_normal_compl_info(char_u *line, int startcol, colnr_T curs_col)
 {
+    int                i;
+    int                char_len;
+    size_t     fuzzy_len;
+    char_u     *fuzzy_pattern;
+
     if ((compl_cont_status & CONT_SOL) || ctrl_x_mode_path_defines())
     {
        if (!compl_status_adding())
@@ -4622,6 +4626,38 @@ get_normal_compl_info(char_u *line, int startcol, 
colnr_T curs_col)
 
     compl_patternlen = STRLEN(compl_pattern);
 
+    if ((get_cot_flags() & COT_FUZZYCOLLECT) != 0)
+    {
+       // Adjust size to avoid buffer overflow
+       fuzzy_len = (size_t)compl_length * 5 + 10;
+       // Allocate enough space
+       fuzzy_pattern = alloc(fuzzy_len);
+       if (fuzzy_pattern == NULL)
+       {
+           compl_patternlen = 0;
+           return FAIL;
+       }
+       // Use 'very magic' mode for simpler syntax
+       STRCPY(fuzzy_pattern, "\v");
+       i = 2; // Start from 2 to skip "\v"
+       while (i < compl_length + 2)
+       {
+           // Append "\k*" before each character
+           STRNCAT(fuzzy_pattern, "\k*", fuzzy_len - STRLEN(fuzzy_pattern) - 
1);
+           // Get length of current multi-byte character
+           char_len = mb_ptr2len(compl_pattern + i);
+           // Concatenate the character safely
+           STRNCAT(fuzzy_pattern, compl_pattern + i, char_len);
+           // Move to the next character
+           i += char_len;
+       }
+       // Append "\k*" at the end to match any characters after the pattern
+       STRNCAT(fuzzy_pattern, "\k*", fuzzy_len - STRLEN(fuzzy_pattern) - 1);
+       vim_free(compl_pattern);
+       compl_pattern = fuzzy_pattern;
+       compl_patternlen = STRLEN(compl_pattern);
+    }
+
     return OK;
 }
 
diff --git a/src/option.h b/src/option.h
index 91d8d95c5..67d0b0462 100644
--- a/src/option.h
+++ b/src/option.h
@@ -520,13 +520,14 @@ EXTERN unsigned   cot_flags;      // flags from 
'completeopt'
 #define COT_ANY_MENU   0x003   // combination of menu flags
 #define COT_LONGEST    0x004   // FALSE: insert full match,
                                // TRUE: insert longest prefix
-#define COT_PREVIEW    0x008
-#define COT_POPUP      0x010
-#define COT_POPUPHIDDEN        0x020
-#define COT_ANY_PREVIEW        0x038   // combination of preview flags
-#define COT_NOINSERT   0x040   // FALSE: select & insert, TRUE: noinsert
-#define COT_NOSELECT   0x080   // FALSE: select & insert, TRUE: noselect
-#define COT_FUZZY      0x100   // TRUE: fuzzy match enabled
+#define COT_PREVIEW        0x008
+#define COT_POPUP          0x010
+#define COT_POPUPHIDDEN            0x020
+#define COT_ANY_PREVIEW            0x038   // combination of preview flags
+#define COT_NOINSERT       0x040   // FALSE: select & insert, TRUE: noinsert
+#define COT_NOSELECT       0x080   // FALSE: select & insert, TRUE: noselect
+#define COT_FUZZY          0x100   // TRUE: fuzzy match enabled
+#define COT_FUZZYCOLLECT    0x200   // TRUE: fuzzy collect enabled
 #ifdef BACKSLASH_IN_FILENAME
 EXTERN char_u  *p_csl;         // 'completeslash'
 #endif
diff --git a/src/optionstr.c b/src/optionstr.c
index d72298194..170d48e9a 100644
--- a/src/optionstr.c
+++ b/src/optionstr.c
@@ -118,7 +118,7 @@ static char *(p_fdm_values[]) = {"manual", "expr", 
"marker", "indent", "syntax",
                                NULL};
 static char *(p_fcl_values[]) = {"all", NULL};
 #endif
-static char *(p_cot_values[]) = {"menu", "menuone", "longest", "preview", 
"popup", "popuphidden", "noinsert", "noselect", "fuzzy", NULL};
+static char *(p_cot_values[]) = {"menu", "menuone", "longest", "preview", 
"popup", "popuphidden", "noinsert", "noselect", "fuzzy", "fuzzycollect", NULL};
 #ifdef BACKSLASH_IN_FILENAME
 static char *(p_csl_values[]) = {"slash", "backslash", NULL};
 #endif
diff --git a/src/testdir/test_ins_complete.vim 
b/src/testdir/test_ins_complete.vim
index 48589ce18..61c8ac764 100644
--- a/src/testdir/test_ins_complete.vim
+++ b/src/testdir/test_ins_complete.vim
@@ -2586,6 +2586,37 @@ func Test_complete_fuzzy_match()
   call feedkeys("A\<C-X>\<C-N>\<Esc>0", 'tx!')
   call assert_equal('hello help hero h', getline('.'))
 
+  set completeopt=fuzzycollect
+  call setline(1, ['xyz  yxz  x'])
+  call feedkeys("A\<C-X>\<C-N>\<Esc>0", 'tx!')
+  call assert_equal('xyz  yxz  xyz', getline('.'))
+  " can fuzzy get yxz when use Ctrl-N twice
+  call setline(1, ['xyz  yxz  x'])
+  call feedkeys("A\<C-X>\<C-N>\<C-N>\<Esc>0", 'tx!')
+  call assert_equal('xyz  yxz  yxz', getline('.'))
+
+  call setline(1, ['one two o'])
+  call feedkeys("A\<C-X>\<C-N>\<Esc>0", 'tx!')
+  call assert_equal('one two one', getline('.'))
+
+  call setline(1, ['你好 你'])
+  call feedkeys("A\<C-X>\<C-N>\<Esc>0", 'tx!')
+  call assert_equal('你好 你好', getline('.'))
+  call setline(1, ['你的 我的 的'])
+  call feedkeys("A\<C-X>\<C-N>\<Esc>0", 'tx!')
+  call assert_equal('你的 我的 你的', getline('.'))
+  " can fuzzy get multiple-byte word when use Ctrl-N twice
+  call setline(1, ['你的 我的 的'])
+  call feedkeys("A\<C-X>\<C-N>\<C-N>\<Esc>0", 'tx!')
+  call assert_equal('你的 我的 我的', getline('.'))
+
+  "respect noinsert
+  set completeopt=fuzzycollect,menu,menuone,noinsert
+  call setline(1, ['one two o'])
+  call feedkeys("A\<C-X>\<C-N>", 'tx')
+  call assert_equal('one', g:word)
+  call assert_equal('one two o', getline('.'))
+
   " clean up
   set omnifunc=
   bw!
diff --git a/src/version.c b/src/version.c
index 3afaefa48..a366f35af 100644
--- a/src/version.c
+++ b/src/version.c
@@ -704,6 +704,8 @@ static char *(features[]) =
 
 static int included_patches[] =
 {   /* Add new patch number below this line */
+/**/
+    503,
 /**/
     502,
 /**/

-- 
-- 
You received this message from the "vim_dev" maillist.
Do not top-post! Type your reply below the text you are replying to.
For more information, visit http://www.vim.org/maillist.php

--- 
You received this message because you are subscribed to the Google Groups 
"vim_dev" group.
To unsubscribe from this group and stop receiving emails from it, send an email 
to vim_dev+unsubscr...@googlegroups.com.
To view this discussion on the web visit 
https://groups.google.com/d/msgid/vim_dev/E1sK04P-005IIp-Ci%40256bit.org.

Raspunde prin e-mail lui