[FFmpeg-devel] [PATCH] Expose QCELP's floating-point LSP-to-LPC function

Robert Swain robert.swain
Mon Aug 3 10:34:37 CEST 2009


Hey,

2009/8/1 Michael Niedermayer <michaelni at gmx.at>:
> On Fri, Jul 31, 2009 at 09:18:42PM +0100, Colin McQuillan wrote:
>> I'd like to use the code in qcelp_lpc.c function for AMR. First, the
>> naming should be made consistent.
>>
>> rename_qcelp_lsp.patch description:
>>
>> Make the LSP naming more consistent
>> Use the convention from lsp.c: an LSF is a frequency, an LSP is the
>> cosine of an LSF, and LSP functions should have an ff_acelp prefix.
>> Use a "d" suffix to specify doubles.
>
> iam not maintainer of qcelp but as iam not sure if reynaldo is busy,
> ive not seen a mail from him recently IIRC, and soc time is limited
> i approve this one, if reynaldo disagrees it can be reverted

Applied.

>> move_qcelp_lsp.patch description:
>>
>> Expose QCELP's floating-point LSP-to-LPC function
>> qcelp_lsp exported a single function, ff_acelp_lspd2lpc, which was not
>> specific to qcelp. It can be kept with its fixed-point version
>> ff_acelp_lsp2lpc in lpc.c.
>
> similarly ok

Applied.

I will revert if Reynaldo has any complaints but I didn't want to hold up Colin.

Regards,
Rob



More information about the ffmpeg-devel mailing list