patch [#32556] Stupid endian conversion in avi format
authorCampbell Barton <ideasman42@gmail.com>
Sun, 7 Oct 2012 06:06:28 +0000 (06:06 +0000)
committerCampbell Barton <ideasman42@gmail.com>
Sun, 7 Oct 2012 06:06:28 +0000 (06:06 +0000)
from Andreas Schwab (schwab)

modified to use code from BLI_endian_switch.

source/blender/avi/intern/endian.c
source/blender/blenlib/BLI_endian_switch_inline.h

index c9b95d258105de24ef791185ba914dccbe7d1e45..8e5ed80f5a0d8297821517cb4bf90add8579b1a1 100644 (file)
 #endif
 
 #ifdef __BIG_ENDIAN__
+
+/* copied from BLI_endian_switch_inline.h */
 static void invert(int *num)
 {
-       int new = 0, i, j;
-
-       for (j = 0; j < 4; j++) {
-               for (i = 0; i < 8; i++) {
-                       new |= ((*num >> (j * 8 + i)) & 1) << ((3 - j) * 8 + i);
-               }
-       }
-       
-       *num = new;
+       int tval = *val;
+       *val = ((tval >> 24))             |
+              ((tval << 8) & 0x00ff0000) |
+              ((tval >> 8) & 0x0000ff00) |
+              ((tval << 24));
 }
 
-static void sinvert(short int *num)
+static void sinvert(short int *val)
 {
-       short int new = 0;
-       int i, j;
-
-       for (j = 0; j < 2; j++) {
-               for (i = 0; i < 8; i++) {
-                       new |= ((*num >> (j * 8 + i)) & 1) << ((1 - j) * 8 + i);
-               }
-       }
-
-       *num = new;
+       short tval = *val;
+       *val = (tval >> 8) |
+              (tval << 8);
 }
 
 static void Ichunk(AviChunk *chunk)
index 6694276a995c1e819a5fc3791fcbe7661ed17fd0..948d462fde4275c9e05335aa438bc97f91d69a9f 100644 (file)
@@ -40,7 +40,7 @@ BLI_INLINE void BLI_endian_switch_int16(short *val)
 {
        short tval = *val;
        *val = (tval >> 8) |
-           (tval << 8);
+              (tval << 8);
 
 }
 BLI_INLINE void BLI_endian_switch_uint16(unsigned short *val)