[PATCH 2/2] Cygwin: pty: Discard "OSC Ps; ? BEL/ST" in pseudo console output.

Corinna Vinschen corinna-cygwin@cygwin.com
Mon Nov 23 08:58:59 GMT 2020


Hi Takashi,

just a small style nit:

On Nov 23 14:28, Takashi Yano via Cygwin-patches wrote:
> - If vim is executed in WSL in mintty, some garbage string caused
>   by "OSC Ps;? BEL/ST" will be shown in some situations. This patch
>   fixes the issue by removing "OSC Ps;? BEL/ST" from pseudo console
>   output.
> ---
>  winsup/cygwin/fhandler_tty.cc | 30 ++++++++++++++++++++++++++++++
>  1 file changed, 30 insertions(+)
> 
> diff --git a/winsup/cygwin/fhandler_tty.cc b/winsup/cygwin/fhandler_tty.cc
> index 911945675..38285c7f4 100644
> --- a/winsup/cygwin/fhandler_tty.cc
> +++ b/winsup/cygwin/fhandler_tty.cc
> @@ -2069,6 +2069,36 @@ fhandler_pty_master::pty_master_fwd_thread ()
>  	    else
>  	      state = 0;
>  
> +	  /* Remove OSC Ps ; ? BEL/ST */
> +	  for (DWORD i=0; i<rlen; i++)
                     ^^^  ^^^^^^
                     spaces

Thanks,
Corinna


More information about the Cygwin-patches mailing list