[FFmpeg-devel] [PATCH] G.729 and G.729D decoders

[Vladimir Voroshilov]

Hi, Michael

On Sun, Apr 20, 2008 at 6:22 PM, Michael Niedermayer <michaelni at gmx.at> wrote:
> On Sun, Apr 20, 2008 at 12:31:44PM +0700, Vladimir Voroshilov wrote:
>  > Hi, Michael.

[...]

>  > I hope, lookup tables and build api can be separated.
>  > Otherwise i can't create uncompressed <100k patch.
>
>  Well this patch started out at 77k and then grew to 98k during review
>  now its even with compressed tables over 110k
>  77k is already very large and hard to review
>  Iam simply drawing a line and rejecting it until it returns to <100k
>  It was possible in previous iterations so it should still be. If not
>  which of my change requests caused such size increase?
>  You have added support for g729d thats nice but it has to be
>  split into a seperate patch. I dont know if it also makes sense to
>  split it into several files.

1. G.729A -> G.729 added 12k of source code (due to huge long-term
filter, large tilt-compensation filter and additional lookup tables
for them)
2. G.729D support added additionally about 8k

Here is 93k size G.729-only patch.
I don't know what else can be removed from it.

-- 
Regards,
Vladimir Voroshilov     mailto:voroshil at gmail.com
JID: voroshil at gmail.com, voroshil at jabber.ru
ICQ: 95587719
-------------- next part --------------
An embedded and charset-unspecified text was scrubbed...
Name: g729_17.diff
URL: <http://lists.mplayerhq.hu/pipermail/ffmpeg-devel/attachments/20080420/11246f7f/attachment.txt>