[FFmpeg-devel] [PATCH] Some ra144.c simplifications

[Vitor Sessak]

Michael Niedermayer wrote:
> On Tue, May 27, 2008 at 09:16:09PM +0200, Vitor Sessak wrote:
>> Michael Niedermayer wrote:
>>> On Sun, May 25, 2008 at 07:11:52PM +0200, Vitor Sessak wrote:
>>>> Michael Niedermayer wrote:
>>>>> On Sun, May 25, 2008 at 06:05:15PM +0200, Vitor Sessak wrote:
>>>>> [...]
>>>>>>>> ok
>>>>>>> One more...
>>>>>> ... and some more cleanup:
>>>>>>
>>>>>> ra144_vector_add_wav.diff: Make add_wav() receive a vector instead of 
>>>>>> three integers
>>>>>>
>>>>>> ra144_params_dec2.diff: Do not calculate anything based in l, it is 
>>>>>> unrolled in the loop anyway
>>>>> ok
>>>> Now s/(unsigned) short/(u)int16_t.
>>> ok
>> Next one. dec2() interpolates the block coefficients from the previous one 
>> and fall back to a block-dependent schema if the interpolation results in 
>> an unstable filter...
> [...]
>> +    // Interpolate block coefficients from the this frame forth block and
>> +    // last frame forth block
>>      for (x=0; x<30; x++)
>> -        decsp[x] = (a * inp[x] + b * inp2[x]) >> 2;
>> +        decsp[x] = (a * ractx->lpc_coef[x] + b * ractx->lpc_coef_old[x])>> 2;
> 
> ff_acelp_weighted_vector_sum()

Ok, but to do that I need to use int16_t. So I propose to apply my 
original patch and then the attached one.

-Vitor
-------------- next part --------------
A non-text attachment was scrubbed...
Name: ra144_use_ff_vec_sum.diff
Type: text/x-patch
Size: 2574 bytes
Desc: not available
URL: <http://lists.mplayerhq.hu/pipermail/ffmpeg-devel/attachments/20080528/b71a6b0e/attachment.bin>