| commit | 9c65ee15ee11c9b9d2fc9d7306fb1e238b35d0c1 | |
| author | Ben Walton <bdwalton@gmail.com> | |
| Fri, 30 May 2014 15:22:40 +0000 (30 16:22 +0100) | ||
| committer | Junio C Hamano <gitster@pobox.com> | |
| Fri, 30 May 2014 18:48:28 +0000 (30 11:48 -0700) | ||
| tree | 28e142253e0dfa9f0541828ca08a2bc59eee73f9 | treesnapshot (tar.gz zip) |
| parent | 839fa9c500c7771d15831e2bcf77697cd70620f9 | commitdiff |
compat/bswap.h: fix endianness detection
The changes to make detection of endianness more portable had a bug
that breaks on (at least) Solaris x86.
The bug appears to be a simple copy/paste typo. It checks for
_BIG_ENDIAN and not _LITTLE_ENDIAN for both the case where we would
decide the system is big endian and little endian. Instead, the
second test should be for _LITTLE_ENDIAN and not _BIG_ENDIAN.
Two fixes were possible:
1. Change the negation order of the conditions in the second test.
2. Reverse the order of the conditions in the second test.
Use the second option so that the condition we expect is always a
positive check.
Signed-off-by: Ben Walton <bdwalton@gmail.com>
Signed-off-by: Junio C Hamano <gitster@pobox.com>
The changes to make detection of endianness more portable had a bug
that breaks on (at least) Solaris x86.
The bug appears to be a simple copy/paste typo. It checks for
_BIG_ENDIAN and not _LITTLE_ENDIAN for both the case where we would
decide the system is big endian and little endian. Instead, the
second test should be for _LITTLE_ENDIAN and not _BIG_ENDIAN.
Two fixes were possible:
1. Change the negation order of the conditions in the second test.
2. Reverse the order of the conditions in the second test.
Use the second option so that the condition we expect is always a
positive check.
Signed-off-by: Ben Walton <bdwalton@gmail.com>
Signed-off-by: Junio C Hamano <gitster@pobox.com>
| compat/bswap.h | diffblobblamehistory |