[FFmpeg-devel] [PATCH 2/2] mpeg12dec: BW10 support

[Reimar Döffinger]

On Fri, Jan 13, 2012 at 05:57:30AM +0100, Michael Niedermayer wrote:
> @@ -2268,7 +2273,10 @@ static int mpeg_decode_frame(AVCodecContext *avctx,
>              return buf_size;
>      }
>  
> -    if (s->mpeg_enc_ctx_allocated == 0 && avctx->codec_tag == AV_RL32("VCR2"))
> +    s2->codec_tag = avpriv_toupper4(avctx->codec_tag);

IMO this needs some explaining comment, particularly since riff.c
contains only the uppercase variant, so how would a lowercase one
end up here?