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

Mike Scheutzow mike.scheutzow at alcatel-lucent.com
Thu Jul 5 17:50:54 CEST 2012


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
-------------- next part --------------
An embedded and charset-unspecified text was scrubbed...
Name: avi-mp3-v2.patch
URL: <http://ffmpeg.org/pipermail/ffmpeg-devel/attachments/20120705/bcd53caf/attachment.ksh>


More information about the ffmpeg-devel mailing list