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

[Michael Niedermayer]

On Tue, Nov 10, 2009 at 08:31:00PM +0100, Vitor Sessak wrote:
> 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

>  lsp.c |    2 +-
>  lsp.h |    2 +-
>  2 files changed, 2 insertions(+), 2 deletions(-)
> 921c6197c176765e10ad4be3ffe5cb74447ebd41  lsp_double.diff

ok

[...]

-- 
Michael     GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB

Good people do not need laws to tell them to act responsibly, while bad
people will find a way around the laws. -- Plato
-------------- 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-soc/attachments/20091111/b8c0f4c7/attachment.pgp>