# HG changeset patch # User Bram Moolenaar # Date 1637435703 -3600 # Node ID 41b3718d84c349077ae008b51ce75c8c11e358d5 # Parent 9dab308df27ea582b33b05457b36878fdd8b054d patch 8.2.3629: command completion in cmdline window uses global commands Commit: https://github.com/vim/vim/commit/a1198124370a366ff02811a43845a631b5c6e7f0 Author: mityu Date: Sat Nov 20 19:13:39 2021 +0000 patch 8.2.3629: command completion in cmdline window uses global commands Problem: Command completion in cmdline window uses global user commands, not local commands for the window where it was opened from. Solution: Use local commands. (closes #9168) diff --git a/src/evalvars.c b/src/evalvars.c --- a/src/evalvars.c +++ b/src/evalvars.c @@ -2074,8 +2074,7 @@ get_user_var_name(expand_T *xp, int idx) ht = #ifdef FEAT_CMDWIN // In cmdwin, the alternative buffer should be used. - (cmdwin_type != 0 && get_cmdline_type() == NUL) ? - &prevwin->w_buffer->b_vars->dv_hashtab : + is_in_cmdwin() ? &prevwin->w_buffer->b_vars->dv_hashtab : #endif &curbuf->b_vars->dv_hashtab; if (bdone < ht->ht_used) @@ -2093,8 +2092,7 @@ get_user_var_name(expand_T *xp, int idx) ht = #ifdef FEAT_CMDWIN // In cmdwin, the alternative window should be used. - (cmdwin_type != 0 && get_cmdline_type() == NUL) ? - &prevwin->w_vars->dv_hashtab : + is_in_cmdwin() ? &prevwin->w_vars->dv_hashtab : #endif &curwin->w_vars->dv_hashtab; if (wdone < ht->ht_used) diff --git a/src/ex_getln.c b/src/ex_getln.c --- a/src/ex_getln.c +++ b/src/ex_getln.c @@ -4485,6 +4485,15 @@ open_cmdwin(void) return cmdwin_result; } + +/* + * Return TRUE if in the cmdwin, not editing the command line. + */ + int +is_in_cmdwin(void) +{ + return cmdwin_type != 0 && get_cmdline_type() == NUL; +} #endif // FEAT_CMDWIN /* diff --git a/src/proto/ex_getln.pro b/src/proto/ex_getln.pro --- a/src/proto/ex_getln.pro +++ b/src/proto/ex_getln.pro @@ -38,6 +38,7 @@ int get_cmdline_type(void); int get_cmdline_firstc(void); int get_list_range(char_u **str, int *num1, int *num2); char *check_cedit(void); +int is_in_cmdwin(void); char_u *script_get(exarg_T *eap, char_u *cmd); void get_user_input(typval_T *argvars, typval_T *rettv, int inputdialog, int secret); /* vim: set ft=c : */ diff --git a/src/testdir/test_ins_complete.vim b/src/testdir/test_ins_complete.vim --- a/src/testdir/test_ins_complete.vim +++ b/src/testdir/test_ins_complete.vim @@ -373,6 +373,14 @@ func Test_compl_feedkeys() set completeopt& endfunc +func s:ComplInCmdwin_GlobalCompletion(a, l, p) + return 'global' +endfunc + +func s:ComplInCmdwin_LocalCompletion(a, l, p) + return 'local' +endfunc + func Test_compl_in_cmdwin() CheckFeature cmdwin @@ -411,6 +419,47 @@ func Test_compl_in_cmdwin() call feedkeys("q::GetInput b:test_\\:q\", 'tx!') call assert_equal('b:test_', input) + + " Argument completion of buffer-local command + func s:ComplInCmdwin_GlobalCompletionList(a, l, p) + return ['global'] + endfunc + + func s:ComplInCmdwin_LocalCompletionList(a, l, p) + return ['local'] + endfunc + + func s:ComplInCmdwin_CheckCompletion(arg) + call assert_equal('local', a:arg) + endfunc + + com! -nargs=1 -complete=custom,ComplInCmdwin_GlobalCompletion + \ TestCommand call s:ComplInCmdwin_CheckCompletion() + com! -buffer -nargs=1 -complete=custom,ComplInCmdwin_LocalCompletion + \ TestCommand call s:ComplInCmdwin_CheckCompletion() + call feedkeys("q:iTestCommand \\", 'tx!') + + com! -nargs=1 -complete=customlist,ComplInCmdwin_GlobalCompletionList + \ TestCommand call s:ComplInCmdwin_CheckCompletion() + com! -buffer -nargs=1 -complete=customlist,ComplInCmdwin_LocalCompletionList + \ TestCommand call s:ComplInCmdwin_CheckCompletion() + + call feedkeys("q:iTestCommand \\", 'tx!') + + func! s:ComplInCmdwin_CheckCompletion(arg) + call assert_equal('global', a:arg) + endfunc + new + call feedkeys("q:iTestCommand \\", 'tx!') + quit + + delfunc s:ComplInCmdwin_GlobalCompletion + delfunc s:ComplInCmdwin_LocalCompletion + delfunc s:ComplInCmdwin_GlobalCompletionList + delfunc s:ComplInCmdwin_LocalCompletionList + delfunc s:ComplInCmdwin_CheckCompletion + + delcom -buffer TestCommand delcom TestCommand delcom GetInput unlet w:test_winvar diff --git a/src/usercmd.c b/src/usercmd.c --- a/src/usercmd.c +++ b/src/usercmd.c @@ -141,7 +141,11 @@ find_ucmd( /* * Look for buffer-local user commands first, then global ones. */ - gap = &curbuf->b_ucmds; + gap = +#ifdef FEAT_CMDWIN + is_in_cmdwin() ? &prevwin->w_buffer->b_ucmds : +#endif + &curbuf->b_ucmds; for (;;) { for (j = 0; j < gap->ga_len; ++j) @@ -303,7 +307,7 @@ get_user_commands(expand_T *xp UNUSED, i // In cmdwin, the alternative buffer should be used. buf_T *buf = #ifdef FEAT_CMDWIN - (cmdwin_type != 0 && get_cmdline_type() == NUL) ? prevwin->w_buffer : + is_in_cmdwin() ? prevwin->w_buffer : #endif curbuf; @@ -330,10 +334,9 @@ get_user_command_name(int idx, int cmdid // In cmdwin, the alternative buffer should be used. buf_T *buf = #ifdef FEAT_CMDWIN - (cmdwin_type != 0 && get_cmdline_type() == NUL) - ? prevwin->w_buffer : + is_in_cmdwin() ? prevwin->w_buffer : #endif - curbuf; + curbuf; if (idx < buf->b_ucmds.ga_len) return USER_CMD_GA(&buf->b_ucmds, idx)->uc_name; @@ -420,10 +423,9 @@ uc_list(char_u *name, size_t name_len) // In cmdwin, the alternative buffer should be used. gap = #ifdef FEAT_CMDWIN - (cmdwin_type != 0 && get_cmdline_type() == NUL) ? - &prevwin->w_buffer->b_ucmds : + is_in_cmdwin() ? &prevwin->w_buffer->b_ucmds : #endif - &curbuf->b_ucmds; + &curbuf->b_ucmds; for (;;) { for (i = 0; i < gap->ga_len; ++i) diff --git a/src/version.c b/src/version.c --- a/src/version.c +++ b/src/version.c @@ -758,6 +758,8 @@ static char *(features[]) = static int included_patches[] = { /* Add new patch number below this line */ /**/ + 3629, +/**/ 3628, /**/ 3627,