[FFmpeg-devel] [PATCH] Remove unessesary assignment of CODEC_ID_BMP

Måns Rullgård mans
Sun Jul 8 13:54:09 CEST 2007


"mark cox" <melbournemark+ffmpeg at gmail.com> writes:

> This patch removes the assignment of CODEC_ID_BMP because, it is already
> assigned.
> Regards,
> mark
>
> Index: libavcodec/bmp.c
> ===================================================================
> --- libavcodec/bmp.c	(revision 9532)
> +++ libavcodec/bmp.c	(working copy)
> @@ -111,7 +111,7 @@
>          rgb[2] = bytestream_get_le32(&buf);
>      }
>  
> -    avctx->codec_id = CODEC_ID_BMP;
> +    assert(avctx->codec_id == CODEC_ID_BMP);
>      avctx->width = width;
>      avctx->height = height > 0? height: -height;

Why the assert?  We don't do that for most of the codecs.

-- 
M?ns Rullg?rd
mans at mansr.com




More information about the ffmpeg-devel mailing list