lei: completion: bash: generalize nospace usage

We'll be completing more options with ':', '//' and '=' in the
future, so make it easier to disable trailing spaces on
completions.

diff --git a/contrib/completion/lei-completion.bash b/contrib/completion/lei-completion.bash
index 619805fbe2b2f656f1f59d499a4ba232113f9328..2c28d44a56463ab4e56f71fe02d51ae041e19502 100644 (file)
--- a/contrib/completion/lei-completion.bash
+++ b/contrib/completion/lei-completion.bash
@@ -4,14 +4,12 @@
 # preliminary bash completion support for lei (Local Email Interface)
 # Needs a lot of work, see `lei__complete' in lib/PublicInbox::LEI.pm
 _lei() {
-       case ${COMP_WORDS[@]} in
-       *' add-external h'* | *' --mirror h'*)
-               compopt -o nospace
-               ;;
+       local wordlist="$(lei _complete ${COMP_WORDS[@]})"
+       case $wordlist in
+       *':'* | *'='* | '//'*) compopt -o nospace ;;
        *) compopt +o nospace ;; # the default
        esac
-       COMPREPLY=($(compgen -W "$(lei _complete ${COMP_WORDS[@]})" \
-                       -- "${COMP_WORDS[COMP_CWORD]}"))
+       COMPREPLY=($(compgen -W "$wordlist" -- "${COMP_WORDS[COMP_CWORD]}"))
        return 0
 }
 complete -o default -o bashdefault -F _lei lei