mirrors/neovim
1
0
Fork 0
mirror of https://github.com/neovim/neovim.git synced 2025-09-18 01:08:20 +00:00
Code Issues Packages Projects Releases Wiki Activity

refactor: make wildmenu code closer to Vim (#19870)

Browse Source 
This is a small refactor that makes `compl_match_array` static and
doesn't change any behavior.
This commit is contained in:
zeertzjq
2022-08-21 11:06:07 +08:00
committed by GitHub
parent 61ff37952a
commit 506a3ec913
2 changed files with 39 additions and 27 deletions
Download Patch File Download Diff File Expand all files Collapse all files

36
src/nvim/cmdexpand.c
View File

@@ -57,7 +57,7 @@ static int cmd_showtail; ///< Only show path tail in lists ?
/// "compl_match_array" points the currently displayed list of entries in the
/// popup menu. It is NULL when there is no popup menu.
pumitem_T *compl_match_array = NULL; // TODO(zeertzjq): make this static in cmdexpand.c
static pumitem_T *compl_match_array = NULL;
static int compl_match_arraysize;
/// First column in cmdline of the matched item for completion.
static int compl_startcol;
@@ -256,6 +256,19 @@ void cmdline_pum_display(bool changed_array)
changed_array, compl_startcol);
}
bool cmdline_pum_active(void)
{
// return p_wmnu && pum_visible() && compl_match_array != NULL;
return compl_match_array != NULL;
}
/// Remove the cmdline completion popup menu
void cmdline_pum_remove(void)
{
pum_undisplay(true);
XFREE_CLEAR(compl_match_array);
}
/// Do wildcard expansion on the string 'str'.
/// Chars that should not be expanded must be preceded with a backslash.
/// Return a pointer to allocated memory containing the new string.
@@ -545,10 +558,14 @@ int showmatches(expand_T *xp, int wildmenu)
if (compl_use_pum) {
assert(num_files >= 0);
compl_match_arraysize = num_files;
compl_match_array = xcalloc((size_t)compl_match_arraysize,
sizeof(pumitem_T));
compl_match_array = xmalloc(sizeof(pumitem_T) * (size_t)compl_match_arraysize);
for (i = 0; i < num_files; i++) {
compl_match_array[i].pum_text = (char_u *)L_SHOWFILE(i);
compl_match_array[i] = (pumitem_T){
.pum_text = (char_u *)L_SHOWFILE(i),
.pum_info = NULL,
.pum_extra = NULL,
.pum_kind = NULL,
};
}
char_u *endpos = (char_u *)(showtail ? sm_gettail(xp->xp_pattern, true) : xp->xp_pattern);
if (ui_has(kUICmdline)) {
@@ -2514,15 +2531,16 @@ int wildmenu_translate_key(CmdlineInfo *cclp, int key, expand_T *xp, int did_wil
{
int c = key;
if (did_wild_list && p_wmnu) {
if (did_wild_list) {
if (c == K_LEFT) {
c = Ctrl_P;
} else if (c == K_RIGHT) {
c = Ctrl_N;
}
}
// Hitting CR after "emenu Name.": complete submenu
if (xp->xp_context == EXPAND_MENUNAMES && p_wmnu
if (xp->xp_context == EXPAND_MENUNAMES
&& cclp->cmdpos > 1
&& cclp->cmdbuff[cclp->cmdpos - 1] == '.'
&& cclp->cmdbuff[cclp->cmdpos - 2] != '\\'
@@ -2548,10 +2566,6 @@ int wildmenu_process_key(CmdlineInfo *cclp, int key, expand_T *xp)
{
int c = key;
if (!p_wmnu) {
return c;
}
// Special translations for 'wildmenu'
if (xp->xp_context == EXPAND_MENUNAMES) {
// Hitting <Down> after "emenu Name.": complete submenu
@@ -2696,7 +2710,7 @@ void wildmenu_cleanup(CmdlineInfo *cclp)
}
const bool skt = KeyTyped;
int old_RedrawingDisabled = RedrawingDisabled;
const int old_RedrawingDisabled = RedrawingDisabled;
if (cclp->input_fn) {
RedrawingDisabled = 0;