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

[Alex Converse]

On Mon, Mar 14, 2011 at 12:54 AM, Anton Khirnov <anton at khirnov.net> wrote:
> 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.
>

In that case it should be updated as cosmetic only changes and should
change the comments too.