[FFmpeg-devel] [PATCH] G.722 decoder

[Martin Storsjö]

On Thu, 9 Sep 2010, Michael Niedermayer wrote:

> On Thu, Sep 09, 2010 at 11:04:39AM +0300, Martin Storsj? wrote:
> > On Thu, 9 Sep 2010, Carl Eugen Hoyos wrote:
> > 
> > > Martin Storsj? <martin <at> martin.st> writes:
> > > 
> > > > Implemented it, and it works more or less. I had to add an A flag to the 
> > > > lowres option in lavc/options.c.
> 
> ok and i think you should look at how width/height is handled

Ah, yes, now I found it - ffmpeg.c shifts the width/height (used for the 
output) if lowres is set, before opening the decoder - trivial to do the 
same for audio.

Updated patch set attached, should be just fine now, afaik.

// Martin
-------------- next part --------------
A non-text attachment was scrubbed...
Name: 0001-Initial-G.722-decoder-patch.patch
Type: text/x-diff
Size: 14407 bytes
Desc: 
URL: <http://lists.mplayerhq.hu/pipermail/ffmpeg-devel/attachments/20100909/fe78c4c3/attachment.patch>
-------------- next part --------------
A non-text attachment was scrubbed...
Name: 0002-Allow-the-lowres-option-to-affect-audio-codecs-too.patch
Type: text/x-diff
Size: 2739 bytes
Desc: 
URL: <http://lists.mplayerhq.hu/pipermail/ffmpeg-devel/attachments/20100909/fe78c4c3/attachment-0001.patch>
-------------- next part --------------
A non-text attachment was scrubbed...
Name: 0003-G.722-muxer-demuxer.patch
Type: text/x-diff
Size: 2874 bytes
Desc: 
URL: <http://lists.mplayerhq.hu/pipermail/ffmpeg-devel/attachments/20100909/fe78c4c3/attachment-0002.patch>