8svx: use a more direct condition.
authorNicolas George <nicolas.george@normalesup.org>
Sun, 29 Jul 2012 10:33:35 +0000 (12:33 +0200)
committerNicolas George <nicolas.george@normalesup.org>
Tue, 31 Jul 2012 15:23:31 +0000 (17:23 +0200)
esc->table was inited based on codec->id: re-testing codec->id
is code duplication and can lead to inconsistencies.

libavcodec/8svx.c

index 6aa52d1..3bece41 100644 (file)
@@ -112,13 +112,13 @@ static int eightsvx_decode_frame(AVCodecContext *avctx, void *data,
     if (!esc->samples && avpkt) {
         uint8_t *deinterleaved_samples, *p = NULL;
 
-        esc->samples_size = avctx->codec->id == CODEC_ID_8SVX_RAW || avctx->codec->id ==CODEC_ID_PCM_S8_PLANAR?
+        esc->samples_size = !esc->table ?
             avpkt->size : avctx->channels + (avpkt->size-avctx->channels) * 2;
         if (!(esc->samples = av_malloc(esc->samples_size)))
             return AVERROR(ENOMEM);
 
         /* decompress */
-        if (avctx->codec->id == CODEC_ID_8SVX_FIB || avctx->codec->id == CODEC_ID_8SVX_EXP) {
+        if (esc->table) {
             const uint8_t *buf = avpkt->data;
             int buf_size = avpkt->size;
             int n = esc->samples_size;
@@ -166,7 +166,7 @@ static int eightsvx_decode_frame(AVCodecContext *avctx, void *data,
         *dst++ = *src++ + 128;
     esc->samples_idx += out_data_size;
 
-    return avctx->codec->id == CODEC_ID_8SVX_FIB || avctx->codec->id == CODEC_ID_8SVX_EXP ?
+    return esc->table ?
         (avctx->frame_number == 0)*2 + out_data_size / 2 :
         out_data_size;
 }