This is the mail archive of the cygwin-patches mailing list for the Cygwin project.

Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]
Other format: [Raw text]

Re: [PATCH v3 0/1] Cygwin: pty: Fix the behaviour of Ctrl-C in the pseudo console mode.

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/ |  33 +++++----
>   winsup/cygwin/       |   2 +-
>   winsup/cygwin/        | 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.


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.

Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]