[polypaudio-commits] r456 - /trunk/polyp/endianmacros.h

[svnmailer-noreply at 0pointer.de]

Author: ossman
Date: Mon Jan 16 14:35:25 2006
New Revision: 456

URL: http://0pointer.de/cgi-bin/viewcvs.cgi?rev=456&root=polypaudio&view=rev
Log:
We need a logical, not an arithmetic shift here. So use unsigned types when
doing the shifting.

Modified:
    trunk/polyp/endianmacros.h

Modified: trunk/polyp/endianmacros.h
URL: http://0pointer.de/cgi-bin/viewcvs.cgi/trunk/polyp/endianmacros.h?rev=456&root=polypaudio&r1=455&r2=456&view=diff
==============================================================================
--- trunk/polyp/endianmacros.h (original)
+++ trunk/polyp/endianmacros.h Mon Jan 16 14:35:25 2006
@@ -28,9 +28,9 @@
 #include <config.h>
 #endif
 
-#define INT16_SWAP(x) ((int16_t)(((int16_t) x >> 8) | ((int16_t) x << 8)))
+#define INT16_SWAP(x) ((int16_t)(((uint16_t) x >> 8) | ((uint16_t) x << 8)))
 #define UINT16_SWAP(x) ((uint16_t)(((uint16_t) x >> 8) | ((uint16_t) x << 8)))
-#define INT32_SWAP(x) ((int32_t)(((int32_t) x >> 24) | ((int32_t) x << 24) | (((int32_t) x & 0xFF00) << 16) | (((int32_t) x) >> 16) & 0xFF00))
+#define INT32_SWAP(x) ((int32_t)(((uint32_t) x >> 24) | ((uint32_t) x << 24) | (((uint32_t) x & 0xFF00) << 16) | (((uint32_t) x) >> 16) & 0xFF00))
 #define UINT32_SWAP(x) ((uint32_t)(((uint32_t) x >> 24) | ((uint32_t) x << 24) | (((uint32_t) x & 0xFF00) << 16) | ((((uint32_t) x) >> 16) & 0xFF00)))
 
 #ifdef WORDS_BIGENDIAN