Re: [PATCH] Fix inaccuracies in _brace_parameter completion wording

[Oliver Kiddle <okiddle@xxxxxxxxxxx>]

dana wrote:
> The _brace_parameter completion function indicates that the expansion flags
> B, E, M, and R can be used with the ${foo/bar/baz} syntax. According to the
> documentation (zshexpn), this is not so:

Thanks for this. I've applied the patch.

Looking at that function the description for :? also looks wrong - the
sense is inverted.

Oliver

diff --git a/Completion/Zsh/Context/_brace_parameter b/Completion/Zsh/Context/_brace_parameter
index f22db8895..e1e9adc41 100644
--- a/Completion/Zsh/Context/_brace_parameter
+++ b/Completion/Zsh/Context/_brace_parameter
@@ -195,7 +195,7 @@ elif compset -P '*:'; then
       '+:substitute alternate value if parameter is non-null'
       '=:substitute and assign alternate value if parameter is null'
       '\:=:unconditionally assign value to parameter'
-      '?:print error if parameter is set and non-null'
+      '?:print error if parameter is null'
       '#:filter value matching pattern'
       '/:replace whole word matching pattern'
       '|:set difference'