[FFmpeg-devel] [PATCH] Common ACELP routines (3/3) - LPC decoding routines

Michael Niedermayer michaelni
Wed Apr 23 17:33:45 CEST 2008


On Wed, Apr 23, 2008 at 03:45:11PM +0100, Robert Swain wrote:
> On 23 Apr 2008, at 14:42, Michael Niedermayer wrote:
> > On Wed, Apr 23, 2008 at 02:02:50PM +0100, Robert Swain wrote:
> >> On 23 Apr 2008, at 01:38, Michael Niedermayer wrote:
> >>> On Tue, Apr 22, 2008 at 01:21:24AM +0700, Vladimir Voroshilov wrote:
> >>>> This patch contains several routines related to LP filter
> >>>> coefficients
> >>>> decoding (not all, but those which
> >>>> looks like AMR's).
> >>>
> >>> [...]
> >>>
> >>>> diff --git a/libavcodec/acelp_lpc.c b/libavcodec/acelp_lpc.c
> >>>> new file mode 100644
> >>>> index 0000000..65b870d
> >>>> --- /dev/null
> >>>> +++ b/libavcodec/acelp_lpc.c
> >>>
> >>> lsp.c
> >>
> >> I'd prefer *lpc.* as the code relates to linear predictive coding
> >> functions and not just line spectral pairs. But as you please - what
> >> is in a name? :)
> >
> > they could also be split in lsp.c lpc.c :)
> 
> I don't think it's worth splitting that much.
> 

> >> Isn't there some LPC code in FLAC? Is any of that useful?
> 
> No comment?

flac doesnt use LSP, so its just a single loop or so that could be
shared i think ...
but RTFS instead of listening to me :)


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

Let us carefully observe those good qualities wherein our enemies excel us
and endeavor to excel them, by avoiding what is faulty, and imitating what
is excellent in them. -- Plutarch
-------------- 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/20080423/15267adb/attachment.pgp>



More information about the ffmpeg-devel mailing list