Zsh Mailing List Archive
Messages sorted by: Reverse Date, Date, Thread, Author

Re: Does add-zle-hook-widget violate the contract of ZLE hook widgets?

Roman Perepelitsa wrote on Thu, Jun 24, 2021 at 12:45:53 +0200:
> On Thu, Jun 24, 2021 at 12:34 PM Marlon Richert
> <marlon.richert@xxxxxxxxx> wrote:
> >
> > [...] the documentation in
> > https://zsh.sourceforge.io/Doc/Release/Zsh-Line-Editor.html#Special-Widgets
> > could be updated to guide the user towards azhw.
> >
> > [...] 3rd-party developers should be strongly
> > encouraged to use azhw and not zle -N zle-X.
> [*] This thread, workers/49084 and workers/49085.

Let's break this down.

This thread is about the fact that a-z-h-w has an «if (( $? )); then
break; fi» in its "loop over all registered hooks and call each in turn"
code.  It has had that check since its first release, and whatever
semantics we desire are easy to implement.  It's not a big deal.

workers/49084 is an edge case involving pre-azhw code trying to wrap
azhw.  It's a teething problem, and shouldn't be too hard to fix.

I haven't fully digested workers/49085, but its author does say there
that a fix is "not too difficult".

> Given the current state of add-zle-hook-widget [*] I would be wary of
> giving this advice. I personally never use it and cannot recommend
> anyone to use it. If your goal is to write a robust plugin that is
> unlikely to break in the presence of other plugins,
> add-zle-hook-widget is a poor choice.

It's not clear to me why you're raising the alarm bells against uses of
a-z-h-w, for two reasons:

One, the specific bugs you cite seem to be easy to fix and uncommon to
run into (they haven't been run into in the 4+ years since a-z-h-w's
first release).

Two, you don't discuss alternatives.  The conclusion "Don't use a-z-h-w"
does not follow from the premise "a-z-h-w has bugs".  It might follow
from a proposition of the form "a-z-h-w is worse than the alternative",
but that was not argued — and, _prima facie_, it's easier to write bugs
without a-z-h-w than with it.

> Roman.

Messages sorted by: Reverse Date, Date, Thread, Author