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

Michael Niedermayer michaelni
Sat Mar 6 17:36:04 CET 2010


On Sat, Mar 06, 2010 at 03:09:33PM +0000, M?ns Rullg?rd wrote:
> Alex Converse <alex.converse at gmail.com> writes:
> 
> > On Sat, Mar 6, 2010 at 9:26 AM, Michael Niedermayer <michaelni at gmx.at> wrote:
> >> 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
> >> ?
> >>
> >
> > that's better. I like DFT_C2R over C2R_DFT but I don't really care
> > about it enough to make a big deal over it.
> 
> I was about to suggest the same change.

be it, whichever way the majority prefers

[...]

-- 
Michael     GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB

Complexity theory is the science of finding the exact solution to an
approximation. Benchmarking OTOH is finding an approximation of the exact
-------------- 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/f536a91f/attachment.pgp>



More information about the ffmpeg-devel mailing list