[FFmpeg-devel] [PATCH 2/3] Create a public API for FFT family of functions

Michael Niedermayer michaelni
Sat Mar 6 15:26:39 CET 2010


On Sat, Mar 06, 2010 at 07:33:16AM -0500, Alex Converse wrote:
> On Fri, Mar 5, 2010 at 10:38 PM, Mans Rullgard <mans at mansr.com> wrote:
> > ---
> > ?libavcodec/Makefile | ? ?4 +-
> > ?libavcodec/avfft.c ?| ?142 +++++++++++++++++++++++++++++++++++++++++++++++++++
> > ?libavcodec/avfft.h ?| ? 89 ++++++++++++++++++++++++++++++++
> > ?libavcodec/fft.h ? ?| ? 28 +++--------
> > ?4 files changed, 240 insertions(+), 23 deletions(-)
> 
> > +
> > +/* Real Discrete Fourier Transform */
> > +
> > +enum RDFTransformType {
> > + ? ?RDFT,
> > + ? ?IRDFT,
> > + ? ?RIDFT,
> > + ? ?IRIDFT,
> > +};
> 
> I never really liked the name IRIDFT it doesn't make a lot of sense to
> me. I just needed something at the time. Maybe we should change it
> before making it public.
> 
> RDFT: real time -> complex freq
> IRDFT: complex freq -> real time
> RIDFT: real freq -> complex time (or alternatively real time to
> complex frequency with alternate sine convention)
> IRIDFT: complex time -> real frequency (or alternatively complex
> frequency to real time with alternate sine converntion)

what about:
R2C_DFT
C2R_IDFT
R2C_IDFT
C2R_DFT
?

[...]
-- 
Michael     GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB

it is not once nor twice but times without number that the same ideas make
their appearance in the world. -- Aristotle
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 189 bytes
Desc: Digital signature
URL: <http://lists.mplayerhq.hu/pipermail/ffmpeg-devel/attachments/20100306/21fb13e0/attachment.pgp>



More information about the ffmpeg-devel mailing list