]> granicus.if.org Git - vim/commitdiff
patch 9.0.1238: :runtime completion can be further improved v9.0.1238
authorzeertzjq <zeertzjq@outlook.com>
Tue, 24 Jan 2023 12:34:03 +0000 (12:34 +0000)
committerBram Moolenaar <Bram@vim.org>
Tue, 24 Jan 2023 12:34:03 +0000 (12:34 +0000)
Problem:    :runtime completion can be further improved.
Solution:   Also complete the {where} argument values and adjust the
            completion for that. (closes #11874)

runtime/doc/builtin.txt
src/cmdexpand.c
src/proto/scriptfile.pro
src/scriptfile.c
src/testdir/test_packadd.vim
src/version.c
src/vim.h

index 4fa6b34fb6e864c6340058b31ad07e3a9f69233b..265111d912cc4255197bf3978beda48d2ae21c9d 100644 (file)
@@ -3528,7 +3528,7 @@ getcompletion({pat}, {type} [, {filtered}])               *getcompletion()*
                messages        |:messages| suboptions
                option          options
                packadd         optional package |pack-add| names
-               runtime         runtime file names |:runtime|
+               runtime         |:runtime| completion
                scriptnames     sourced script names |:scriptnames|
                shellcmd        Shell command
                sign            |:sign| suboptions
index 17f3878a0b11dc09c93add5a833ba0de1f19b067..8ca5ecddc399b1f727b7d979718024868833dff8 100644 (file)
@@ -3025,10 +3025,6 @@ ExpandFromContext(
        return ExpandRTDir(pat, DIP_START + DIP_OPT, numMatches, matches,
                                                                directories);
     }
-    if (xp->xp_context == EXPAND_RUNTIME)
-    {
-       return expand_runtime_cmd(pat, numMatches, matches);
-    }
     if (xp->xp_context == EXPAND_COMPILER)
     {
        char *directories[] = {"compiler", NULL};
@@ -3050,6 +3046,8 @@ ExpandFromContext(
 #endif
     if (xp->xp_context == EXPAND_PACKADD)
        return ExpandPackAddDir(pat, numMatches, matches);
+    if (xp->xp_context == EXPAND_RUNTIME)
+       return expand_runtime_cmd(pat, numMatches, matches);
 
     // When expanding a function name starting with s:, match the <SNR>nr_
     // prefix.
index df89a6aed0d74ed30b9e15a38eafe4ddab5605d3..24e53892e61d0aa32c601bf8198dfb3e3a59e978 100644 (file)
@@ -7,7 +7,6 @@ estack_T *estack_pop(void);
 char_u *estack_sfile(estack_arg_T which);
 void ex_runtime(exarg_T *eap);
 void set_context_in_runtime_cmd(expand_T *xp, char_u *arg);
-int expand_runtime_cmd(char_u *pat, int *numMatches, char_u ***matches);
 int find_script_by_name(char_u *name);
 int get_new_scriptitem_for_fname(int *error, char_u *fname);
 int do_in_path(char_u *path, char_u *name, int flags, void (*callback)(char_u *fname, void *ck), void *cookie);
@@ -21,6 +20,7 @@ void ex_packloadall(exarg_T *eap);
 void ex_packadd(exarg_T *eap);
 void remove_duplicates(garray_T *gap);
 int ExpandRTDir(char_u *pat, int flags, int *num_file, char_u ***file, char *dirnames[]);
+int expand_runtime_cmd(char_u *pat, int *numMatches, char_u ***matches);
 int ExpandPackAddDir(char_u *pat, int *num_file, char_u ***file);
 void ex_source(exarg_T *eap);
 void ex_options(exarg_T *eap);
index aa563d90807cbd9335289bb0278fb5c9a82960c4..630a11696a2ed9f1b9c090c00a9941215f582ac8 100644 (file)
@@ -230,37 +230,35 @@ estack_sfile(estack_arg_T which UNUSED)
 }
 
 /*
- * Get DIP_ flags from the [what] argument of a :runtime command.
- * "*argp" is advanced to after the [what] argument.
+ * Get DIP_ flags from the [where] argument of a :runtime command.
+ * "*argp" is advanced to after the [where] argument if it is found.
  */
     static int
-get_runtime_cmd_flags(char_u **argp)
+get_runtime_cmd_flags(char_u **argp, size_t where_len)
 {
     char_u *arg = *argp;
-    char_u  *p = skiptowhite(arg);
-    int            what_len = (int)(p - arg);
 
-    if (what_len == 0)
+    if (where_len == 0)
        return 0;
 
-    if (STRNCMP(arg, "START", what_len) == 0)
+    if (STRNCMP(arg, "START", where_len) == 0)
     {
-       *argp = skipwhite(arg + what_len);
+       *argp = skipwhite(arg + where_len);
        return DIP_START + DIP_NORTP;
     }
-    if (STRNCMP(arg, "OPT", what_len) == 0)
+    if (STRNCMP(arg, "OPT", where_len) == 0)
     {
-       *argp = skipwhite(arg + what_len);
+       *argp = skipwhite(arg + where_len);
        return DIP_OPT + DIP_NORTP;
     }
-    if (STRNCMP(arg, "PACK", what_len) == 0)
+    if (STRNCMP(arg, "PACK", where_len) == 0)
     {
-       *argp = skipwhite(arg + what_len);
+       *argp = skipwhite(arg + where_len);
        return DIP_START + DIP_OPT + DIP_NORTP;
     }
-    if (STRNCMP(arg, "ALL", what_len) == 0)
+    if (STRNCMP(arg, "ALL", where_len) == 0)
     {
-       *argp = skipwhite(arg + what_len);
+       *argp = skipwhite(arg + where_len);
        return DIP_START + DIP_OPT;
     }
 
@@ -268,15 +266,15 @@ get_runtime_cmd_flags(char_u **argp)
 }
 
 /*
- * ":runtime [what] {name}"
+ * ":runtime [where] {name}"
  */
     void
 ex_runtime(exarg_T *eap)
 {
     char_u  *arg = eap->arg;
     int            flags = eap->forceit ? DIP_ALL : 0;
-
-    flags += get_runtime_cmd_flags(&arg);
+    char_u  *p = skiptowhite(arg);
+    flags += get_runtime_cmd_flags(&arg, p - arg);
     source_runtime(arg, flags);
 }
 
@@ -288,22 +286,13 @@ static int runtime_expand_flags;
     void
 set_context_in_runtime_cmd(expand_T *xp, char_u *arg)
 {
-    runtime_expand_flags = DIP_KEEPEXT + get_runtime_cmd_flags(&arg);
+    char_u  *p = skiptowhite(arg);
+    runtime_expand_flags
+       = *p != NUL ? get_runtime_cmd_flags(&arg, p - arg) : 0;
     xp->xp_context = EXPAND_RUNTIME;
     xp->xp_pattern = arg;
 }
 
-/*
- * Handle command line completion for :runtime command.
- */
-    int
-expand_runtime_cmd(char_u *pat, int *numMatches, char_u ***matches)
-{
-    char *directories[] = {"", NULL};
-    return ExpandRTDir(pat, runtime_expand_flags, numMatches, matches,
-                                                                 directories);
-}
-
     static void
 source_callback(char_u *fname, void *cookie)
 {
@@ -997,44 +986,23 @@ remove_duplicates(garray_T *gap)
        }
 }
 
-/*
- * Expand runtime file names.
- * Search from 'runtimepath':
- *   'runtimepath'/{dirnames}/{pat}.vim
- * When "flags" has DIP_START: search also from "start" of 'packpath':
- *   'packpath'/pack/ * /start/ * /{dirnames}/{pat}.vim
- * When "flags" has DIP_OPT: search also from "opt" of 'packpath':
- *   'packpath'/pack/ * /opt/ * /{dirnames}/{pat}.vim
- * "dirnames" is an array with one or more directory names.
- */
-    int
-ExpandRTDir(
+    static void
+ExpandRTDir_int(
     char_u     *pat,
+    size_t     pat_len,
     int                flags,
-    int                *num_file,
-    char_u     ***file,
+    int                keep_ext,
+    garray_T   *gap,
     char       *dirnames[])
 {
-    char_u     *s;
-    char_u     *e;
-    char_u     *match;
-    garray_T   ga;
-    int                i;
-    int                pat_len;
-
-    *num_file = 0;
-    *file = NULL;
-    pat_len = (int)STRLEN(pat);
-    ga_init2(&ga, sizeof(char *), 10);
-
-    for (i = 0; dirnames[i] != NULL; ++i)
+    for (int i = 0; dirnames[i] != NULL; ++i)
     {
        size_t          buf_len = STRLEN(dirnames[i]) + pat_len + 22;
        char            *buf = alloc(buf_len);
        if (buf == NULL)
        {
-           ga_clear_strings(&ga);
-           return FAIL;
+           ga_clear_strings(gap);
+           return;
        }
        char            *tail = buf + 15;
        size_t          tail_buflen = buf_len - 15;
@@ -1048,18 +1016,18 @@ ExpandRTDir(
 
 expand:
        if ((flags & DIP_NORTP) == 0)
-           globpath(p_rtp, (char_u *)tail, &ga, glob_flags, expand_dirs);
+           globpath(p_rtp, (char_u *)tail, gap, glob_flags, expand_dirs);
 
        if (flags & DIP_START)
        {
            memcpy(tail - 15, "pack/*/start/*/", 15);
-           globpath(p_pp, (char_u *)tail - 15, &ga, glob_flags, expand_dirs);
+           globpath(p_pp, (char_u *)tail - 15, gap, glob_flags, expand_dirs);
        }
 
        if (flags & DIP_OPT)
        {
            memcpy(tail - 13, "pack/*/opt/*/", 13);
-           globpath(p_pp, (char_u *)tail - 13, &ga, glob_flags, expand_dirs);
+           globpath(p_pp, (char_u *)tail - 13, gap, glob_flags, expand_dirs);
        }
 
        if (*(dirnames[i]) == NUL && !expand_dirs)
@@ -1075,17 +1043,16 @@ expand:
     }
 
     int pat_pathsep_cnt = 0;
-    for (i = 0; i < pat_len; ++i)
+    for (size_t i = 0; i < pat_len; ++i)
        if (vim_ispathsep(pat[i]))
            ++pat_pathsep_cnt;
 
-    for (i = 0; i < ga.ga_len; ++i)
+    for (int i = 0; i < gap->ga_len; ++i)
     {
-       match = ((char_u **)ga.ga_data)[i];
-       s = match;
-       e = s + STRLEN(s);
-       if (e - 4 > s && (flags & DIP_KEEPEXT) == 0
-                                           && STRNICMP(e - 4, ".vim", 4) == 0)
+       char_u *match = ((char_u **)gap->ga_data)[i];
+       char_u *s = match;
+       char_u *e = s + STRLEN(s);
+       if (e - 4 > s && !keep_ext && STRNICMP(e - 4, ".vim", 4) == 0)
        {
            e -= 4;
            *e = NUL;
@@ -1097,22 +1064,89 @@ expand:
                                     && ++match_pathsep_cnt > pat_pathsep_cnt))
                break;
        ++s;
-       *e = NUL;
-       mch_memmove(match, s, e - s + 1);
+       if (s != match)
+           mch_memmove(match, s, e - s + 1);
     }
 
-    if (ga.ga_len == 0)
-       return FAIL;
+    if (gap->ga_len == 0)
+       return;
 
     // Sort and remove duplicates which can happen when specifying multiple
     // directories in dirnames.
-    remove_duplicates(&ga);
+    remove_duplicates(gap);
+}
+
+/*
+ * Expand runtime file names.
+ * Search from 'runtimepath':
+ *   'runtimepath'/{dirnames}/{pat}.vim
+ * When "flags" has DIP_START: search also from "start" of 'packpath':
+ *   'packpath'/pack/ * /start/ * /{dirnames}/{pat}.vim
+ * When "flags" has DIP_OPT: search also from "opt" of 'packpath':
+ *   'packpath'/pack/ * /opt/ * /{dirnames}/{pat}.vim
+ * "dirnames" is an array with one or more directory names.
+ */
+    int
+ExpandRTDir(
+    char_u     *pat,
+    int                flags,
+    int                *num_file,
+    char_u     ***file,
+    char       *dirnames[])
+{
+    *num_file = 0;
+    *file = NULL;
+
+    garray_T   ga;
+    ga_init2(&ga, sizeof(char *), 10);
+
+    ExpandRTDir_int(pat, STRLEN(pat), flags, FALSE, &ga, dirnames);
+
+    if (ga.ga_len == 0)
+       return FAIL;
 
     *file = ga.ga_data;
     *num_file = ga.ga_len;
     return OK;
 }
 
+/*
+ * Handle command line completion for :runtime command.
+ */
+    int
+expand_runtime_cmd(char_u *pat, int *numMatches, char_u ***matches)
+{
+    *numMatches = 0;
+    *matches = NULL;
+
+    garray_T   ga;
+    ga_init2(&ga, sizeof(char *), 10);
+
+    size_t pat_len = (int)STRLEN(pat);
+    char *dirnames[] = {"", NULL};
+    ExpandRTDir_int(pat, pat_len, runtime_expand_flags, TRUE, &ga, dirnames);
+
+    // Try to complete values for [where] argument when none was found.
+    if (runtime_expand_flags == 0)
+    {
+       char *where_values[] = {"START", "OPT", "PACK", "ALL"};
+       for (size_t i = 0; i < ARRAY_LENGTH(where_values); ++i)
+           if (STRNCMP(pat, where_values[i], pat_len) == 0)
+           {
+               char_u *p = vim_strsave((char_u *)where_values[i]);
+               if (p != NULL && ga_add_string(&ga, p) == FAIL)
+                   vim_free(p);
+           }
+    }
+
+    if (ga.ga_len == 0)
+       return FAIL;
+
+    *matches = ga.ga_data;
+    *numMatches = ga.ga_len;
+    return OK;
+}
+
 /*
  * Expand loadplugin names:
  * 'packpath'/pack/ * /opt/{pat}
index b5583ae73ab96f8694a49952cc2deb4f7d8dec45..332f0094c3477fdefbb7ca487f0815c74c34596b 100644 (file)
@@ -370,63 +370,75 @@ func Test_runtime()
 endfunc
 
 func Test_runtime_completion()
-  let rundir = &packpath . '/runtime/Xextra'
-  let startdir = &packpath . '/pack/mine/start/foo/Xextra'
-  let optdir = &packpath . '/pack/mine/opt/bar/Xextra'
-  call mkdir(rundir . '/Xrunbaz', 'p')
-  call mkdir(startdir . '/Xstartbaz', 'p')
-  call mkdir(optdir . '/Xoptbaz', 'p')
-  call writefile([], rundir . '/../Xrunfoo.vim')
-  call writefile([], rundir . '/Xrunbar.vim')
-  call writefile([], rundir . '/Xunrelated')
-  call writefile([], rundir . '/../Xunrelated')
-  call writefile([], startdir . '/../Xstartfoo.vim')
-  call writefile([], startdir . '/Xstartbar.vim')
-  call writefile([], startdir . '/Xunrelated')
-  call writefile([], startdir . '/../Xunrelated')
-  call writefile([], optdir . '/../Xoptfoo.vim')
-  call writefile([], optdir . '/Xoptbar.vim')
-  call writefile([], optdir . '/Xunrelated')
-  call writefile([], optdir . '/../Xunrelated')
+  let rundir = &packpath . '/runtime/Aextra'
+  let startdir = &packpath . '/pack/mine/start/foo/Aextra'
+  let optdir = &packpath . '/pack/mine/opt/bar/Aextra'
+  call mkdir(rundir . '/Arunbaz', 'p')
+  call mkdir(startdir . '/Astartbaz', 'p')
+  call mkdir(optdir . '/Aoptbaz', 'p')
+  call writefile([], rundir . '/../Arunfoo.vim')
+  call writefile([], rundir . '/Arunbar.vim')
+  call writefile([], rundir . '/Aunrelated')
+  call writefile([], rundir . '/../Aunrelated')
+  call writefile([], startdir . '/../Astartfoo.vim')
+  call writefile([], startdir . '/Astartbar.vim')
+  call writefile([], startdir . '/Aunrelated')
+  call writefile([], startdir . '/../Aunrelated')
+  call writefile([], optdir . '/../Aoptfoo.vim')
+  call writefile([], optdir . '/Aoptbar.vim')
+  call writefile([], optdir . '/Aunrelated')
+  call writefile([], optdir . '/../Aunrelated')
   exe 'set rtp=' . &packpath . '/runtime'
 
   func Check_runtime_completion(arg, arg1, res)
     call feedkeys(':runtime ' .. a:arg .. "\<C-A>\<C-B>\"\<CR>", 'xt')
     call assert_equal('"runtime ' .. a:arg1 .. join(a:res), @:)
     call assert_equal(a:res, getcompletion(a:arg, 'runtime'))
-
-    call feedkeys(':runtime ' .. a:arg .. "X\<C-A>\<C-B>\"\<CR>", 'xt')
-    call assert_equal('"runtime ' .. a:arg1 .. join(a:res), @:)
-    call assert_equal(a:res, getcompletion(a:arg .. 'X', 'runtime'))
   endfunc
 
   call Check_runtime_completion('', '',
-        \ ['Xextra/', 'Xrunfoo.vim'])
-  call Check_runtime_completion('Xextra/', '',
-        \ ['Xextra/Xrunbar.vim', 'Xextra/Xrunbaz/'])
+        \ ['Aextra/', 'Arunfoo.vim', 'START', 'OPT', 'PACK', 'ALL'])
+  call Check_runtime_completion('S', '',
+        \ ['START'])
+  call Check_runtime_completion('O', '',
+        \ ['OPT'])
+  call Check_runtime_completion('P', '',
+        \ ['PACK'])
+  call Check_runtime_completion('A', '',
+        \ ['Aextra/', 'Arunfoo.vim', 'ALL'])
+  call Check_runtime_completion('Aextra/', '',
+        \ ['Aextra/Arunbar.vim', 'Aextra/Arunbaz/'])
 
   call Check_runtime_completion('START ', 'START ',
-        \ ['Xextra/', 'Xstartfoo.vim'])
-  call Check_runtime_completion('START Xextra/', 'START ',
-        \ ['Xextra/Xstartbar.vim', 'Xextra/Xstartbaz/'])
+        \ ['Aextra/', 'Astartfoo.vim'])
+  call Check_runtime_completion('START A', 'START ',
+        \ ['Aextra/', 'Astartfoo.vim'])
+  call Check_runtime_completion('START Aextra/', 'START ',
+        \ ['Aextra/Astartbar.vim', 'Aextra/Astartbaz/'])
 
   call Check_runtime_completion('OPT ', 'OPT ',
-        \ ['Xextra/', 'Xoptfoo.vim'])
-  call Check_runtime_completion('OPT Xextra/', 'OPT ',
-        \ ['Xextra/Xoptbar.vim', 'Xextra/Xoptbaz/'])
+        \ ['Aextra/', 'Aoptfoo.vim'])
+  call Check_runtime_completion('OPT A', 'OPT ',
+        \ ['Aextra/', 'Aoptfoo.vim'])
+  call Check_runtime_completion('OPT Aextra/', 'OPT ',
+        \ ['Aextra/Aoptbar.vim', 'Aextra/Aoptbaz/'])
 
   call Check_runtime_completion('PACK ', 'PACK ',
-        \ ['Xextra/', 'Xoptfoo.vim', 'Xstartfoo.vim'])
-  call Check_runtime_completion('PACK Xextra/', 'PACK ',
-        \ ['Xextra/Xoptbar.vim', 'Xextra/Xoptbaz/',
-        \ 'Xextra/Xstartbar.vim', 'Xextra/Xstartbaz/'])
+        \ ['Aextra/', 'Aoptfoo.vim', 'Astartfoo.vim'])
+  call Check_runtime_completion('PACK A', 'PACK ',
+        \ ['Aextra/', 'Aoptfoo.vim', 'Astartfoo.vim'])
+  call Check_runtime_completion('PACK Aextra/', 'PACK ',
+        \ ['Aextra/Aoptbar.vim', 'Aextra/Aoptbaz/',
+        \ 'Aextra/Astartbar.vim', 'Aextra/Astartbaz/'])
 
   call Check_runtime_completion('ALL ', 'ALL ',
-        \ ['Xextra/', 'Xoptfoo.vim', 'Xrunfoo.vim', 'Xstartfoo.vim'])
-  call Check_runtime_completion('ALL Xextra/', 'ALL ',
-        \ ['Xextra/Xoptbar.vim', 'Xextra/Xoptbaz/',
-        \ 'Xextra/Xrunbar.vim', 'Xextra/Xrunbaz/',
-        \ 'Xextra/Xstartbar.vim', 'Xextra/Xstartbaz/'])
+        \ ['Aextra/', 'Aoptfoo.vim', 'Arunfoo.vim', 'Astartfoo.vim'])
+  call Check_runtime_completion('ALL A', 'ALL ',
+        \ ['Aextra/', 'Aoptfoo.vim', 'Arunfoo.vim', 'Astartfoo.vim'])
+  call Check_runtime_completion('ALL Aextra/', 'ALL ',
+        \ ['Aextra/Aoptbar.vim', 'Aextra/Aoptbaz/',
+        \ 'Aextra/Arunbar.vim', 'Aextra/Arunbaz/',
+        \ 'Aextra/Astartbar.vim', 'Aextra/Astartbaz/'])
 
   delfunc Check_runtime_completion
 endfunc
index 30e9819965a93b6853f76ba7a437906de36d3910..9eff9696b1b496cb89c474db4cd8d0378b189ffc 100644 (file)
@@ -695,6 +695,8 @@ static char *(features[]) =
 
 static int included_patches[] =
 {   /* Add new patch number below this line */
+/**/
+    1238,
 /**/
     1237,
 /**/
index 258c8955b72b0d7346909a463ed5ac8ca501382e..67bb23e515955a8c4906366ca521225c233005a6 100644 (file)
--- a/src/vim.h
+++ b/src/vim.h
@@ -2672,7 +2672,6 @@ typedef enum {
 #define DIP_NORTP   0x20       // do not use 'runtimepath'
 #define DIP_NOAFTER 0x40       // skip "after" directories
 #define DIP_AFTER   0x80       // only use "after" directories
-#define DIP_KEEPEXT  0x100     // for completion: include file extension
 
 // Lowest number used for window ID. Cannot have this many windows.
 #define LOWEST_WIN_ID 1000