On 2004.02.23, Rob Crittenden <[EMAIL PROTECTED]> wrote:
> Or here I am with a better fix rather than changing the semantics of an
> existing function (what was I thinking!?).

Seriously!  When I saw the patch against pthread.c, I nearly choked.
:-)

> Still using Dobby's patch, change the way the timeout is set from
>
>    timeout.sec = 10;
>    timeout.usec = 0;
>
> to
>
>    Ns_GetTime(&timeout);
>    timeout.sec += 10;

Yeah, that's what I was thinking.  As you said, maybe the timeout needs
to be configurable ...

Thanks for reminding me that the timeout is absolute time, not relative
time.  Duh.  :-)

Okay, the patch has been updated:

    http://panoptic.com/aolserver/premature-prebound-closing-patch.txt

If this is good, and nobody sees a reason why this shouldn't get
committed, I'll commit into CVS later today.


-- Dossy

--
Dossy Shiobara                       mail: [EMAIL PROTECTED]
Panoptic Computer Network             web: http://www.panoptic.com/
  "He realized the fastest way to change is to laugh at your own
    folly -- then you can let go and quickly move on." (p. 70)


--
AOLserver - http://www.aolserver.com/

To Remove yourself from this list, simply send an email to <[EMAIL PROTECTED]> with the
body of "SIGNOFF AOLSERVER" in the email message. You can leave the Subject: field of 
your email blank.

Reply via email to