endian: use byteswap.h where available
authorRusty Russell <rusty@rustcorp.com.au>
Tue, 1 Mar 2011 07:21:22 +0000 (17:51 +1030)
committerRusty Russell <rusty@rustcorp.com.au>
Tue, 1 Mar 2011 07:21:22 +0000 (17:51 +1030)
And use those names, too.

ccan/endian/_info
ccan/endian/endian.h
ccan/endian/test/run.c

index d9d173c32d74fd10bf4b2d2ae1470c0d4c1aee1d..a239d253a14a2d5c7d47ee2daa9523489def9dff 100644 (file)
@@ -33,7 +33,7 @@
  *             printf("native:        %08x\n", value);
  *             printf("little-endian: %08x\n", cpu_to_le32(value));
  *             printf("big-endian:    %08x\n", cpu_to_be32(value));
- *             printf("byte-reversed: %08x\n", swab_u32(value));
+ *             printf("byte-reversed: %08x\n", bswap_32(value));
  *             exit(0);
  *     }
  *
index 81db5d948f368505375a81f32fee7591d065973d..baee60be144e624b7dee3f61118f3090ba6ec3c2 100644 (file)
@@ -3,33 +3,51 @@
 #include <stdint.h>
 #include "config.h"
 
+#if HAVE_BYTESWAP_H
+#include <byteswap.h>
+#else
 /**
- * swab_u16 - reverse bytes in a uint16_t value.
+ * bswap_16 - reverse bytes in a uint16_t value.
  * @val: value whose bytes to swap.
+ *
+ * Example:
+ *     // Output contains "1024 is 4 as two bytes reversed"
+ *     printf("1024 is %u as two bytes reversed\n", bswap_16(1024));
  */
-static inline uint16_t swab_u16(uint16_t val)
+static inline uint16_t bswap_16(uint16_t val)
 {
        return ((val & (uint16_t)0x00ffU) << 8)
                | ((val & (uint16_t)0xff00U) >> 8);
 }
 
 /**
- * swab_u32 - reverse bytes in a uint32_t value.
+ * bswap_32 - reverse bytes in a uint32_t value.
  * @val: value whose bytes to swap.
+ *
+ * Example:
+ *     // Output contains "1024 is 262144 as four bytes reversed"
+ *     printf("1024 is %u as four bytes reversed\n", bswap_32(1024));
  */
-static inline uint32_t swab_u32(uint32_t val)
+static inline uint32_t bswap_32(uint32_t val)
 {
        return ((val & (uint32_t)0x000000ffUL) << 24)
                | ((val & (uint32_t)0x0000ff00UL) <<  8)
                | ((val & (uint32_t)0x00ff0000UL) >>  8)
                | ((val & (uint32_t)0xff000000UL) >> 24);
 }
+#endif /* !HAVE_BYTESWAP_H */
 
+#if !HAVE_BSWAP_64
 /**
- * swab_u64 - reverse bytes in a uint64_t value.
+ * bswap_64 - reverse bytes in a uint64_t value.
  * @val: value whose bytes to swap.
+ *
+ * Example:
+ *     // Output contains "1024 is 1125899906842624 as eight bytes reversed"
+ *     printf("1024 is %llu as eight bytes reversed\n",
+ *             (unsigned long long)bswap_64(1024));
  */
-static inline uint64_t swab_u64(uint64_t val)
+static inline uint64_t bswap_64(uint64_t val)
 {
        return ((val & (uint64_t)0x00000000000000ffULL) << 56)
                | ((val & (uint64_t)0x000000000000ff00ULL) << 40)
@@ -40,6 +58,7 @@ static inline uint64_t swab_u64(uint64_t val)
                | ((val & (uint64_t)0x00ff000000000000ULL) >> 40)
                | ((val & (uint64_t)0xff00000000000000ULL) >> 56);
 }
+#endif
 
 /* Sanity check the defines.  We don't handle weird endianness. */
 #if !HAVE_LITTLE_ENDIAN && !HAVE_BIG_ENDIAN
@@ -57,7 +76,7 @@ static inline uint64_t cpu_to_le64(uint64_t native)
 #if HAVE_LITTLE_ENDIAN
        return native;
 #else
-       return swab_u64(native);
+       return bswap_64(native);
 #endif
 }
 
@@ -70,7 +89,7 @@ static inline uint32_t cpu_to_le32(uint32_t native)
 #if HAVE_LITTLE_ENDIAN
        return native;
 #else
-       return swab_u32(native);
+       return bswap_32(native);
 #endif
 }
 
@@ -83,7 +102,7 @@ static inline uint16_t cpu_to_le16(uint16_t native)
 #if HAVE_LITTLE_ENDIAN
        return native;
 #else
-       return swab_u16(native);
+       return bswap_16(native);
 #endif
 }
 
@@ -96,7 +115,7 @@ static inline uint64_t le64_to_cpu(uint64_t le_val)
 #if HAVE_LITTLE_ENDIAN
        return le_val;
 #else
-       return swab_u64(le_val);
+       return bswap_64(le_val);
 #endif
 }
 
@@ -109,7 +128,7 @@ static inline uint32_t le32_to_cpu(uint32_t le_val)
 #if HAVE_LITTLE_ENDIAN
        return le_val;
 #else
-       return swab_u32(le_val);
+       return bswap_32(le_val);
 #endif
 }
 
@@ -122,7 +141,7 @@ static inline uint16_t le16_to_cpu(uint16_t le_val)
 #if HAVE_LITTLE_ENDIAN
        return le_val;
 #else
-       return swab_u16(le_val);
+       return bswap_16(le_val);
 #endif
 }
 
@@ -133,7 +152,7 @@ static inline uint16_t le16_to_cpu(uint16_t le_val)
 static inline uint64_t cpu_to_be64(uint64_t native)
 {
 #if HAVE_LITTLE_ENDIAN
-       return swab_u64(native);
+       return bswap_64(native);
 #else
        return native;
 #endif
@@ -146,7 +165,7 @@ static inline uint64_t cpu_to_be64(uint64_t native)
 static inline uint32_t cpu_to_be32(uint32_t native)
 {
 #if HAVE_LITTLE_ENDIAN
-       return swab_u32(native);
+       return bswap_32(native);
 #else
        return native;
 #endif
@@ -159,7 +178,7 @@ static inline uint32_t cpu_to_be32(uint32_t native)
 static inline uint16_t cpu_to_be16(uint16_t native)
 {
 #if HAVE_LITTLE_ENDIAN
-       return swab_u16(native);
+       return bswap_16(native);
 #else
        return native;
 #endif
@@ -172,7 +191,7 @@ static inline uint16_t cpu_to_be16(uint16_t native)
 static inline uint64_t be64_to_cpu(uint64_t be_val)
 {
 #if HAVE_LITTLE_ENDIAN
-       return swab_u64(be_val);
+       return bswap_64(be_val);
 #else
        return be_val;
 #endif
@@ -185,7 +204,7 @@ static inline uint64_t be64_to_cpu(uint64_t be_val)
 static inline uint32_t be32_to_cpu(uint32_t be_val)
 {
 #if HAVE_LITTLE_ENDIAN
-       return swab_u32(be_val);
+       return bswap_32(be_val);
 #else
        return be_val;
 #endif
@@ -198,7 +217,7 @@ static inline uint32_t be32_to_cpu(uint32_t be_val)
 static inline uint16_t be16_to_cpu(uint16_t be_val)
 {
 #if HAVE_LITTLE_ENDIAN
-       return swab_u16(be_val);
+       return bswap_16(be_val);
 #else
        return be_val;
 #endif
index cba96f0b4d68dcd7e14844edf2a1d566a1ed943e..a00fce74e456c42ce8ad9120418ad807967a4020 100644 (file)
@@ -29,7 +29,7 @@ int main(int argc, char *argv[])
        u64.u64_bytes[5] = 0x55;
        u64.u64_bytes[6] = 0x66;
        u64.u64_bytes[7] = 0x77;
-       u64.u64 = swab_u64(u64.u64);
+       u64.u64 = bswap_64(u64.u64);
        ok1(u64.u64_bytes[7] == 0x00);
        ok1(u64.u64_bytes[6] == 0x11);
        ok1(u64.u64_bytes[5] == 0x22);
@@ -43,7 +43,7 @@ int main(int argc, char *argv[])
        u32.u32_bytes[1] = 0x11;
        u32.u32_bytes[2] = 0x22;
        u32.u32_bytes[3] = 0x33;
-       u32.u32 = swab_u32(u32.u32);
+       u32.u32 = bswap_32(u32.u32);
        ok1(u32.u32_bytes[3] == 0x00);
        ok1(u32.u32_bytes[2] == 0x11);
        ok1(u32.u32_bytes[1] == 0x22);
@@ -51,7 +51,7 @@ int main(int argc, char *argv[])
 
        u16.u16_bytes[0] = 0x00;
        u16.u16_bytes[1] = 0x11;
-       u16.u16 = swab_u16(u16.u16);
+       u16.u16 = bswap_16(u16.u16);
        ok1(u16.u16_bytes[1] == 0x00);
        ok1(u16.u16_bytes[0] == 0x11);