Re: [PATCH] _add-zsh-hook: add -L

[Oliver Kiddle <okiddle@xxxxxxxxxxx>]

Daniel Hahler wrote:
>
> --- a/Completion/Zsh/Function/_add-zsh-hook

> @@ -14,8 +14,9 @@ _add-zsh-hook() {
> -    '(-D)-d[remove HOOK from the array]' \
> -    '(-d)-D[interpret HOOK as pattern to remove from the array]' \
> +    '(-D,-L)-d[remove HOOK from the array]' \
> +    '(-d,-L)-D[interpret HOOK as pattern to remove from the array]' \
> +    '(-d,-D)-L[list current values of the array with typeset]' \

You need spaces rather commas inside the exclusion list.

I'd also convert HOOK to lowercase while you're there. Those
descriptions look like they have been copied and pasted from
documentation where full usage has been presented first - something
like:

    add-zsh-hook [ -L | -dD ] [ -Uzk ] HOOK FUNCTION

In completion, we're not presenting the information that the first
normal argument is 'HOOK'.

Oliver