[Toybox] [PATCH] microcom, stty: Use TCSADRAIN to set tty device attribute

[enh]

On Fri, May 31, 2024, 11:08 Rob Landley <rob at landley.net> wrote:

> On 5/20/24 23:43, Yi-Yo Chiang via Toybox wrote:
> > Don't flush the tty device input queue when setting device attribute.
> >
> > Rationale:
> >   * microcom: The tty device might already have a _good enough_ termios
> >     to read data from. Flushing the input queue just to set the terminal
> >     attribute would result in data loss in this case.
> >   * stty: This command doesn't read nor write data to the tty device, so
> >     flushing the input queue doesn't make sense here. The program
> >     actually talking to the tty should decide if it wants the tty
> >     flushed or not.
> >     Note: other implementations of stty also uses TCSANOW (bsd) or
> >     TCSADRAIN (coreutils), so I think this assumption is fine.
>
> Was commit 2043855a4bd5 sufficient or do you still have a use case that
> needs
> this? (Email dated the 20th containing a patch dated the 21st vs a git
> commit
> applied the 23rd...)
>

yochiang: was this for the host? do you need me to update the host prebuilt?

Rob
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.landley.net/pipermail/toybox-landley.net/attachments/20240531/4b216b5f/attachment.htm>