diff options
| -rw-r--r-- | ChangeLog | 8 | ||||
| -rw-r--r-- | Completion/Unix/Command/_perforce | 18 |
2 files changed, 23 insertions, 3 deletions
@@ -1,3 +1,9 @@ +2009-11-17 Peter Stephenson <pws@csr.com> + + * unposted: Completion/Unix/Command/_perforce: improve + whole-path completion so that it doesn't take ages looking + for clients after typing two slashes. + 2009-11-15 Peter Stephenson <p.w.stephenson@ntlworld.com> * Greg Klanderman: 27409: Completion/Unix/Command/_subversion, @@ -12348,5 +12354,5 @@ ***************************************************** * This is used by the shell to define $ZSH_PATCHLEVEL -* $Revision: 1.4814 $ +* $Revision: 1.4815 $ ***************************************************** diff --git a/Completion/Unix/Command/_perforce b/Completion/Unix/Command/_perforce index 672c2299e..d451d7002 100644 --- a/Completion/Unix/Command/_perforce +++ b/Completion/Unix/Command/_perforce @@ -1106,8 +1106,22 @@ _perforce_files() { # no point trying to look for unmaintained files. Assume # the user knows what they're doing. local -a altfiles + integer whole_path - if [[ $PREFIX = //[^/]# ]]; then + for type in $types; do + _perforce_whole_path $type && whole_path=1 + done + + # If we're doing whole-path completion, and the user starts + # a completion early, assume they want just those files, + # rather than a client spec. This isn't necessarily the case, + # but there's an excellent chance it does fit the user's intention + # in a case where it's not really worth adding a special option. + # A client list can be huge and they're not actually used very + # often to refer to files. In fact, this whole completion + # probably ought to be optional (you can do it with tags if + # you really want). + if [[ $PREFIX = //[^/]# && $whole_path -eq 0 ]]; then # Complete //clientname spec. Don't complete non-directories... # I don't actually know if they are valid here. altfiles+=("clients:Perforce client:_perforce_clients") @@ -1118,8 +1132,8 @@ _perforce_files() { ! zstyle -t ":completion:${curcontext}:" all-files; then for type in $types; do altfiles+=("$type-files:$type file:_perforce_${type}_files") - _perforce_whole_path $type && nodirs=1 done + (( whole_path )) && nodirs=1 else altfiles+=("depot-files:file in depot:_perforce_depot_files") fi |