On Fri, Oct 31, 2014 at 5:46 PM, Ben Pfaff <b...@nicira.com> wrote: > On Fri, Oct 31, 2014 at 04:53:00PM -0700, Jesse Gross wrote: >> WORDS_BIGENDIAN is defined by configure so it is a more portable >> way to test for endianness. >> >> Reported-by: Ben Pfaff <b...@nicira.com> >> Signed-off-by: Jesse Gross <je...@nicira.com> > > Based on what I see , LITTLE_ENDIAN isn't a test for alignment at all, > it's just a constant. If you want a test for endianness, you have to > write something like "#if BYTE_ORDER == __LITTLE_ENDIAN". > > Acked-by: Ben Pfaff <b...@nicira.com>
Thanks for the additional background - I fixed the commit message to better reflect reality and pushed this. _______________________________________________ dev mailing list dev@openvswitch.org http://openvswitch.org/mailman/listinfo/dev