diff options
author | Torstein Hegge <hegge@resisty.net> | 2013-03-17 13:35:25 +0100 |
---|---|---|
committer | Frank Terbeck <ft@bewatermyfriend.org> | 2013-04-20 22:29:02 +0200 |
commit | cc7437bf2fa92059564da2e2513980e9c5f687fa (patch) | |
tree | 661f097aab6b4099de6d0f8b5aab17b2b0957655 /Completion/Unix/Command/_git | |
parent | fa7c51971d93d407f50757fbe2f81c91c227878d (diff) | |
download | zsh-cc7437bf2fa92059564da2e2513980e9c5f687fa.tar.gz zsh-cc7437bf2fa92059564da2e2513980e9c5f687fa.tar.xz zsh-cc7437bf2fa92059564da2e2513980e9c5f687fa.zip |
31159: git: Pass prefix filter to ls-files even if it matches no files
When a branch or tag name is completed with zsh in a large git repo, the completion is slow if the given prefix doesn't match a file or directory in the current working directory. Testing with linux.git, which contains release tags like v3.9 and a directory virt/: git log v<tab> takes about 0.5 seconds, while git log v3<tab> takes about 25 seconds. (Timed using zsh 4.3.17, on a fairly slow cpu. zsh from git appears to be quite a bit faster, but the difference between completing v and v3 is still large.) The difference between the two is that v<tab> passes the result of v* to git ls-files while v3<tab> determines that v3* matches no files, and passes an empty prefix to git ls-files. So git ls-files lists all files in the repo and passes that on to _multi_parts. Making git do the expansion of the * after the prefix lets git ls-files v3* return an empty list, making _multi_parts job easier. This does not affect the behavior of git log <tab>, but improves the performance of partial tag and branch tab-completion in the common case where file names and tag/branch names don't overlap.
Diffstat (limited to 'Completion/Unix/Command/_git')
-rw-r--r-- | Completion/Unix/Command/_git | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/Completion/Unix/Command/_git b/Completion/Unix/Command/_git index 32d139acb..a76877180 100644 --- a/Completion/Unix/Command/_git +++ b/Completion/Unix/Command/_git @@ -5345,7 +5345,7 @@ __git_files () { # TODO: --directory should probably be added to $opts when --others is given. local pref=$gitcdup$gitprefix$PREFIX - files=(${(0)"$(_call_program files git ls-files -z --exclude-standard $opts -- ${pref:+$pref\*} 2>/dev/null)"}) + files=(${(0)"$(_call_program files git ls-files -z --exclude-standard $opts -- ${pref:+$pref\\\*} 2>/dev/null)"}) __git_command_successful $pipestatus || return # _wanted $tag expl $description _files -g '{'${(j:,:)files}'}' $compadd_opts - |