[FFmpeg-devel] [PATCH] Fix timestamp calculation when demuxing MP3 frames in a .avi container.

[Mike Scheutzow]

On 7/11/2012 9:34 PM, Michael Niedermayer wrote:
>> +                        if (fps > 41) {
>> +                            // highest valid frame per sec is 48000/1152
>> +                            av_log(s, AV_LOG_WARNING, "st%d: MP3 frame rate is invalid (%d FPS)\n", st->id, fps);
>> +                        } else {
> did this if() make any difference in any of your tests ?
> iam asking because it looks kind of ugly not saying iam against it if
> it is needed

Results from web search indicate that some (broken) software tools 
create avi files with the audio sample rate in stream's header 
information, rather than the audio frame rate that we need. This is a 
check for that problem. It will also help to notify the user why the avi 
file is not playing back properly.

I did not bother to actually find such a sample file. The check serves a 
worthwhile purpose.

Mike Scheutzow