[FFmpeg-devel] [PATCH] fix in ff_celp_lp_synthesis_filterf
Robert Swain
robert.swain
Mon Feb 23 21:25:39 CET 2009
2009/2/23 Vitor Sessak <vitor1001 at gmail.com>:
> Kenan Gillet wrote:
>> i have noticed while working on the AMR SoC that the
>> ff_celp_lp_synthesis_filterf
>> apply backwardly the filter adding instead of subtracting the filtered
>> signal.
>>
>> The patches has been tested for the QCELP decoder, which is so far the
>> only
>> code using this function, and it gives same bit ouput.
>
> I like this patch, it allows also a nice simplification of ra288.c
> (attached). Who is supposed to review the original patch? Michael, Reynaldo
> or both?
I don't know, but for what it's worth, the simplification it enables
to AMR looks good to me.
Regards,
Rob
More information about the ffmpeg-devel
mailing list