Hi Takashi,

On 9/6/2019 10:51 AM, Takashi Yano wrote:
> - When the I/O pipe is switched to the pseudo console side, the
>    behaviour of Ctrl-C is unstable. This rarely happens, however,
>    for example, shell sometimes crashes by Ctrl-C in that situation.
>    This patch fixes that issue.
> 
> v3:
> Fix mistake in v2.
> 
> v2:
> Remove the code which accidentally clears ENABLE_ECHO_INPUT flag.
> 
> Takashi Yano (1):
>    Cygwin: pty: Fix the behaviour of Ctrl-C in the pseudo console mode.
> 
>   winsup/cygwin/fhandler.h      |   4 +-
>   winsup/cygwin/fhandler_tty.cc |  33 +++++----
>   winsup/cygwin/select.cc       |   2 +-
>   winsup/cygwin/spawn.cc        | 128 +++++++++++++++++-----------------
>   4 files changed, 89 insertions(+), 78 deletions(-)

I had several problems after applying this patch.

1. I noticed some display glitches when building cygwin (with -j13 if that's 
relevant).  For example, there were some unexpected blank lines and indented 
lines.

2. At one point the build wouldn't complete at all.  It hung and had to be 
killed with Ctrl-C.

3. I used ssh from my normal account to log into an administrator account.  I 
ran a script that produced a lot of output and piped it to less.  I pressed 'q' 
after the first screen was displayed, and the displayed text didn't get cleared.

Ken

P.S. I'm leaving tomorrow for a short vacation, so I might not have time to 
review any more patches until I return in about a week.

Reply via email to