diff options
author | Piotr Pawel Stefaniak <pstef@FreeBSD.org> | 2021-09-22 16:23:29 +0000 |
---|---|---|
committer | Piotr Pawel Stefaniak <pstef@FreeBSD.org> | 2021-09-25 08:34:38 +0000 |
commit | ec5849afc86ca6b2df9e7fad5e7d3a1ce92fe714 (patch) | |
tree | c04400dfb425f495e712f880e8ac2005d2365e96 | |
parent | 14fb8056ceef592c6f392f0f678dc45d11b7417e (diff) | |
download | src-ec5849afc86ca6b2df9e7fad5e7d3a1ce92fe714.tar.gz src-ec5849afc86ca6b2df9e7fad5e7d3a1ce92fe714.zip |
sh: improve command completion
When there are many matches, find the longest common substring starting
from the beginning of each command and use that to replace input.
As an example: on my system, llv<tab> will be autocompleted to llvm-
and another <tab> will print all matching llvm commands.
(cherry picked from commit c866d0c798a20b8f0a92df524f4ddd0d81511c88)
-rw-r--r-- | bin/sh/histedit.c | 22 |
1 files changed, 18 insertions, 4 deletions
diff --git a/bin/sh/histedit.c b/bin/sh/histedit.c index 7c366d252743..f3100221d6ad 100644 --- a/bin/sh/histedit.c +++ b/bin/sh/histedit.c @@ -538,7 +538,7 @@ static char const char *dirname; char **matches = NULL; size_t i = 0, size = 16, uniq; - size_t curpos = end - start; + size_t curpos = end - start, lcstring = -1; if (start > 0 || memchr("/.~", text[0], 3) != NULL) return (NULL); @@ -594,11 +594,20 @@ out: if (i > 1) { qsort_s(matches + 1, i, sizeof(matches[0]), comparator, (void *)(intptr_t)curpos); - for (size_t k = 2; k <= i; k++) - if (strcmp(matches[uniq] + curpos, matches[k] + curpos) == 0) + for (size_t k = 2; k <= i; k++) { + const char *l = matches[uniq] + curpos; + const char *r = matches[k] + curpos; + size_t common = 0; + + while (*l != '\0' && *r != '\0' && *l == *r) + (void)l++, r++, common++; + if (common < lcstring) + lcstring = common; + if (*l == *r) free(matches[k]); else matches[++uniq] = matches[k]; + } } matches[uniq + 1] = NULL; /* @@ -610,7 +619,12 @@ out: * string in matches[0] which is the reason to copy the full name of the * only match. */ - matches[0] = strdup(uniq == 1 ? matches[1] : text); + if (uniq == 1) + matches[0] = strdup(matches[1]); + else if (lcstring != (size_t)-1) + matches[0] = strndup(matches[1], curpos + lcstring); + else + matches[0] = strdup(text); if (matches[0] == NULL) { for (size_t k = 1; k <= uniq; k++) free(matches[k]); |