On Fri, Feb 21, 2014 at 1:39 PM, Ben Pfaff <b...@nicira.com> wrote:
> On Fri, Feb 21, 2014 at 01:14:35PM -0800, Gurucharan Shetty wrote:
>> +#ifndef _WIN32
>>      pfd.events = POLLOUT;
>> +#else
>> +    pfd.events = POLLWRNORM;
>> +#endif
>
> Does this mean that Windows has POLLWRNORM but not POLLOUT?  POSIX
> says they're equivalent, so we could either use POLLWRNORM everywhere
> or just #define POLLOUT POLLWRNORM in poll-loop.h.  I lean toward the
> latter, since POLLOUT is more "idiomatic" in my experience.
I see that winsock2.h has (The documentation did not mention it):
#define POLLOUT     (POLLWRNORM)

So, I will get rid of the #ifdef

>
> Otherwise,
> Acked-by: Ben Pfaff <b...@nicira.com>
Thanks.
_______________________________________________
dev mailing list
dev@openvswitch.org
http://openvswitch.org/mailman/listinfo/dev

Reply via email to