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

[Michael Niedermayer]

On Wed, May 28, 2008 at 06:56:45PM +0200, Vitor Sessak wrote:
> 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.

hmm, ok

[...]
-- 
Michael     GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB

The educated differ from the uneducated as much as the living from the
dead. -- Aristotle 
-------------- 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-devel/attachments/20080528/632b1137/attachment.pgp>