[FFmpeg-devel] [PATCH 07/16] vmdaudio: use macros and a local variable for block type.

Kostya kostya.shishkov
Tue Feb 22 22:50:07 CET 2011


On Tue, Feb 22, 2011 at 02:05:26PM -0500, Justin Ruggles wrote:
> ---
>  libavcodec/vmdav.c |   13 ++++++++++---
>  1 files changed, 10 insertions(+), 3 deletions(-)
> 

> diff --git a/libavcodec/vmdav.c b/libavcodec/vmdav.c
> index ed3c5f8..0a77d37 100644
> --- a/libavcodec/vmdav.c
> +++ b/libavcodec/vmdav.c
> @@ -414,6 +414,10 @@ static av_cold int vmdvideo_decode_end(AVCodecContext *avctx)
>   * Audio Decoder
>   */
>  
> +#define BLOCK_TYPE_AUDIO    1
> +#define BLOCK_TYPE_INITIAL  2
> +#define BLOCK_TYPE_SILENCE  3
> +
>  typedef struct VmdAudioContext {
>      AVCodecContext *avctx;
>      int channels;
> @@ -501,6 +505,7 @@ static int vmdaudio_decode_frame(AVCodecContext *avctx,
>      const uint8_t *buf = avpkt->data;
>      int buf_size = avpkt->size;
>      VmdAudioContext *s = avctx->priv_data;
> +    int block_type;
>      unsigned char *output_samples = (unsigned char *)data;
>  
>      /* point to the start of the encoded data */
> @@ -509,10 +514,12 @@ static int vmdaudio_decode_frame(AVCodecContext *avctx,
>      if (buf_size < 16)
>          return buf_size;
>  
> -    if (buf[6] == 1) {
> +    block_type = buf[6];
> +
> +    if (block_type == BLOCK_TYPE_AUDIO) {
>          /* the chunk contains audio */
>          *data_size = vmdaudio_loadsound(s, output_samples, p, 0, buf_size - 16);
> -    } else if (buf[6] == 2) {
> +    } else if (block_type == BLOCK_TYPE_INITIAL) {
>          /* initial chunk, may contain audio and silence */
>          uint32_t flags = AV_RB32(p);
>          int raw_block_size = s->block_align *
> @@ -525,7 +532,7 @@ static int vmdaudio_decode_frame(AVCodecContext *avctx,
>          output_samples += raw_block_size * silent_chunks;
>          *data_size = raw_block_size * silent_chunks;
>          *data_size += vmdaudio_loadsound(s, output_samples, p + 4, 0, buf_size - 20);
> -    } else if (buf[6] == 3) {
> +    } else if (block_type == BLOCK_TYPE_SILENCE) {
>          /* silent chunk */
>          *data_size = vmdaudio_loadsound(s, output_samples, p, 1, s->block_align);
>      }

fine with me



More information about the ffmpeg-devel mailing list