Re: PATCH: Move _history_modifiers call to _parameter

[Sebastian Gniazdowski <sgniazdowski@gmail.com>]

[-- Attachment #1: Type: text/plain, Size: 1622 bytes --]

I created a mew function with your patch applied. Advice on changing from
_pa…terrs to the new function, when at: $ ::<TAB>?

On Thu, 14 Sept 2023 at 07:10, Mikael Magnusson <mikachu@gmail.com> wrote:

> _parameters is used in many contexts, like assignments in command
> position, assignment after typeset, etc, where history modifiers are not
> valid. _brace_parameter already calls _history_modifiers explicitly so
> this seems more in line with that.
> ---
>  Completion/Zsh/Context/_parameter | 5 +++++
>  Completion/Zsh/Type/_parameters   | 5 -----
>  2 files changed, 5 insertions(+), 5 deletions(-)
>
> diff --git a/Completion/Zsh/Context/_parameter
> b/Completion/Zsh/Context/_parameter
> index 7e7788535c..b08d665f58 100644
> --- a/Completion/Zsh/Context/_parameter
> +++ b/Completion/Zsh/Context/_parameter
> @@ -1,3 +1,8 @@
>  #compdef -parameter-
>
> +if compset -P '*:'; then
> +  _history_modifiers p
> +  return
> +fi
> +
>  _parameters -e
> diff --git a/Completion/Zsh/Type/_parameters
> b/Completion/Zsh/Type/_parameters
> index 7f6f33e0ee..7b7692a3c7 100644
> --- a/Completion/Zsh/Type/_parameters
> +++ b/Completion/Zsh/Type/_parameters
> @@ -6,11 +6,6 @@
>  # If you specify a -g option with a pattern, the pattern will be used to
>  # restrict the type of parameters matched.
>
> -if compset -P '*:'; then
> -  _history_modifiers p
> -  return
> -fi
> -
>  local i pfilt
>  local -i nm=$compstate[nmatches]
>  local -a expl pattern=( -g \* ) normal described verbose faked fakes tmp
> --
> 2.38.1
>
>
>

-- 
Best regards,
Sebastian Gniazdowski

[-- Attachment #2: Type: text/html, Size: 2255 bytes --]