[FFmpeg-devel] [PATCH 2/2] Move MDCT function pointers to MDCTContext

Måns Rullgård mans
Thu Sep 17 01:51:06 CEST 2009


Michael Niedermayer <michaelni at gmx.at> writes:

> On Thu, Sep 17, 2009 at 12:33:11AM +0100, M?ns Rullg?rd wrote:
>> Mans Rullgard <mans at mansr.com> writes:
>> 
>> > ---
>> >  libavcodec/arm/fft_init_arm.c |    6 +++---
>> >  libavcodec/dsputil.h          |   12 ++++++------
>> >  libavcodec/mdct.c             |    6 +++---
>> >  libavcodec/x86/fft.c          |   12 ++++++------
>> >  4 files changed, 18 insertions(+), 18 deletions(-)
>> 
>> Ping.  What about this, assuming the first patch is OK?
>
> Its the first patch iam uncertain about
> it does " 41 insertions(+), 14 deletions(-)"
> which is it makes the code bigger and iam not sure what that is good
> for ..

I thought it made sense in preparation for the patch I just sent,
which will then be followed by another patch making the NEON MDCT a
few % faster.  Feel free to suggest a better way.

-- 
M?ns Rullg?rd
mans at mansr.com



More information about the ffmpeg-devel mailing list