[FFmpeg-devel] Review request - ra288.{c,h} ra144.{c,h}
Vitor Sessak
vitor1001
Mon Sep 8 20:56:22 CEST 2008
Hi
Michael Niedermayer wrote:
> On Fri, Sep 05, 2008 at 12:23:58AM +0200, Vitor Sessak wrote:
>> Vitor Sessak wrote:
>>> Hi,
>>>
>>> Those four files never passed a review. I've just finished cleaning them
>>> up, so if anyone wants to review them (Michael already said he will),
>>> now is time.
>> I think now they can go through another review cycle.
>
> ok ra144 reveiw below, i also think that this is my last ra144
> review pass, i dont think id find much more ...
Changed everything as suggested with the exception of the following:
>> static int interp(RA144Context *ractx, int16_t *out, int block_num,
>> int copyold, int energy)
>
> it is slightly ugly that the 2 input lpc coeff vectors are not passed as
> aruments but taken from the context though iam not sure if it would be a
> good idea to add 2 more arguments ...
-Vitor
More information about the ffmpeg-devel
mailing list