Re: [PATCH] Cygwin: pty: Revise code waiting for forwarding by master_fwd_thread.

[Corinna Vinschen <corinna-cygwin at cygwin dot com>]

Hi Takashi,

On Jan 20 11:50, Takashi Yano wrote:
> - Though this rarely happens, sometimes the first printing of non-
>   cygwin process does not displayed correctly. To fix this issue,
>   the code for waiting for forwarding by master_fwd_thread is revised.
> ---
> [...]
> +void
> +fhandler_pty_slave::wait_forwarding (void)
> +{
> +  const DWORD time_to_wait = 40;
> +  DWORD elasped = GetTickCount () - get_ttyp ()->last_fwd_time;
> +  if (elasped < time_to_wait)
> +    Sleep (time_to_wait - elasped);
> +}
> +

Are these 40 ms an experimental value or is that based on knowledge
of implementation details?  The real question is, isn't there any
other, more reliable indicator to see if forwarding will work?


Thanks,
Corinna

-- 
Corinna Vinschen
Cygwin Maintainer

Attachment: signature.asc
Description: PGP signature