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

Vitor Sessak vitor1001 at gmail.com
Wed Nov 11 12:15:40 CET 2009


Michael Niedermayer wrote:
> 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

Applied.

-Vitor


More information about the FFmpeg-soc mailing list