On Fri, Feb 01, 2013 at 11:35:26AM +0900, YAMAMOTO Takashi wrote:
> >> -#ifdef __FreeBSD__
> >> -#define FREE_BSD 1
> >> +#if defined(__FreeBSD__) || defined(__NetBSD__)
> >> +#define FREEBSD_OR_NETBSD 1
> >>  #else
> >> -#define FREE_BSD 0
> >> +#define FREEBSD_OR_NETBSD 0
> >>  #endif
> > 
> > It might be more readable to just use "BSD" in place of
> > FREEBSD_OR_NETBSD, which is a bit long.
> 
> i hesitate to use "BSD" only for FreeBSD and NetBSD given that
> there are too many BSD variants in the world.
> 
> probably a more meaningful name like REGISTER_TUNNEL is better
> in the regard of readability?

OK, that seems reasonable, thanks.
_______________________________________________
dev mailing list
dev@openvswitch.org
http://openvswitch.org/mailman/listinfo/dev

Reply via email to