# HG changeset patch # User Bram Moolenaar # Date 1583010905 -3600 # Node ID ff5048b0ccfecaccb860b9423ec76aefd4c9597d # Parent aae35642cc05017c271d27d466198a7614dcaaa0 patch 8.2.0335: no completion for :disassemble Commit: https://github.com/vim/vim/commit/cc390ff5b2c8725c55b961b24322c470659ede9f Author: Bram Moolenaar Date: Sat Feb 29 22:06:30 2020 +0100 patch 8.2.0335: no completion for :disassemble Problem: No completion for :disassemble. Solution: Make completion work. Also complete script-local functions if the name starts with "s:". diff --git a/runtime/doc/vim9.txt b/runtime/doc/vim9.txt --- a/runtime/doc/vim9.txt +++ b/runtime/doc/vim9.txt @@ -149,6 +149,12 @@ with `substitute(` this will use the fun command instead: > :substitute(pattern (replacement ( +Note that while variables need to be defined before they can be used, +functions can be called before being defined. This is required to be able +have cyclic dependencies between functions. It is slightly less efficient, +since the function has to be looked up by name. And a typo in the function +name will only be found when the call is executed. + No curly braces expansion ~ @@ -275,6 +281,8 @@ script, then script-local variables must *:disa* *:disassemble* :disa[ssemble] {func} Show the instructions generated for {func}. This is for debugging and testing. + Note that for command line completion of {func} you + can prepend "s:" to find script-local functions. ============================================================================== diff --git a/src/cmdexpand.c b/src/cmdexpand.c --- a/src/cmdexpand.c +++ b/src/cmdexpand.c @@ -1550,6 +1550,7 @@ set_one_cmd_context( case CMD_function: case CMD_delfunction: + case CMD_disassemble: xp->xp_context = EXPAND_USER_FUNC; xp->xp_pattern = arg; break; @@ -1978,6 +1979,7 @@ ExpandFromContext( regmatch_T regmatch; int ret; int flags; + char_u *tofree = NULL; flags = EW_DIR; // include directories if (options & WILD_LIST_NOTFOUND) @@ -2115,6 +2117,17 @@ ExpandFromContext( if (xp->xp_context == EXPAND_PACKADD) return ExpandPackAddDir(pat, num_file, file); + // When expanding a function name starting with s:, match the nr_ + // prefix. + if (xp->xp_context == EXPAND_USER_FUNC && STRNCMP(pat, "^s:", 3) == 0) + { + int len = (int)STRLEN(pat) + 20; + + tofree = alloc(len); + snprintf((char *)tofree, len, "^\\d\\+_%s", pat + 3); + pat = tofree; + } + regmatch.regprog = vim_regcomp(pat, p_magic ? RE_MAGIC : 0); if (regmatch.regprog == NULL) return FAIL; @@ -2204,6 +2217,7 @@ ExpandFromContext( } vim_regfree(regmatch.regprog); + vim_free(tofree); return ret; } diff --git a/src/testdir/test_cmdline.vim b/src/testdir/test_cmdline.vim --- a/src/testdir/test_cmdline.vim +++ b/src/testdir/test_cmdline.vim @@ -559,6 +559,17 @@ func Test_cmdline_complete_user_cmd() delcommand Foo endfunc +func s:ScriptLocalFunction() + echo 'yes' +endfunc + +func Test_cmdline_complete_user_func() + call feedkeys(":func Test_cmdline_complete_user\\\"\", 'tx') + call assert_match('"func Test_cmdline_complete_user', @:) + call feedkeys(":func s:ScriptL\\\"\", 'tx') + call assert_match('"func \d\+_ScriptLocalFunction', @:) +endfunc + func Test_cmdline_complete_user_names() if has('unix') && executable('whoami') let whoami = systemlist('whoami')[0] diff --git a/src/version.c b/src/version.c --- a/src/version.c +++ b/src/version.c @@ -739,6 +739,8 @@ static char *(features[]) = static int included_patches[] = { /* Add new patch number below this line */ /**/ + 335, +/**/ 334, /**/ 333,