diff options
| author | Kenneth Heafield <kheafiel@cluster10.lti.ece.cmu.local> | 2011-03-21 22:00:29 -0400 |
|---|---|---|
| committer | Kenneth Heafield <kheafiel@cluster10.lti.ece.cmu.local> | 2011-03-21 22:00:29 -0400 |
| commit | 2d0c2c2bf7c35db2e16d16f39cb9749a04bc91df (patch) | |
| tree | b2f2a9d1559ab5fc32233fc104a0c4267daeb1b8 /klm/util/bit_packing.hh | |
| parent | 78af1ef80f84023b4ff0661c47201850dbd46363 (diff) | |
compiler warnings
Diffstat (limited to 'klm/util/bit_packing.hh')
| -rw-r--r-- | klm/util/bit_packing.hh | 7 |
1 files changed, 5 insertions, 2 deletions
diff --git a/klm/util/bit_packing.hh b/klm/util/bit_packing.hh index 70cfc2d2..5c71c792 100644 --- a/klm/util/bit_packing.hh +++ b/klm/util/bit_packing.hh @@ -28,16 +28,19 @@ namespace util { * but it may be called multiple times when that's inconvenient. */ -inline uint8_t BitPackShift(uint8_t bit, uint8_t length) { + // Fun fact: __BYTE_ORDER is wrong on Solaris Sparc, but the version without __ is correct. #if BYTE_ORDER == LITTLE_ENDIAN +inline uint8_t BitPackShift(uint8_t bit, uint8_t /*length*/) { return bit; +} #elif BYTE_ORDER == BIG_ENDIAN +inline uint8_t BitPackShift(uint8_t bit, uint8_t length) { return 64 - length - bit; +} #else #error "Bit packing code isn't written for your byte order." #endif -} /* Pack integers up to 57 bits using their least significant digits. * The length is specified using mask: |