On Thu, Sep 20, 2007 at 01:52:31PM +0200, Trond Norbye wrote:
> The result of gettime is not used if the timetree is empty (and might
> cause a context switch to get the system clock).
>
> --Trond
> Index: event.c
> ===================================================================
> --- event.c (revision 440)
> +++ event.c (working copy)
> @@ -804,24 +804,26 @@
> void
> timeout_process(struct event_base *base)
> {
> - struct timeval now;
> - struct event *ev, *next;
> + if (!RB_EMPTY(&base->timetree)) {
> + struct timeval now;
> + struct event *ev, *next;
Why not just return if RB_EMPTY() is true rather than adding another level
of indention? Other than that, I also noticed this but didn't consider it as
anything significant. It has never once showed up as any significant source
of cycles in profiling either.
-cl
_______________________________________________
Libevent-users mailing list
[email protected]
http://monkey.org/mailman/listinfo/libevent-users
