Re: [PATCH v2 0/1] MR5691: timeout: Add new program.

[Myah Caron (＠qsniyg)]

On Mon May 20 21:34:08 2024 +0000, eric pouech wrote:
> a couple of comments
> (and yes msvcrt's kbhit looks wrong, as it expects the last event to be
> a key up, which is not always the case; as noted above, _kbhit() doesn't
> behave as timeout.exe expects)

Thanks for taking the time to look at this. I've fixed everything except for the resources.

There are two more differences I've noticed: CTRL+C in `nobreak` mode is actually trapped, and a newline is printed. Also, `waiting for N seconds` is whitespace-padded when the number is larger than 10.

However, implementing either of these feels like more work than it's worth to me, as they're just minor visual changes.