Re: [PATCH v2] Cygwin: pty: Reduce unecessary input transfer.

2021-12-14 Thread Takashi Yano
On Mon, 13 Dec 2021 21:33:31 +0900 Takashi Yano wrote: > On Mon, 13 Dec 2021 18:09:35 +0900 > Takashi Yano wrote: > > On Sat, 11 Dec 2021 22:40:30 +0900 > > Takashi Yano wrote: > > > On Fri, 10 Dec 2021 12:12:44 +0100 (CET) > > > Johannes Schindelin wrote: > > > > On Fri, 10 Dec 2021, Takashi Yano

Re: [PATCH v2] Cygwin: pty: Reduce unecessary input transfer.

2021-12-13 Thread Takashi Yano
On Mon, 13 Dec 2021 18:09:35 +0900 Takashi Yano wrote: > On Sat, 11 Dec 2021 22:40:30 +0900 > Takashi Yano wrote: > > On Fri, 10 Dec 2021 12:12:44 +0100 (CET) > > Johannes Schindelin wrote: > > > On Fri, 10 Dec 2021, Takashi Yano wrote: > > > > Could you please test if the following patch solves th

Re: [PATCH v2] Cygwin: pty: Reduce unecessary input transfer.

2021-12-13 Thread Takashi Yano
On Sat, 11 Dec 2021 22:40:30 +0900 Takashi Yano wrote: > On Fri, 10 Dec 2021 12:12:44 +0100 (CET) > Johannes Schindelin wrote: > > On Fri, 10 Dec 2021, Takashi Yano wrote: > > > Could you please test if the following patch solves the issue? > > > > It does! > > It seems that you already apply thi

Re: [PATCH v2] Cygwin: pty: Reduce unecessary input transfer.

2021-12-11 Thread Takashi Yano
On Fri, 10 Dec 2021 12:12:44 +0100 (CET) Johannes Schindelin wrote: > On Fri, 10 Dec 2021, Takashi Yano wrote: > > Could you please test if the following patch solves the issue? > > It does! It seems that you already apply this patch to msys2, however, this is just an experimental patch to identi

Re: [PATCH v2] Cygwin: pty: Reduce unecessary input transfer.

2021-12-10 Thread Johannes Schindelin
Hi Takashi, On Fri, 10 Dec 2021, Takashi Yano wrote: > On Fri, 10 Dec 2021 00:05:27 +0100 (CET) > Johannes Schindelin wrote: > > sorry for responding to a patch you sent almost 10 months ago... but... I > > am struggling with it. > > > > First of all, let me describe the problem I am seeing (see

Re: [PATCH v2] Cygwin: pty: Reduce unecessary input transfer.

2021-12-10 Thread Takashi Yano
On Fri, 10 Dec 2021 00:05:27 +0100 (CET) Johannes Schindelin wrote: > sorry for responding to a patch you sent almost 10 months ago... but... I > am struggling with it. > > First of all, let me describe the problem I am seeing (see also > https://github.com/git-for-windows/git/issues/3579): after

Re: [PATCH v2] Cygwin: pty: Reduce unecessary input transfer.

2021-12-09 Thread Johannes Schindelin
Hi Takashi, sorry for responding to a patch you sent almost 10 months ago... but... I am struggling with it. First of all, let me describe the problem I am seeing (see also https://github.com/git-for-windows/git/issues/3579): after upgrading the MSYS2 runtime to v3.3.3 in Git for Windows, wheneve

Re: [PATCH v2] Cygwin: pty: Reduce unecessary input transfer.

2021-02-12 Thread Corinna Vinschen via Cygwin-patches
On Feb 11 18:09, Takashi Yano via Cygwin-patches wrote: > - Currently, input transfer is performed every time one line is read(), > if the non-cygwin app is running in the background. With this patch, > transfer is triggered by setpgid() rather than read() so that the > unnecessary input tran

[PATCH v2] Cygwin: pty: Reduce unecessary input transfer.

2021-02-11 Thread Takashi Yano via Cygwin-patches
- Currently, input transfer is performed every time one line is read(), if the non-cygwin app is running in the background. With this patch, transfer is triggered by setpgid() rather than read() so that the unnecessary input transfer can be reduced much in that situation. --- winsup/cygwin/f