[FFmpeg-devel] [PATCH] AAC: use table for cbrtf(n)*n

Alex Converse alex.converse
Mon Jan 11 20:13:35 CET 2010


2010/1/11 M?ns Rullg?rd <mans at mansr.com>:
> Alex Converse <alex.converse at gmail.com> writes:
>
>> On Mon, Jan 11, 2010 at 8:31 AM, Mans Rullgard <mans at mansr.com> wrote:
>>> Up to 10% faster on Cortex-A8.
>>> ---
>>> ?libavcodec/aac.c | ? 13 +++++++++----
>>> ?1 files changed, 9 insertions(+), 4 deletions(-)
>>>
>>
>> Otherwise LGTM
>
> So OK to apply this one?
>

The only other potential issue I can think of is that MP3 uses this
same 4/3 quantizer and has it's own set of hacks to deal with it. But
I am OK with committing this as is and leaving reconciling the two as
a fight for another day.



More information about the ffmpeg-devel mailing list