[FFmpeg-cvslog] r11853 - trunk/libavformat/nutdec.c

[Rich Felker]

On Mon, Feb 04, 2008 at 11:30:39AM +0100, ods15 wrote:
> Author: ods15
> Date: Mon Feb  4 11:30:38 2008
> New Revision: 11853
> 
> Log:
> Remove old workaround in nutdec.c for libnut bug
> 
> 
> Modified:
>    trunk/libavformat/nutdec.c
> 
> Modified: trunk/libavformat/nutdec.c
> ==============================================================================
> --- trunk/libavformat/nutdec.c	(original)
> +++ trunk/libavformat/nutdec.c	Mon Feb  4 11:30:38 2008
> @@ -336,11 +336,7 @@ static int decode_stream_header(NUTConte
>          ff_get_v(bc); /* csp type */
>      }else if (st->codec->codec_type == CODEC_TYPE_AUDIO){
>          GET_V(st->codec->sample_rate , tmp > 0)
> -        tmp= ff_get_v(bc); // samplerate_den
> -        if(tmp > st->codec->sample_rate){
> -            av_log(s, AV_LOG_ERROR, "Bleh, libnut muxed this ;)\n");
> -            st->codec->sample_rate= tmp;
> -        }
> +        ff_get_v(bc); // samplerate_den

Instead of ignoring denominator, the demuxer should either divide by
it and round the samplerate to and integer, printing a warning at the
same time, or treat it as an error condition (unsupported file).
Likewise it would be nice to check that gcd==1 and many other things.

Rich