[ViewVC] Diff of: cvs/libeio/ecb.h

Comparing libeio/ecb.h (file contents):
Revision 1.13 by root, Sat Jun 2 20:13:26 2012 UTC vs.
Revision 1.14 by root, Sun Jun 17 17:07:25 2012 UTC

   ecb_inline void ecb_unreachable (void) ecb_noreturn;
   ecb_inline void ecb_unreachable (void) { }
 #endif
 /* try to tell the compiler that some condition is definitely true */
-#define ecb_assume(cond) do { if (!(cond)) ecb_unreachable (); } while (0)
+#define ecb_assume(cond) if (!(cond)) ecb_unreachable (); else 0
 ecb_inline unsigned char ecb_byteorder_helper (void) ecb_const;
 ecb_inline unsigned char
 ecb_byteorder_helper (void)
 {
-  const uint32_t u = 0x11223344;
+  /* the union code still generates code under pressure in gcc, */
-  return *(unsigned char *)&u;
+  /* but less than using pointers, and always seem to */
+  /* successfully return a constant. */
+  /* the reason why we have this horrible preprocessor mess */
+  /* is to avoid it in all cases, at least on common architectures */
+  /* and yes, gcc defines __BYTE_ORDER__, g++ does not */
+#if __i386 || __i386__ || _M_X86 || __amd64 || __amd64__ || _M_X64
+  return 0x44;
+#elif __BYTE_ORDER__ && __BYTE_ORDER__ == __ORDER_LITTLE_ENDIAN__
+  return 0x44;
+#elif __BYTE_ORDER__ && __BYTE_ORDER__ == __ORDER_BIG_ENDIAN__
+  retrurn 0x11;
+#else
+  union
+  {
+    uint32_t i;
+    uint8_t c;
+  } u = { 0x11223344 };
+  return u.c;
+#endif
 }
 ecb_inline ecb_bool ecb_big_endian    (void) ecb_const;
 ecb_inline ecb_bool ecb_big_endian    (void) { return ecb_byteorder_helper () == 0x11; }
 ecb_inline ecb_bool ecb_little_endian (void) ecb_const;

-–
+Removed lines
-+
+Added lines
-<
+Changed lines
->
+Changed lines