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

[Michael Niedermayer]

On Sat, Mar 06, 2010 at 04:53:50PM +0000, M?ns Rullg?rd wrote:
> Michael Niedermayer <michaelni at gmx.at> writes:
> 
> > On Sat, Mar 06, 2010 at 03:38:06AM +0000, Mans Rullgard wrote:
> >> ---
> >>  libavcodec/Makefile |    4 +-
> >>  libavcodec/avfft.c  |  142 +++++++++++++++++++++++++++++++++++++++++++++++++++
> >>  libavcodec/avfft.h  |   89 ++++++++++++++++++++++++++++++++
> >>  libavcodec/fft.h    |   28 +++--------
> >>  4 files changed, 240 insertions(+), 23 deletions(-)
> > [...]
> >> +FFTContext *av_fft_init(int nbits, int inverse)
> >> +{
> >> +    FFTContext *s = av_malloc(sizeof(*s));
> >> +
> >> +    if (s)
> >> +        ff_fft_init(s, nbits, inverse);
> >> +
> >> +    return s;
> >> +}
> >
> > ff_fft_init() could be changed to this API and renamed to av_
> 
> Many codecs have an FFTContext as part of a larger struct, so that
> wouldn't work.  Keeping it that way is probably more efficient with
> memory, caches etc.

ok then

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

Many that live deserve death. And some that die deserve life. Can you give
it to them? Then do not be too eager to deal out death in judgement. For
even the very wise cannot see all ends. -- Gandalf
-------------- 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/8e5516ba/attachment.pgp>