From 8ffbd4e074bcc66ad183e77218cc584b46f5acad Mon Sep 17 00:00:00 2001 From: Jim Howard Date: Fri, 6 Dec 2013 22:27:00 -0500 Subject: [PATCH] Remove redundant event loop fd processing Reason: https://github.com/antirez/redis/pull/1450#issuecomment-30432925 Closes #1450 --- src/ae.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/src/ae.c b/src/ae.c index 164f8fdeb..63a1ab4eb 100644 --- a/src/ae.c +++ b/src/ae.c @@ -156,8 +156,9 @@ void aeDeleteFileEvent(aeEventLoop *eventLoop, int fd, int mask) { if (fd >= eventLoop->setsize) return; aeFileEvent *fe = &eventLoop->events[fd]; - if (fe->mask == AE_NONE) return; + + aeApiDelEvent(eventLoop, fd, mask); fe->mask = fe->mask & (~mask); if (fd == eventLoop->maxfd && fe->mask == AE_NONE) { /* Update the max fd */ @@ -167,7 +168,6 @@ void aeDeleteFileEvent(aeEventLoop *eventLoop, int fd, int mask) if (eventLoop->events[j].mask != AE_NONE) break; eventLoop->maxfd = j; } - aeApiDelEvent(eventLoop, fd, mask); } int aeGetFileEvents(aeEventLoop *eventLoop, int fd) {