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

Michael Niedermayer michaelni
Mon Apr 21 18:57:58 CEST 2008


On Mon, Apr 21, 2008 at 09:57:37PM +0700, Vladimir Voroshilov wrote:
> On Mon, Apr 21, 2008 at 12:24 AM, Michael Niedermayer <michaelni at gmx.at> wrote:
> >
> > On Mon, Apr 21, 2008 at 12:11:16AM +0700, Vladimir Voroshilov wrote:
> 
> [...]
> 
> >  >
> >  > Yes, LP decoding in soc/amr looks very similar to mine.
> >  > How those can be joined?
> >  > What about creating new file (celp.c for example) ?
> >
> >  This was exactly my idea (celp.c, lsp.c, ... whatever names makes sense, and
> >  more files with less content are better than few big files)
> 
> currently selected acelp_lpc.* and acelp_math.*
> 
> >  > Quick look shows me that lsf2lsp, reorder_lsp can be merged.
> >  >
> 
> More precise look give a trouble: amr code uses "int" type everywhere, while my
> code uses "int" only in necessary places. So common routines (mine and
> amr's) differs
> only in used types.
> I'm not sure that i can correctly replace int with int16_t in amr
> code. (superdump?)
> 
> >  > decode_*_pulses_* routines can be merged too by using lookup tables in amr
> >  > instead of hardcoded shifts  and multiplications (see mine
> >  > decode_fc_vector, unified
> >  > routine in cost of several additional lookup tables).
> >  >
> >  > What patches should i prepare?
> >  > One for celp.c and one for amr soc project in the same mail (in
> >  > separate thread, of course)?
> >
> >  ok
> >
> 
> I'm going to post incorrect patch (with ignorance of "int"<>"int16_t")
> for amr (just for starting discussion on). ok?

ok

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

I count him braver who overcomes his desires than him who conquers his
enemies for the hardest victory is over self. -- Aristotle
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 189 bytes
Desc: Digital signature
URL: <http://lists.mplayerhq.hu/pipermail/ffmpeg-devel/attachments/20080421/25f52720/attachment.pgp>



More information about the ffmpeg-devel mailing list