'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 <b...@nicira.com> --- lib/poll-loop.c | 2 ++ 1 file changed, 2 insertions(+) diff --git a/lib/poll-loop.c b/lib/poll-loop.c index 510903e..788fd16 100644 --- 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. */ -- 1.7.10.4 _______________________________________________ dev mailing list dev@openvswitch.org http://openvswitch.org/mailman/listinfo/dev