Re: [PATCH 2/2] Send alive/byebye announcements three times.



On Thu, 2011-06-16 at 19:03 +0200, Jens Georg wrote:
> > >> Hm, yes. That is indeed more clever than my version :)
> > >
> > > I will revert your change then and apply this patch instead. Is that OK
> > > with you?
> > 
> >   Please dont! While your patch is cleverer, its also slightly more complex.
> > 
> 
> I also just realized that it also applies to ALIVE only and not BYEBYE
> as well.

I wasn't aware that the spec also recommends this behavior for BYEBYE
messages. Will have to look this up again and if that's the recommended
behavior, then I'll change my patch accordingly.


Sven




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