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

Mike Scheutzow mjs973 at optonline.net
Wed Jul 11 15:01:13 CEST 2012


On 7/5/2012 11:50 AM, Mike Scheutzow wrote:
> Mike Scheutzow wrote:
>> Michael Niedermayer wrote:
>>> On Wed, Jun 27, 2012 at 11:51:10AM -0400, Mike Scheutzow wrote:
>>>> The attached patch fixes trac 1432.
>>>>
>>>> The bad code was using the frame byte length to calculate a timestamp,
>>>> which is completely broken. Also set AVPacket.duration, since demux is
>>>> already calculating it. The existing avi muxer already properly handles
>>>> the muxing and index creation for MP3 audio frames.  Fixes the sample
>>>> file in trac 1432.
>>>
>>> code breaks make fate
>>> also please test your patch against cbr & vbr style muxed avi
>
> Updated patch attached. Fate passes on my linux box. Tested with CFR and
> VFR avi files.
>
> If no problems, reviewer can commit.
>
>
> Mike Scheutzow

Ping


More information about the ffmpeg-devel mailing list