On Thu, Sep 20, 2007 at 03:14:51PM +0200, Trond Norbye wrote: [...] > > > >Why not just return if RB_EMPTY() is true rather than adding another level > >of indention? > Some people (I'm not one of them) dislikes multiple return statements > from a function, so I just added the extra level to avoid annoying them ;-)
I'm not one of those people, so I'm rewriting your patch to a simple "if (RB_EMPTY(&base->timetree)) return;" and checking it in. ;) yrs, -- Nick Mathewson
pgpweF3qLpscB.pgp
Description: PGP signature
_______________________________________________ Libevent-users mailing list Libevent-users@monkey.org http://monkey.org/mailman/listinfo/libevent-users