Re: PATCH: query terminal properties on ZLE startup

[Oliver Kiddle <opk@xxxxxxx>]

Bart Schaefer wrote:
> I'd like to see Roman's startup time stats run again with this in
> place.  If querying the terminal significantly delays the printing of
> the first prompt, I'd think this should at least be something that can
> be setopt'd off and explicitly invoked later.

I'm not sure what Roman's startup time stats did precisely but it is
tiny in comparison to the sorts of times you get for starting completion
or any of the more substantial plugins.

Doing `time zmodload zsh/zle` registers a time of 0.000 most of the time
both with and without the patch.

The experiments I did to refine the timeout to use on the reads would
fairly accurately reflect the time for just the terminal queries and
I was intentionally doing those on a 9600 serial link. In a local
xterm/whatever it completes within the 1/100th second that you
get by setting the TIMEOUT constant to -1. Over the serial line 0.2s was
about the minimum to be guaranteed of completion. The timeout provides a
worst case. We could reduce that and perhaps make sure that the key
input code will swallow late arriving terminal responses.

In the current form, there's no setopt but you can disable it all with:

  .term.extensions=( -query ).

Oliver