[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