[FFmpeg-devel] [PATCH] Fix timestamp calculation when demuxing MP3 frames in a .avi container.
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.
-------------- next part --------------
An embedded and charset-unspecified text was scrubbed...
More information about the ffmpeg-devel