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

Reply via email to