Re: [PATCH] Timeout support - Please review



On Tue, 2006-12-05 at 13:06 +0100, Jules Colding wrote:
> On Tue, 2006-12-05 at 10:47 +0000, Gustavo J. A. M. Carneiro wrote:
> > On Ter, 2006-12-05 at 09:50 +0100, Jules Colding wrote:
> > > On Mon, 2006-12-04 at 14:22 +0100, Jules Colding wrote:
> > > > Please review and test this patch. I'm not entirely convinced that I
> > > > haven't introduced a bad bug or two but at least it works(*) here.
> > > 
> > > No responses yet, so it seems that my patch is of the usual flawless
> > > quality ;-) 
> > 
> >   Heh :)
> >   
> >   In fact, the patch sounds really nice, a sign of ORBit2 maturing for
> > TCP/IP connections (which is traditionally seldom tested).
> > 
> >   My only comment is that the changelog entry could use a bit more
> > detail.  You only mention changes in one function, but the patch changes
> > 4 functions 

BTW: The changes to link_wait() is in linc2/ChangeLog.

-- 
  jules




[Date Prev][Date Next]   [Thread Prev][Thread Next]   [Thread Index] [Date Index] [Author Index]