[Ffmpeg-devel] amr file support fix

Michael Niedermayer michaelni
Wed Oct 11 01:48:42 CEST 2006


Hi

On Wed, Oct 11, 2006 at 01:21:07AM +0200, Michael Niedermayer wrote:
> Hi
> 
> On Mon, Oct 09, 2006 at 09:24:11PM +0200, Simon Morlat wrote:
> > Hello,
> > 
> > While using libavformat to read AMR and AMR-WB files in my application, I 
> > noticed that the 'pts' (presentation timestamp) field of the returned 
> > AVPacket was not set (actually set to zero all the time).
> > After some digging in the code I found that the AVPacket::duration field was 
> > not set, and the time_base was left to a probable default value (1/90000) 
> > which is usual for video but very unusual for audio.
> > 
> > Here is a very little patch that fixes those issues. It was generated 
> > with 'svn diff' today.
> > It sets the time_base to 1/8000 for amr and 1/16000 for amr-wb, and makes the 
> > pts incrementing respectively by 160 and 320, just by setting 
> > AVPacket::duration.
> 
> looks ok

libavformat/amr.c is a mess ...
ive cleaned it up a tiny bit, and applied your patch

[...]
-- 
Michael     GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB

In the past you could go to a library and read, borrow or copy any book
Today you'd get arrested for mere telling someone where the library is




More information about the ffmpeg-devel mailing list