[FFmpeg-soc] [soc]: r5433 - amr/amrnbdec.c

Vitor Sessak vitor1001 at gmail.com
Tue Nov 10 20:31:00 CET 2009


Ronald S. Bultje wrote:
> Hi,
> 
> On Tue, Nov 10, 2009 at 9:45 AM, Michael Niedermayer <michaelni at gmx.at> wrote:
>> On Tue, Nov 10, 2009 at 05:49:48AM +0100, Vitor Sessak wrote:
>>> vitor wrote:
>>>> Author: vitor
>>>> Date: Tue Nov 10 05:49:08 2009
>>>> New Revision: 5433
>>>> Log:
>>>> Use the already committed function ff_set_min_dist_lsf().
>>>> Note that this changes slightly the output,
>> which is closer to the correct (that is encoder input)?
> 
> Most likely the double one. I don't mind making LSFs/LPCs doubles,
> there's very few of them (10-16) per set and ff_acelp_lspd2lpc()
> already uses doubles anyway (probably for the same reason). Vitor?
> Reynaldo?

The attached patch makes that the output before and after r5433 is 
unchanged.

-Vitor
-------------- next part --------------
A non-text attachment was scrubbed...
Name: lsp_double.diff
Type: text/x-patch
Size: 941 bytes
Desc: not available
URL: <http://lists.mplayerhq.hu/pipermail/ffmpeg-soc/attachments/20091110/1185db50/attachment.bin>


More information about the FFmpeg-soc mailing list