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

Michael Niedermayer michaelni at gmx.at
Wed Jun 27 23:55:38 CEST 2012

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


Michael     GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB

The greatest way to live with honor in this world is to be what we pretend
to be. -- Socrates
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 198 bytes
Desc: Digital signature
URL: <http://ffmpeg.org/pipermail/ffmpeg-devel/attachments/20120627/5f9bb6dc/attachment.asc>

More information about the ffmpeg-devel mailing list