On Wed, Oct 22, 2014 at 04:06:35PM +0000, Nithin Raju wrote:
> On Oct 22, 2014, at 8:58 AM, Ben Pfaff <b...@nicira.com>
>  wrote:
> 
> > I'm not sure I'm happy about this.
> > 
> > For the only uses of this macro introduced in this patch series, which
> > are assertions on variables that are always present, the code does not
> > have to be Windows-only after the preprocessor.  One could simply write:
> > 
> >    #ifdef _WIN32
> >    enum { WINDOWS = 1 };
> >    #else
> >    enum { WINDOWS = 0 };
> >    #endif
> > 
> > and then add "WINDOWS &&" to the assertions.
> 
> hi Ben,
> That is a good suggestion. All I want to do is to avoid peppering the code 
> with #ifdef _WIN32 for one line code.
> 
> I'll add the enum you suggested to dpif-netlink.c since that is where we 
> primarily need it. We can move it to util.h when/if we need to.
> 
> I'll incorporate your comments in v2.

Sounds good, thanks.
_______________________________________________
dev mailing list
dev@openvswitch.org
http://openvswitch.org/mailman/listinfo/dev

Reply via email to