mlp: split simple inline function that xors 4 bytes into one.
authorRamiro Polla <ramiro.polla@gmail.com>
Thu, 14 Aug 2008 03:58:05 +0000 (03:58 +0000)
committerRamiro Polla <ramiro.polla@gmail.com>
Thu, 14 Aug 2008 03:58:05 +0000 (03:58 +0000)
Originally committed as revision 14747 to svn://svn.ffmpeg.org/ffmpeg/trunk

libavcodec/mlp.c
libavcodec/mlp.h
libavcodec/mlpdec.c

index f2a0276..2ac2e65 100644 (file)
@@ -110,8 +110,7 @@ uint8_t ff_mlp_calculate_parity(const uint8_t *buf, unsigned int buf_size)
     for (; buf < buf_end - 3; buf += 4)
         scratch ^= *((const uint32_t*)buf);
 
     for (; buf < buf_end - 3; buf += 4)
         scratch ^= *((const uint32_t*)buf);
 
-    scratch ^= scratch >> 16;
-    scratch ^= scratch >> 8;
+    scratch = xor_32_to_8(scratch);
 
     for (; buf < buf_end; buf++)
         scratch ^= *buf;
 
     for (; buf < buf_end; buf++)
         scratch ^= *buf;
index dc68567..a434cd1 100644 (file)
@@ -107,4 +107,12 @@ int ff_mlp_init_crc2D(AVCodecParserContext *s);
 
 void ff_mlp_init_crc();
 
 
 void ff_mlp_init_crc();
 
+/** XOR four bytes into one. */
+static inline uint8_t xor_32_to_8(uint32_t value)
+{
+    value ^= value >> 16;
+    value ^= value >>  8;
+    return value;
+}
+
 #endif /* FFMPEG_MLP_H */
 #endif /* FFMPEG_MLP_H */
index d44c9b0..421abde 100644 (file)
@@ -364,10 +364,7 @@ static int read_restart_header(MLPDecodeContext *m, GetBitContext *gbp,
     lossless_check = get_bits(gbp, 8);
     if (substr == m->max_decoded_substream
         && s->lossless_check_data != 0xffffffff) {
     lossless_check = get_bits(gbp, 8);
     if (substr == m->max_decoded_substream
         && s->lossless_check_data != 0xffffffff) {
-        tmp = s->lossless_check_data;
-        tmp ^= tmp >> 16;
-        tmp ^= tmp >> 8;
-        tmp &= 0xff;
+        tmp = xor_32_to_8(s->lossless_check_data);
         if (tmp != lossless_check)
             av_log(m->avctx, AV_LOG_WARNING,
                    "Lossless check failed - expected %02x, calculated %02x.\n",
         if (tmp != lossless_check)
             av_log(m->avctx, AV_LOG_WARNING,
                    "Lossless check failed - expected %02x, calculated %02x.\n",