poll-loop: Ignore 'wevent' in poll_fd_wait_at() on non-Windows.

'wevent' isn't actually used on non-Windows systems, but poll_fd_wait_at()
and find_poll_node() treat events with different 'wevent' as different, so
it seems better to make sure that 'wevent' doesn't matter.

Alternatively, one could ovs_assert(!wevent).  I guess that would catch
a caller accidentally swapping the 'fd' and 'wevent' arguments.

Signed-off-by: Ben Pfaff <blp@nicira.com>
Acked-by: Gurucharan Shetty <gshetty@nicira.com>

diff --git a/lib/poll-loop.c b/lib/poll-loop.c
index 510903e..788fd16 100644 (file)
--- a/lib/poll-loop.c
+++ b/lib/poll-loop.c
@@ -106,6 +106,8 @@ poll_fd_wait_at(int fd, HANDLE wevent, short int events, const char *where)
         VLOG_ERR("No event to wait fd %d", fd);
         return;
     }
+#else
+    wevent = 0;
 #endif
 
     /* Check for duplicate.  If found, "or" the event. */