On Fri, Oct 31, 2014 at 5:46 PM, Ben Pfaff <[email protected]> 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 <[email protected]> >> Signed-off-by: Jesse Gross <[email protected]> > > 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 <[email protected]>
Thanks for the additional background - I fixed the commit message to better reflect reality and pushed this. _______________________________________________ dev mailing list [email protected] http://openvswitch.org/mailman/listinfo/dev
