From ace780c00252b7ab9d3296443b2b9572dfb40af2 Mon Sep 17 00:00:00 2001 From: antirez Date: Mon, 4 Apr 2016 12:23:10 +0200 Subject: [PATCH] ae.c: comment to explain why we have a useless maxId check. --- src/ae.c | 6 +++++- 1 file changed, 5 insertions(+), 1 deletion(-) diff --git a/src/ae.c b/src/ae.c index 79fcde62a..bee9d4ad6 100644 --- a/src/ae.c +++ b/src/ae.c @@ -303,7 +303,11 @@ static int processTimeEvents(aeEventLoop *eventLoop) { continue; } - /* Don't process time events created by time events in this iteration. */ + /* Make sure we don't process time events created by time events in + * this iteration. Note that this check is currently useless: we always + * add new timers on the head, however if we change the implementation + * detail, this check may be useful again: we keep it here for future + * defense. */ if (te->id > maxId) { te = te->next; continue;