Re: [PATCH] Timeout support - Please review
- From: "Gustavo J. A. M. Carneiro" <gjc inescporto pt>
- To: Jules Colding <colding omesc com>
- Cc: ORBit2 <orbit-list gnome org>
- Subject: Re: [PATCH] Timeout support - Please review
- Date: Tue, 05 Dec 2006 13:22:19 +0000
On Ter, 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 and even adds a new commandline option which deserves a
> > mention in the changelog too.
>
> Here is the new and better documented patch.
Wow! I wasn't asking for so much level detail... :)
But I guess this is good, if a bit unusual :)
Best regards,
--
Gustavo J. A. M. Carneiro
<gjc inescporto pt> <gustavo users sourceforge net>
The universe is always one step beyond logic.
[
Date Prev][
Date Next] [
Thread Prev][
Thread Next]
[
Thread Index]
[
Date Index]
[
Author Index]