[FFmpeg-devel] [PATCH 4/8] avfft: add AV prefix to FFTContext.

[Anton Khirnov]

On Sun, Mar 13, 2011 at 01:20:25PM -0700, Alex Converse wrote:
> On Sun, Mar 13, 2011 at 6:48 AM, Anton Khirnov <anton at khirnov.net> wrote:
> > ---
> > ?libavcodec/aac.h ? ? ? ? ? ? ?| ? ?6 ++--
> > ?libavcodec/aacenc.h ? ? ? ? ? | ? ?4 +-
> > ?libavcodec/aacsbr.c ? ? ? ? ? | ? ?4 +-
> > ?libavcodec/ac3dec.h ? ? ? ? ? | ? ?4 +-
> > ?libavcodec/ac3enc_float.h ? ? | ? ?2 +-
> > ?libavcodec/arm/fft_init_arm.c | ? 14 ++++++------
> > ?libavcodec/atrac1.c ? ? ? ? ? | ? ?4 +-
> > ?libavcodec/atrac3.c ? ? ? ? ? | ? ?2 +-
> > ?libavcodec/avfft.c ? ? ? ? ? ?| ? 22 ++++++++++----------
> > ?libavcodec/avfft.h ? ? ? ? ? ?| ? 21 ++++++++++---------
> > ?libavcodec/cook.c ? ? ? ? ? ? | ? ?2 +-
> > ?libavcodec/dca.c ? ? ? ? ? ? ?| ? ?2 +-
> > ?libavcodec/fft-test.c ? ? ? ? | ? ?4 +-
> > ?libavcodec/fft.c ? ? ? ? ? ? ?| ? 12 +++++-----
> > ?libavcodec/fft.h ? ? ? ? ? ? ?| ? 44 ++++++++++++++++++++--------------------
> > ?libavcodec/imc.c ? ? ? ? ? ? ?| ? ?2 +-
> > ?libavcodec/mdct.c ? ? ? ? ? ? | ? 10 ++++----
> > ?libavcodec/nellymoserdec.c ? ?| ? ?2 +-
> > ?libavcodec/nellymoserenc.c ? ?| ? ?2 +-
> > ?libavcodec/ppc/fft_altivec.c ?| ? 10 ++++----
> > ?libavcodec/sbr.h ? ? ? ? ? ? ?| ? ?4 +-
> > ?libavcodec/synth_filter.c ? ? | ? ?2 +-
> > ?libavcodec/synth_filter.h ? ? | ? ?2 +-
> > ?libavcodec/twinvq.c ? ? ? ? ? | ? ?2 +-
> > ?libavcodec/vorbis_dec.c ? ? ? | ? ?2 +-
> > ?libavcodec/vorbis_enc.c ? ? ? | ? ?2 +-
> > ?libavcodec/wma.h ? ? ? ? ? ? ?| ? ?2 +-
> > ?libavcodec/wmaprodec.c ? ? ? ?| ? ?2 +-
> > ?libavcodec/x86/fft.c ? ? ? ? ?| ? ?2 +-
> > ?libavcodec/x86/fft.h ? ? ? ? ?| ? 20 +++++++++---------
> > ?libavcodec/x86/fft_3dn2.c ? ? | ? ?6 ++--
> > ?libavcodec/x86/fft_mmx.asm ? ?| ? 16 +++++++-------
> > ?libavcodec/x86/fft_sse.c ? ? ?| ? ?6 ++--
> > ?33 files changed, 121 insertions(+), 120 deletions(-)
> >
> 
> [...]
> 
> > diff --git a/libavcodec/x86/fft_mmx.asm b/libavcodec/x86/fft_mmx.asm
> > index 2ccadf0..5b6daae 100644
> > --- a/libavcodec/x86/fft_mmx.asm
> > +++ b/libavcodec/x86/fft_mmx.asm
> > @@ -35,7 +35,7 @@
> > ?%define pointer resd
> > ?%endif
> >
> > -struc FFTContext
> > +struc AVFFTContext
> > ? ? .nbits: ? ?resd 1
> > ? ? .reverse: ?resd 1
> > ? ? .revtab: ? pointer 1
> > @@ -566,7 +566,7 @@ INIT_XMM
> > ? ? jl ? ? ? .post
> > ?%endmacro
> >
> 
> Is the name of the yasm struct really exposed publically?

I admit that I have no clue how this asm magic works, but it seems to me
that it's identical to the C version of FFTContext, so it'd make sense
for it to have the same name.

-- 
Anton Khirnov
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 198 bytes
Desc: Digital signature
URL: <http://lists.mplayerhq.hu/pipermail/ffmpeg-devel/attachments/20110314/e5bb1d9c/attachment.pgp>