From: Danny Al-Gaaf <danny.al-gaaf@bisect.de>
Date: Tue, 3 Feb 2015 21:36:27 +0000 (+0100)
Subject: EventKqueue.cc: clarify calculation precedence for '&' and '?'
X-Git-Tag: suse_latest~36^2~9^2~2
X-Git-Url: http://git-server-git.apps.pok.os.sepia.ceph.com/?a=commitdiff_plain;h=aa7c968ae5fbbf1ade6bb89e215a678a0a6ba855;p=ceph.git

EventKqueue.cc: clarify calculation precedence for '&' and '?'

Signed-off-by: Danny Al-Gaaf <danny.al-gaaf@bisect.de>
---

diff --git a/src/msg/async/EventKqueue.cc b/src/msg/async/EventKqueue.cc
index 10156da1775..d0056f1e51c 100644
--- a/src/msg/async/EventKqueue.cc
+++ b/src/msg/async/EventKqueue.cc
@@ -50,8 +50,8 @@ int KqueueDriver::add_event(int fd, int cur_mask, int add_mask)
                  << "add_mask" << add_mask << dendl;
   struct kevent ke;
   int filter = 0;
-  filter |= add_mask & EVENT_READABLE ? EVFILT_READ : 0;
-  filter |= add_mask & EVENT_WRITABLE ? EVFILT_WRITE : 0;
+  filter |= (add_mask & EVENT_READABLE) ? EVFILT_READ : 0;
+  filter |= (add_mask & EVENT_WRITABLE) ? EVFILT_WRITE : 0;
 
   if (filter) {
     EV_SET(&ke, fd, filter, EV_ADD, 0, 0, NULL);
@@ -72,8 +72,8 @@ void KqueueDriver::del_event(int fd, int cur_mask, int delmask)
   struct kevent ee;
   struct kevent ke;
   int filter = 0;
-  filter |= delmask & EVENT_READABLE ? EVFILT_READ : 0;
-  filter |= delmask & EVENT_WRITABLE ? EVFILT_WRITE : 0;
+  filter |= (delmask & EVENT_READABLE) ? EVFILT_READ : 0;
+  filter |= (delmask & EVENT_WRITABLE) ? EVFILT_WRITE : 0;
 
   if (filter) {
     EV_SET(&ke, fd, filter, EV_DELETE, 0, 0, NULL);