Mailing-List: contact cygwin-help AT cygwin DOT com; run by ezmlm
List-Subscribe: <mailto:cygwin-subscribe AT cygwin DOT com>
List-Archive: <http://sourceware.org/ml/cygwin/>
List-Post: <mailto:cygwin AT cygwin DOT com>
List-Help: <mailto:cygwin-help AT cygwin DOT com>, <http://sourceware.org/ml/#faqs>
Sender: cygwin-owner AT cygwin DOT com
Mail-Followup-To: cygwin AT cygwin DOT com
Delivered-To: mailing list cygwin AT cygwin DOT com
Date: Thu, 07 Oct 2004 11:11:09 -0600
From: Mark Paulus <mark DOT paulus AT mci DOT com>
Subject: Re: Patch for fhandler_tty.cc
In-reply-to: <20041007080149.GA21753@cygbert.vinschen.de>
To: "cygwin AT cygwin DOT com" <cygwin AT cygwin DOT com>
Message-id: <0I58003H952LPM@pmismtp01.mcilink.com>
MIME-version: 1.0
Content-type: text/plain; charset=iso-8859-1
Content-transfer-encoding: 7bit
Priority: Normal
X-IsSubscribed: yes

Corinna,

Thanks for applying this, and I apologize for not following
protocol.  Subsequent patches will go to cygwin-patches,
and will be accompanied by ChangeLog entries as well.

On Thu, 07 Oct 2004 10:01:49 +0200, Corinna Vinschen wrote:

>On Oct  6 12:10, Mark Paulus wrote:
>> --- fhandler_tty_old.cc 2004-10-06 12:05:54.196444800 -0600
>> +++ fhandler_tty.cc     2004-10-06 12:06:13.465645500 -0600
>> @@ -766,7 +766,7 @@ fhandler_tty_slave::read (void *ptr, siz
>>    w4[0] = signal_arrived;
>>    w4[1] = input_available_event;
>> 
>> -  DWORD waiter = !ptr ? 0 : INFINITE;
>> +  DWORD waiter = time_to_wait;
>>    while (len)
>>      {
>>        rc = WaitForMultipleObjects (2, w4, FALSE, waiter);

>Looks good.  I've applied it.  Just a matching ChangeLog entry would
>have been nice.


>Thanks,
>Corinna

>-- 
>Corinna Vinschen                  Please, send mails regarding Cygwin to
>Cygwin Project Co-Leader          mailto:cygwin AT cygwin DOT com
>Red Hat, Inc.

>--
>Unsubscribe info:      http://cygwin.com/ml/#unsubscribe-simple
>Problem reports:       http://cygwin.com/problems.html
>Documentation:         http://cygwin.com/docs.html
>FAQ:                   http://cygwin.com/faq/





--
Unsubscribe info:      http://cygwin.com/ml/#unsubscribe-simple
Problem reports:       http://cygwin.com/problems.html
Documentation:         http://cygwin.com/docs.html
FAQ:                   http://cygwin.com/faq/