* [FFmpeg-devel] [PATCH 1/2] ffplay: port to lavu/tx @ 2023-07-24 21:58 Lynne [not found] ` <Na8uIYC--3-9@lynne.ee-Na8uLym----9> 0 siblings, 1 reply; 7+ messages in thread From: Lynne @ 2023-07-24 21:58 UTC (permalink / raw) To: Ffmpeg Devel [-- Attachment #1: Type: text/plain, Size: 17 bytes --] Patch attached. [-- Attachment #2: 0001-ffplay-port-to-lavu-tx.patch --] [-- Type: text/x-diff, Size: 5056 bytes --] From a3e8ab975d21b3240a63f3f09fe486e05083ab7d Mon Sep 17 00:00:00 2001 From: Lynne <dev@lynne.ee> Date: Sat, 18 Feb 2023 13:14:31 +0100 Subject: [PATCH 1/2] ffplay: port to lavu/tx --- fftools/ffplay.c | 42 +++++++++++++++++++++++++++--------------- 1 file changed, 27 insertions(+), 15 deletions(-) diff --git a/fftools/ffplay.c b/fftools/ffplay.c index 5212ad053e..006da7ab57 100644 --- a/fftools/ffplay.c +++ b/fftools/ffplay.c @@ -47,7 +47,7 @@ #include "libavdevice/avdevice.h" #include "libswscale/swscale.h" #include "libavutil/opt.h" -#include "libavcodec/avfft.h" +#include "libavutil/tx.h" #include "libswresample/swresample.h" #include "libavfilter/avfilter.h" @@ -262,9 +262,11 @@ typedef struct VideoState { int16_t sample_array[SAMPLE_ARRAY_SIZE]; int sample_array_index; int last_i_start; - RDFTContext *rdft; + AVTXContext *rdft; + av_tx_fn rdft_fn; int rdft_bits; - FFTSample *rdft_data; + float *real_data; + AVComplexFloat *rdft_data; int xpos; double last_vis_time; SDL_Texture *vis_texture; @@ -1120,6 +1122,7 @@ static void video_audio_display(VideoState *s) fill_rectangle(s->xleft, y, s->width, 1); } } else { + int err = 0; if (realloc_texture(&s->vis_texture, SDL_PIXELFORMAT_ARGB8888, s->width, s->height, SDL_BLENDMODE_NONE, 1) < 0) return; @@ -1127,31 +1130,39 @@ static void video_audio_display(VideoState *s) s->xpos = 0; nb_display_channels= FFMIN(nb_display_channels, 2); if (rdft_bits != s->rdft_bits) { - av_rdft_end(s->rdft); - av_free(s->rdft_data); - s->rdft = av_rdft_init(rdft_bits, DFT_R2C); + const float rdft_scale = 1.0; + av_tx_uninit(&s->rdft); + av_freep(&s->real_data); + av_freep(&s->rdft_data); s->rdft_bits = rdft_bits; - s->rdft_data = av_malloc_array(nb_freq, 4 *sizeof(*s->rdft_data)); + s->real_data = av_malloc_array(nb_freq, 4 *sizeof(*s->real_data)); + s->rdft_data = av_malloc_array(nb_freq + 1, 2 *sizeof(*s->rdft_data)); + err = av_tx_init(&s->rdft, &s->rdft_fn, AV_TX_FLOAT_RDFT, + 0, 1 << rdft_bits, &rdft_scale, 0); } - if (!s->rdft || !s->rdft_data){ + if (err < 0 || !s->rdft_data) { av_log(NULL, AV_LOG_ERROR, "Failed to allocate buffers for RDFT, switching to waves display\n"); s->show_mode = SHOW_MODE_WAVES; } else { - FFTSample *data[2]; + float *data_in[2]; + AVComplexFloat *data[2]; SDL_Rect rect = {.x = s->xpos, .y = 0, .w = 1, .h = s->height}; uint32_t *pixels; int pitch; for (ch = 0; ch < nb_display_channels; ch++) { - data[ch] = s->rdft_data + 2 * nb_freq * ch; + data_in[ch] = s->real_data + 2 * nb_freq * ch; + data[ch] = s->rdft_data + nb_freq * ch; i = i_start + ch; for (x = 0; x < 2 * nb_freq; x++) { double w = (x-nb_freq) * (1.0 / nb_freq); - data[ch][x] = s->sample_array[i] * (1.0 - w * w); + data_in[ch][x] = s->sample_array[i] * (1.0 - w * w); i += channels; if (i >= SAMPLE_ARRAY_SIZE) i -= SAMPLE_ARRAY_SIZE; } - av_rdft_calc(s->rdft, data[ch]); + s->rdft_fn(s->rdft, data[ch], data_in[ch], sizeof(float)); + data[ch][0].im = data[ch][nb_freq].re; + data[ch][nb_freq].re = 0; } /* Least efficient way to do this, we should of course * directly access it but it is more than fast enough. */ @@ -1160,8 +1171,8 @@ static void video_audio_display(VideoState *s) pixels += pitch * s->height; for (y = 0; y < s->height; y++) { double w = 1 / sqrt(nb_freq); - int a = sqrt(w * sqrt(data[0][2 * y + 0] * data[0][2 * y + 0] + data[0][2 * y + 1] * data[0][2 * y + 1])); - int b = (nb_display_channels == 2 ) ? sqrt(w * hypot(data[1][2 * y + 0], data[1][2 * y + 1])) + int a = sqrt(w * sqrt(data[0][y].re * data[0][y].re + data[0][y].im * data[0][y].im)); + int b = (nb_display_channels == 2 ) ? sqrt(w * hypot(data[1][y].re, data[1][y].im)) : a; a = FFMIN(a, 255); b = FFMIN(b, 255); @@ -1197,7 +1208,8 @@ static void stream_component_close(VideoState *is, int stream_index) is->audio_buf = NULL; if (is->rdft) { - av_rdft_end(is->rdft); + av_tx_uninit(&is->rdft); + av_freep(&is->real_data); av_freep(&is->rdft_data); is->rdft = NULL; is->rdft_bits = 0; -- 2.40.1 [-- Attachment #3: Type: text/plain, Size: 251 bytes --] _______________________________________________ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org https://ffmpeg.org/mailman/listinfo/ffmpeg-devel To unsubscribe, visit link above, or email ffmpeg-devel-request@ffmpeg.org with subject "unsubscribe". ^ permalink raw reply [flat|nested] 7+ messages in thread
[parent not found: <Na8uIYC--3-9@lynne.ee-Na8uLym----9>]
* [FFmpeg-devel] [PATCH 2/2] lavc/avfft: deprecate the API [not found] ` <Na8uIYC--3-9@lynne.ee-Na8uLym----9> @ 2023-07-24 22:00 ` Lynne 2023-07-24 22:20 ` Andreas Rheinhardt 0 siblings, 1 reply; 7+ messages in thread From: Lynne @ 2023-07-24 22:00 UTC (permalink / raw) To: FFmpeg development discussions and patches [-- Attachment #1: Type: text/plain, Size: 60 bytes --] This deprecates the currently unused API. Patch attached. [-- Attachment #2: 0002-lavc-avfft-deprecate-the-API.patch --] [-- Type: text/x-diff, Size: 5070 bytes --] From 8cf7041345ebef47e710b65395095190ea88dd4a Mon Sep 17 00:00:00 2001 From: Lynne <dev@lynne.ee> Date: Mon, 24 Jul 2023 23:55:55 +0200 Subject: [PATCH 2/2] lavc/avfft: deprecate the API This deprecates the currently unused API. --- doc/APIchanges | 4 ++++ libavcodec/avfft.h | 25 +++++++++++++++++++++++++ libavcodec/tests/fft.c | 4 ++++ libavcodec/version.h | 2 +- libavcodec/version_major.h | 2 ++ 5 files changed, 36 insertions(+), 1 deletion(-) diff --git a/doc/APIchanges b/doc/APIchanges index 5afe8bcb75..38fc0b1459 100644 --- a/doc/APIchanges +++ b/doc/APIchanges @@ -2,6 +2,10 @@ The last version increases of all libraries were on 2023-02-09 API changes, most recent first: +2023-07-xx - xxxxxxxxxx - lavc 60.23.100 - avfft.h + Deprecate av_dct, av_rdft, av_fft and av_mdct. + Replaced by libavutil/tx.h + 2023-07-xx - xxxxxxxxxx - lavc 60 - avcodec.h Deprecate AV_CODEC_FLAG_DROPCHANGED without replacement. diff --git a/libavcodec/avfft.h b/libavcodec/avfft.h index 0c0f9b8d8d..6401833089 100644 --- a/libavcodec/avfft.h +++ b/libavcodec/avfft.h @@ -19,6 +19,10 @@ #ifndef AVCODEC_AVFFT_H #define AVCODEC_AVFFT_H +#include "libavutil/attributes.h" +#include "version_major.h" +#if FF_API_AVFFT + /** * @file * @ingroup lavc_fft @@ -44,26 +48,38 @@ typedef struct FFTContext FFTContext; * Set up a complex FFT. * @param nbits log2 of the length of the input array * @param inverse if 0 perform the forward transform, if 1 perform the inverse + * @deprecated use AVTXContext from libavutil/tx.h */ +attribute_deprecated FFTContext *av_fft_init(int nbits, int inverse); /** * Do the permutation needed BEFORE calling ff_fft_calc(). + * @deprecated use AVTXContext from libavutil/tx.h */ +attribute_deprecated void av_fft_permute(FFTContext *s, FFTComplex *z); /** * Do a complex FFT with the parameters defined in av_fft_init(). The * input data must be permuted before. No 1.0/sqrt(n) normalization is done. + * @deprecated use AVTXContext from libavutil/tx.h */ +attribute_deprecated void av_fft_calc(FFTContext *s, FFTComplex *z); +attribute_deprecated void av_fft_end(FFTContext *s); +attribute_deprecated FFTContext *av_mdct_init(int nbits, int inverse, double scale); +attribute_deprecated void av_imdct_calc(FFTContext *s, FFTSample *output, const FFTSample *input); +attribute_deprecated void av_imdct_half(FFTContext *s, FFTSample *output, const FFTSample *input); +attribute_deprecated void av_mdct_calc(FFTContext *s, FFTSample *output, const FFTSample *input); +attribute_deprecated void av_mdct_end(FFTContext *s); /* Real Discrete Fourier Transform */ @@ -81,9 +97,13 @@ typedef struct RDFTContext RDFTContext; * Set up a real FFT. * @param nbits log2 of the length of the input array * @param trans the type of transform + * @deprecated use AVTXContext from libavutil/tx.h */ +attribute_deprecated RDFTContext *av_rdft_init(int nbits, enum RDFTransformType trans); +attribute_deprecated void av_rdft_calc(RDFTContext *s, FFTSample *data); +attribute_deprecated void av_rdft_end(RDFTContext *s); /* Discrete Cosine Transform */ @@ -106,13 +126,18 @@ enum DCTTransformType { * @param type the type of transform * * @note the first element of the input of DST-I is ignored + * @deprecated use AVTXContext from libavutil/tx.h */ +attribute_deprecated DCTContext *av_dct_init(int nbits, enum DCTTransformType type); +attribute_deprecated void av_dct_calc(DCTContext *s, FFTSample *data); +attribute_deprecated void av_dct_end (DCTContext *s); /** * @} */ +#endif /* FF_API_AVFFT */ #endif /* AVCODEC_AVFFT_H */ diff --git a/libavcodec/tests/fft.c b/libavcodec/tests/fft.c index 163f3e89c4..a9ea457bc9 100644 --- a/libavcodec/tests/fft.c +++ b/libavcodec/tests/fft.c @@ -18,6 +18,8 @@ * Foundation, Inc., 51 Franklin Street, Fifth Floor, Boston, MA 02110-1301 USA */ +FF_DISABLE_DEPRECATION_WARNINGS + /** * @file * FFT and MDCT tests. @@ -675,3 +677,5 @@ cleanup: return !!err; } + +FF_ENABLE_DEPRECATION_WARNINGS diff --git a/libavcodec/version.h b/libavcodec/version.h index 9411511e04..728ab8839d 100644 --- a/libavcodec/version.h +++ b/libavcodec/version.h @@ -29,7 +29,7 @@ #include "version_major.h" -#define LIBAVCODEC_VERSION_MINOR 22 +#define LIBAVCODEC_VERSION_MINOR 23 #define LIBAVCODEC_VERSION_MICRO 100 #define LIBAVCODEC_VERSION_INT AV_VERSION_INT(LIBAVCODEC_VERSION_MAJOR, \ diff --git a/libavcodec/version_major.h b/libavcodec/version_major.h index 95c5aec0c5..cfd0b5023e 100644 --- a/libavcodec/version_major.h +++ b/libavcodec/version_major.h @@ -50,6 +50,8 @@ #define FF_API_TICKS_PER_FRAME (LIBAVCODEC_VERSION_MAJOR < 61) #define FF_API_DROPCHANGED (LIBAVCODEC_VERSION_MAJOR < 61) +#define FF_API_AVFFT (LIBAVCODEC_VERSION_MAJOR < 62) + // reminder to remove CrystalHD decoders on next major bump #define FF_CODEC_CRYSTAL_HD (LIBAVCODEC_VERSION_MAJOR < 61) -- 2.40.1 [-- Attachment #3: Type: text/plain, Size: 251 bytes --] _______________________________________________ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org https://ffmpeg.org/mailman/listinfo/ffmpeg-devel To unsubscribe, visit link above, or email ffmpeg-devel-request@ffmpeg.org with subject "unsubscribe". ^ permalink raw reply [flat|nested] 7+ messages in thread
* Re: [FFmpeg-devel] [PATCH 2/2] lavc/avfft: deprecate the API 2023-07-24 22:00 ` [FFmpeg-devel] [PATCH 2/2] lavc/avfft: deprecate the API Lynne @ 2023-07-24 22:20 ` Andreas Rheinhardt 2023-07-24 22:47 ` Lynne 0 siblings, 1 reply; 7+ messages in thread From: Andreas Rheinhardt @ 2023-07-24 22:20 UTC (permalink / raw) To: ffmpeg-devel Lynne: > Subject: [PATCH 2/2] lavc/avfft: deprecate the API > > This deprecates the currently unused API. ^ superseded > --- > doc/APIchanges | 4 ++++ > libavcodec/avfft.h | 25 +++++++++++++++++++++++++ > libavcodec/tests/fft.c | 4 ++++ > libavcodec/version.h | 2 +- > libavcodec/version_major.h | 2 ++ > 5 files changed, 36 insertions(+), 1 deletion(-) > > diff --git a/doc/APIchanges b/doc/APIchanges > index 5afe8bcb75..38fc0b1459 100644 > --- a/doc/APIchanges > +++ b/doc/APIchanges > @@ -2,6 +2,10 @@ The last version increases of all libraries were on 2023-02-09 > > API changes, most recent first: > > +2023-07-xx - xxxxxxxxxx - lavc 60.23.100 - avfft.h > + Deprecate av_dct, av_rdft, av_fft and av_mdct. This should mention that the whole header is deprecated and will be removed. > + Replaced by libavutil/tx.h > + > 2023-07-xx - xxxxxxxxxx - lavc 60 - avcodec.h > Deprecate AV_CODEC_FLAG_DROPCHANGED without replacement. > > diff --git a/libavcodec/avfft.h b/libavcodec/avfft.h > index 0c0f9b8d8d..6401833089 100644 > --- a/libavcodec/avfft.h > +++ b/libavcodec/avfft.h > @@ -19,6 +19,10 @@ > #ifndef AVCODEC_AVFFT_H > #define AVCODEC_AVFFT_H > > +#include "libavutil/attributes.h" > +#include "version_major.h" > +#if FF_API_AVFFT > + > /** > * @file > * @ingroup lavc_fft > @@ -44,26 +48,38 @@ typedef struct FFTContext FFTContext; > * Set up a complex FFT. > * @param nbits log2 of the length of the input array > * @param inverse if 0 perform the forward transform, if 1 perform the inverse > + * @deprecated use AVTXContext from libavutil/tx.h > */ > +attribute_deprecated > FFTContext *av_fft_init(int nbits, int inverse); > > /** > * Do the permutation needed BEFORE calling ff_fft_calc(). > + * @deprecated use AVTXContext from libavutil/tx.h > */ > +attribute_deprecated > void av_fft_permute(FFTContext *s, FFTComplex *z); > > /** > * Do a complex FFT with the parameters defined in av_fft_init(). The > * input data must be permuted before. No 1.0/sqrt(n) normalization is done. > + * @deprecated use AVTXContext from libavutil/tx.h This deprecation warning is supposed to apply to av_fft_calc() and apparently AVTXContext is not a replacement for av_fft_calc(). Probably better to write something like: "Use the AVTX API from libavutil/tx.h." > */ > +attribute_deprecated > void av_fft_calc(FFTContext *s, FFTComplex *z); > > +attribute_deprecated > void av_fft_end(FFTContext *s); > > +attribute_deprecated > FFTContext *av_mdct_init(int nbits, int inverse, double scale); > +attribute_deprecated > void av_imdct_calc(FFTContext *s, FFTSample *output, const FFTSample *input); > +attribute_deprecated > void av_imdct_half(FFTContext *s, FFTSample *output, const FFTSample *input); > +attribute_deprecated > void av_mdct_calc(FFTContext *s, FFTSample *output, const FFTSample *input); > +attribute_deprecated > void av_mdct_end(FFTContext *s); > > /* Real Discrete Fourier Transform */ > @@ -81,9 +97,13 @@ typedef struct RDFTContext RDFTContext; > * Set up a real FFT. > * @param nbits log2 of the length of the input array > * @param trans the type of transform > + * @deprecated use AVTXContext from libavutil/tx.h > */ > +attribute_deprecated > RDFTContext *av_rdft_init(int nbits, enum RDFTransformType trans); > +attribute_deprecated > void av_rdft_calc(RDFTContext *s, FFTSample *data); > +attribute_deprecated > void av_rdft_end(RDFTContext *s); > > /* Discrete Cosine Transform */ > @@ -106,13 +126,18 @@ enum DCTTransformType { > * @param type the type of transform > * > * @note the first element of the input of DST-I is ignored > + * @deprecated use AVTXContext from libavutil/tx.h > */ > +attribute_deprecated > DCTContext *av_dct_init(int nbits, enum DCTTransformType type); > +attribute_deprecated > void av_dct_calc(DCTContext *s, FFTSample *data); > +attribute_deprecated > void av_dct_end (DCTContext *s); > > /** > * @} > */ > > +#endif /* FF_API_AVFFT */ > #endif /* AVCODEC_AVFFT_H */ > diff --git a/libavcodec/tests/fft.c b/libavcodec/tests/fft.c > index 163f3e89c4..a9ea457bc9 100644 > --- a/libavcodec/tests/fft.c > +++ b/libavcodec/tests/fft.c > @@ -18,6 +18,8 @@ > * Foundation, Inc., 51 Franklin Street, Fifth Floor, Boston, MA 02110-1301 USA > */ > > +FF_DISABLE_DEPRECATION_WARNINGS > + > /** > * @file > * FFT and MDCT tests. > @@ -675,3 +677,5 @@ cleanup: > > return !!err; > } > + > +FF_ENABLE_DEPRECATION_WARNINGS > diff --git a/libavcodec/version.h b/libavcodec/version.h > index 9411511e04..728ab8839d 100644 > --- a/libavcodec/version.h > +++ b/libavcodec/version.h > @@ -29,7 +29,7 @@ > > #include "version_major.h" > > -#define LIBAVCODEC_VERSION_MINOR 22 > +#define LIBAVCODEC_VERSION_MINOR 23 > #define LIBAVCODEC_VERSION_MICRO 100 > > #define LIBAVCODEC_VERSION_INT AV_VERSION_INT(LIBAVCODEC_VERSION_MAJOR, \ > diff --git a/libavcodec/version_major.h b/libavcodec/version_major.h > index 95c5aec0c5..cfd0b5023e 100644 > --- a/libavcodec/version_major.h > +++ b/libavcodec/version_major.h > @@ -50,6 +50,8 @@ > #define FF_API_TICKS_PER_FRAME (LIBAVCODEC_VERSION_MAJOR < 61) > #define FF_API_DROPCHANGED (LIBAVCODEC_VERSION_MAJOR < 61) > > +#define FF_API_AVFFT (LIBAVCODEC_VERSION_MAJOR < 62) > + > // reminder to remove CrystalHD decoders on next major bump > #define FF_CODEC_CRYSTAL_HD (LIBAVCODEC_VERSION_MAJOR < 61) > This patch is very much orthogonal to my patch (as expected). - Andreas _______________________________________________ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org https://ffmpeg.org/mailman/listinfo/ffmpeg-devel To unsubscribe, visit link above, or email ffmpeg-devel-request@ffmpeg.org with subject "unsubscribe". ^ permalink raw reply [flat|nested] 7+ messages in thread
* Re: [FFmpeg-devel] [PATCH 2/2] lavc/avfft: deprecate the API 2023-07-24 22:20 ` Andreas Rheinhardt @ 2023-07-24 22:47 ` Lynne 2023-07-24 23:19 ` Andreas Rheinhardt 2023-07-25 23:17 ` Michael Niedermayer 0 siblings, 2 replies; 7+ messages in thread From: Lynne @ 2023-07-24 22:47 UTC (permalink / raw) To: FFmpeg development discussions and patches [-- Attachment #1: Type: text/plain, Size: 3658 bytes --] Jul 25, 2023, 00:19 by andreas.rheinhardt@outlook.com: > Lynne: > >> Subject: [PATCH 2/2] lavc/avfft: deprecate the API >> >> This deprecates the currently unused API. >> > ^ > superseded > >> --- >> doc/APIchanges | 4 ++++ >> libavcodec/avfft.h | 25 +++++++++++++++++++++++++ >> libavcodec/tests/fft.c | 4 ++++ >> libavcodec/version.h | 2 +- >> libavcodec/version_major.h | 2 ++ >> 5 files changed, 36 insertions(+), 1 deletion(-) >> >> diff --git a/doc/APIchanges b/doc/APIchanges >> index 5afe8bcb75..38fc0b1459 100644 >> --- a/doc/APIchanges >> +++ b/doc/APIchanges >> @@ -2,6 +2,10 @@ The last version increases of all libraries were on 2023-02-09 >> >> API changes, most recent first: >> >> +2023-07-xx - xxxxxxxxxx - lavc 60.23.100 - avfft.h >> + Deprecate av_dct, av_rdft, av_fft and av_mdct. >> > > This should mention that the whole header is deprecated and will be removed. > >> + Replaced by libavutil/tx.h >> + >> 2023-07-xx - xxxxxxxxxx - lavc 60 - avcodec.h >> Deprecate AV_CODEC_FLAG_DROPCHANGED without replacement. >> >> diff --git a/libavcodec/avfft.h b/libavcodec/avfft.h >> index 0c0f9b8d8d..6401833089 100644 >> --- a/libavcodec/avfft.h >> +++ b/libavcodec/avfft.h >> @@ -19,6 +19,10 @@ >> #ifndef AVCODEC_AVFFT_H >> #define AVCODEC_AVFFT_H >> >> +#include "libavutil/attributes.h" >> +#include "version_major.h" >> +#if FF_API_AVFFT >> + >> /** >> * @file >> * @ingroup lavc_fft >> @@ -44,26 +48,38 @@ typedef struct FFTContext FFTContext; >> * Set up a complex FFT. >> * @param nbits log2 of the length of the input array >> * @param inverse if 0 perform the forward transform, if 1 perform the inverse >> + * @deprecated use AVTXContext from libavutil/tx.h >> */ >> +attribute_deprecated >> FFTContext *av_fft_init(int nbits, int inverse); >> >> /** >> * Do the permutation needed BEFORE calling ff_fft_calc(). >> + * @deprecated use AVTXContext from libavutil/tx.h >> */ >> +attribute_deprecated >> void av_fft_permute(FFTContext *s, FFTComplex *z); >> >> /** >> * Do a complex FFT with the parameters defined in av_fft_init(). The >> * input data must be permuted before. No 1.0/sqrt(n) normalization is done. >> + * @deprecated use AVTXContext from libavutil/tx.h >> > > This deprecation warning is supposed to apply to av_fft_calc() and > apparently AVTXContext is not a replacement for av_fft_calc(). Probably > better to write something like: "Use the AVTX API from libavutil/tx.h." > v2 attached. Added a comment to both APIchanges on a replacement, and added comments to this file to indicate how to replace each functionality. >> >> -#define LIBAVCODEC_VERSION_MINOR 22 >> +#define LIBAVCODEC_VERSION_MINOR 23 >> #define LIBAVCODEC_VERSION_MICRO 100 >> >> #define LIBAVCODEC_VERSION_INT AV_VERSION_INT(LIBAVCODEC_VERSION_MAJOR, \ >> diff --git a/libavcodec/version_major.h b/libavcodec/version_major.h >> index 95c5aec0c5..cfd0b5023e 100644 >> --- a/libavcodec/version_major.h >> +++ b/libavcodec/version_major.h >> @@ -50,6 +50,8 @@ >> #define FF_API_TICKS_PER_FRAME (LIBAVCODEC_VERSION_MAJOR < 61) >> #define FF_API_DROPCHANGED (LIBAVCODEC_VERSION_MAJOR < 61) >> >> +#define FF_API_AVFFT (LIBAVCODEC_VERSION_MAJOR < 62) >> + >> // reminder to remove CrystalHD decoders on next major bump >> #define FF_CODEC_CRYSTAL_HD (LIBAVCODEC_VERSION_MAJOR < 61) >> > > This patch is very much orthogonal to my patch (as expected). > This patch is - but my next series of patches will enable full removal of all FFT code from lavc - which I would prefer to do all at once, if possible. [-- Attachment #2: v2-0002-lavc-avfft-deprecate-the-API.patch --] [-- Type: text/x-diff, Size: 5475 bytes --] From fc7891b0cfeb296525919d1ff8baaa2c0b68bbeb Mon Sep 17 00:00:00 2001 From: Lynne <dev@lynne.ee> Date: Mon, 24 Jul 2023 23:55:55 +0200 Subject: [PATCH v2 2/2] lavc/avfft: deprecate the API This deprecates the currently unused API. --- doc/APIchanges | 5 +++++ libavcodec/avfft.h | 31 +++++++++++++++++++++++++++++++ libavcodec/tests/fft.c | 4 ++++ libavcodec/version.h | 2 +- libavcodec/version_major.h | 2 ++ 5 files changed, 43 insertions(+), 1 deletion(-) diff --git a/doc/APIchanges b/doc/APIchanges index 5afe8bcb75..6f6c3b4aa6 100644 --- a/doc/APIchanges +++ b/doc/APIchanges @@ -2,6 +2,11 @@ The last version increases of all libraries were on 2023-02-09 API changes, most recent first: +2023-07-xx - xxxxxxxxxx - lavc 60.23.100 - avfft.h + The entire header will be deprecated and removed in two major bumps. + For a replacement to av_dct, av_rdft, av_fft and av_mdct, use + the new API from libavutil/tx.h. + 2023-07-xx - xxxxxxxxxx - lavc 60 - avcodec.h Deprecate AV_CODEC_FLAG_DROPCHANGED without replacement. diff --git a/libavcodec/avfft.h b/libavcodec/avfft.h index 0c0f9b8d8d..e3a0da1eb9 100644 --- a/libavcodec/avfft.h +++ b/libavcodec/avfft.h @@ -19,6 +19,10 @@ #ifndef AVCODEC_AVFFT_H #define AVCODEC_AVFFT_H +#include "libavutil/attributes.h" +#include "version_major.h" +#if FF_API_AVFFT + /** * @file * @ingroup lavc_fft @@ -44,26 +48,42 @@ typedef struct FFTContext FFTContext; * Set up a complex FFT. * @param nbits log2 of the length of the input array * @param inverse if 0 perform the forward transform, if 1 perform the inverse + * @deprecated use av_tx_init from libavutil/tx.h with a type of AV_TX_FLOAT_FFT */ +attribute_deprecated FFTContext *av_fft_init(int nbits, int inverse); /** * Do the permutation needed BEFORE calling ff_fft_calc(). + * @deprecated without replacement */ +attribute_deprecated void av_fft_permute(FFTContext *s, FFTComplex *z); /** * Do a complex FFT with the parameters defined in av_fft_init(). The * input data must be permuted before. No 1.0/sqrt(n) normalization is done. + * @deprecated use the av_tx_fn value returned by av_tx_init, which also does permutation */ +attribute_deprecated void av_fft_calc(FFTContext *s, FFTComplex *z); +attribute_deprecated void av_fft_end(FFTContext *s); +/** + * @deprecated use av_tx_init from libavutil/tx.h with a type of AV_TX_FLOAT_MDCT, + * with a flag of AV_TX_FULL_IMDCT for a replacement to av_imdct_calc. + */ +attribute_deprecated FFTContext *av_mdct_init(int nbits, int inverse, double scale); +attribute_deprecated void av_imdct_calc(FFTContext *s, FFTSample *output, const FFTSample *input); +attribute_deprecated void av_imdct_half(FFTContext *s, FFTSample *output, const FFTSample *input); +attribute_deprecated void av_mdct_calc(FFTContext *s, FFTSample *output, const FFTSample *input); +attribute_deprecated void av_mdct_end(FFTContext *s); /* Real Discrete Fourier Transform */ @@ -81,9 +101,14 @@ typedef struct RDFTContext RDFTContext; * Set up a real FFT. * @param nbits log2 of the length of the input array * @param trans the type of transform + * + * @deprecated use av_tx_init from libavutil/tx.h with a type of AV_TX_FLOAT_RDFT */ +attribute_deprecated RDFTContext *av_rdft_init(int nbits, enum RDFTransformType trans); +attribute_deprecated void av_rdft_calc(RDFTContext *s, FFTSample *data); +attribute_deprecated void av_rdft_end(RDFTContext *s); /* Discrete Cosine Transform */ @@ -106,13 +131,19 @@ enum DCTTransformType { * @param type the type of transform * * @note the first element of the input of DST-I is ignored + * + * @deprecated use av_tx_init from libavutil/tx.h with an appropriate type of AV_TX_FLOAT_DCT */ +attribute_deprecated DCTContext *av_dct_init(int nbits, enum DCTTransformType type); +attribute_deprecated void av_dct_calc(DCTContext *s, FFTSample *data); +attribute_deprecated void av_dct_end (DCTContext *s); /** * @} */ +#endif /* FF_API_AVFFT */ #endif /* AVCODEC_AVFFT_H */ diff --git a/libavcodec/tests/fft.c b/libavcodec/tests/fft.c index 163f3e89c4..a9ea457bc9 100644 --- a/libavcodec/tests/fft.c +++ b/libavcodec/tests/fft.c @@ -18,6 +18,8 @@ * Foundation, Inc., 51 Franklin Street, Fifth Floor, Boston, MA 02110-1301 USA */ +FF_DISABLE_DEPRECATION_WARNINGS + /** * @file * FFT and MDCT tests. @@ -675,3 +677,5 @@ cleanup: return !!err; } + +FF_ENABLE_DEPRECATION_WARNINGS diff --git a/libavcodec/version.h b/libavcodec/version.h index 9411511e04..728ab8839d 100644 --- a/libavcodec/version.h +++ b/libavcodec/version.h @@ -29,7 +29,7 @@ #include "version_major.h" -#define LIBAVCODEC_VERSION_MINOR 22 +#define LIBAVCODEC_VERSION_MINOR 23 #define LIBAVCODEC_VERSION_MICRO 100 #define LIBAVCODEC_VERSION_INT AV_VERSION_INT(LIBAVCODEC_VERSION_MAJOR, \ diff --git a/libavcodec/version_major.h b/libavcodec/version_major.h index 95c5aec0c5..cfd0b5023e 100644 --- a/libavcodec/version_major.h +++ b/libavcodec/version_major.h @@ -50,6 +50,8 @@ #define FF_API_TICKS_PER_FRAME (LIBAVCODEC_VERSION_MAJOR < 61) #define FF_API_DROPCHANGED (LIBAVCODEC_VERSION_MAJOR < 61) +#define FF_API_AVFFT (LIBAVCODEC_VERSION_MAJOR < 62) + // reminder to remove CrystalHD decoders on next major bump #define FF_CODEC_CRYSTAL_HD (LIBAVCODEC_VERSION_MAJOR < 61) -- 2.40.1 [-- Attachment #3: Type: text/plain, Size: 251 bytes --] _______________________________________________ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org https://ffmpeg.org/mailman/listinfo/ffmpeg-devel To unsubscribe, visit link above, or email ffmpeg-devel-request@ffmpeg.org with subject "unsubscribe". ^ permalink raw reply [flat|nested] 7+ messages in thread
* Re: [FFmpeg-devel] [PATCH 2/2] lavc/avfft: deprecate the API 2023-07-24 22:47 ` Lynne @ 2023-07-24 23:19 ` Andreas Rheinhardt 2023-07-25 23:17 ` Michael Niedermayer 1 sibling, 0 replies; 7+ messages in thread From: Andreas Rheinhardt @ 2023-07-24 23:19 UTC (permalink / raw) To: ffmpeg-devel Lynne: > Jul 25, 2023, 00:19 by andreas.rheinhardt@outlook.com: > >> Lynne: >> >>> Subject: [PATCH 2/2] lavc/avfft: deprecate the API >>> >>> This deprecates the currently unused API. >>> >> ^ >> superseded >> >>> --- >>> doc/APIchanges | 4 ++++ >>> libavcodec/avfft.h | 25 +++++++++++++++++++++++++ >>> libavcodec/tests/fft.c | 4 ++++ >>> libavcodec/version.h | 2 +- >>> libavcodec/version_major.h | 2 ++ >>> 5 files changed, 36 insertions(+), 1 deletion(-) >>> >>> diff --git a/doc/APIchanges b/doc/APIchanges >>> index 5afe8bcb75..38fc0b1459 100644 >>> --- a/doc/APIchanges >>> +++ b/doc/APIchanges >>> @@ -2,6 +2,10 @@ The last version increases of all libraries were on 2023-02-09 >>> >>> API changes, most recent first: >>> >>> +2023-07-xx - xxxxxxxxxx - lavc 60.23.100 - avfft.h >>> + Deprecate av_dct, av_rdft, av_fft and av_mdct. >>> >> >> This should mention that the whole header is deprecated and will be removed. >> >>> + Replaced by libavutil/tx.h >>> + >>> 2023-07-xx - xxxxxxxxxx - lavc 60 - avcodec.h >>> Deprecate AV_CODEC_FLAG_DROPCHANGED without replacement. >>> >>> diff --git a/libavcodec/avfft.h b/libavcodec/avfft.h >>> index 0c0f9b8d8d..6401833089 100644 >>> --- a/libavcodec/avfft.h >>> +++ b/libavcodec/avfft.h >>> @@ -19,6 +19,10 @@ >>> #ifndef AVCODEC_AVFFT_H >>> #define AVCODEC_AVFFT_H >>> >>> +#include "libavutil/attributes.h" >>> +#include "version_major.h" >>> +#if FF_API_AVFFT >>> + >>> /** >>> * @file >>> * @ingroup lavc_fft >>> @@ -44,26 +48,38 @@ typedef struct FFTContext FFTContext; >>> * Set up a complex FFT. >>> * @param nbits log2 of the length of the input array >>> * @param inverse if 0 perform the forward transform, if 1 perform the inverse >>> + * @deprecated use AVTXContext from libavutil/tx.h >>> */ >>> +attribute_deprecated >>> FFTContext *av_fft_init(int nbits, int inverse); >>> >>> /** >>> * Do the permutation needed BEFORE calling ff_fft_calc(). >>> + * @deprecated use AVTXContext from libavutil/tx.h >>> */ >>> +attribute_deprecated >>> void av_fft_permute(FFTContext *s, FFTComplex *z); >>> >>> /** >>> * Do a complex FFT with the parameters defined in av_fft_init(). The >>> * input data must be permuted before. No 1.0/sqrt(n) normalization is done. >>> + * @deprecated use AVTXContext from libavutil/tx.h >>> >> >> This deprecation warning is supposed to apply to av_fft_calc() and >> apparently AVTXContext is not a replacement for av_fft_calc(). Probably >> better to write something like: "Use the AVTX API from libavutil/tx.h." >> > > v2 attached. Added a comment to both APIchanges > on a replacement, and added comments to this file to > indicate how to replace each functionality. > Looks good to me. But wait a day so that others (Anton?) can take a look at it, too. - Andreas _______________________________________________ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org https://ffmpeg.org/mailman/listinfo/ffmpeg-devel To unsubscribe, visit link above, or email ffmpeg-devel-request@ffmpeg.org with subject "unsubscribe". ^ permalink raw reply [flat|nested] 7+ messages in thread
* Re: [FFmpeg-devel] [PATCH 2/2] lavc/avfft: deprecate the API 2023-07-24 22:47 ` Lynne 2023-07-24 23:19 ` Andreas Rheinhardt @ 2023-07-25 23:17 ` Michael Niedermayer 2023-07-26 7:28 ` Lynne 1 sibling, 1 reply; 7+ messages in thread From: Michael Niedermayer @ 2023-07-25 23:17 UTC (permalink / raw) To: FFmpeg development discussions and patches [-- Attachment #1.1: Type: text/plain, Size: 5674 bytes --] On Tue, Jul 25, 2023 at 12:47:12AM +0200, Lynne wrote: > Jul 25, 2023, 00:19 by andreas.rheinhardt@outlook.com: > > > Lynne: > > > >> Subject: [PATCH 2/2] lavc/avfft: deprecate the API > >> > >> This deprecates the currently unused API. > >> > > ^ > > superseded > > > >> --- > >> doc/APIchanges | 4 ++++ > >> libavcodec/avfft.h | 25 +++++++++++++++++++++++++ > >> libavcodec/tests/fft.c | 4 ++++ > >> libavcodec/version.h | 2 +- > >> libavcodec/version_major.h | 2 ++ > >> 5 files changed, 36 insertions(+), 1 deletion(-) > >> > >> diff --git a/doc/APIchanges b/doc/APIchanges > >> index 5afe8bcb75..38fc0b1459 100644 > >> --- a/doc/APIchanges > >> +++ b/doc/APIchanges > >> @@ -2,6 +2,10 @@ The last version increases of all libraries were on 2023-02-09 > >> > >> API changes, most recent first: > >> > >> +2023-07-xx - xxxxxxxxxx - lavc 60.23.100 - avfft.h > >> + Deprecate av_dct, av_rdft, av_fft and av_mdct. > >> > > > > This should mention that the whole header is deprecated and will be removed. > > > >> + Replaced by libavutil/tx.h > >> + > >> 2023-07-xx - xxxxxxxxxx - lavc 60 - avcodec.h > >> Deprecate AV_CODEC_FLAG_DROPCHANGED without replacement. > >> > >> diff --git a/libavcodec/avfft.h b/libavcodec/avfft.h > >> index 0c0f9b8d8d..6401833089 100644 > >> --- a/libavcodec/avfft.h > >> +++ b/libavcodec/avfft.h > >> @@ -19,6 +19,10 @@ > >> #ifndef AVCODEC_AVFFT_H > >> #define AVCODEC_AVFFT_H > >> > >> +#include "libavutil/attributes.h" > >> +#include "version_major.h" > >> +#if FF_API_AVFFT > >> + > >> /** > >> * @file > >> * @ingroup lavc_fft > >> @@ -44,26 +48,38 @@ typedef struct FFTContext FFTContext; > >> * Set up a complex FFT. > >> * @param nbits log2 of the length of the input array > >> * @param inverse if 0 perform the forward transform, if 1 perform the inverse > >> + * @deprecated use AVTXContext from libavutil/tx.h > >> */ > >> +attribute_deprecated > >> FFTContext *av_fft_init(int nbits, int inverse); > >> > >> /** > >> * Do the permutation needed BEFORE calling ff_fft_calc(). > >> + * @deprecated use AVTXContext from libavutil/tx.h > >> */ > >> +attribute_deprecated > >> void av_fft_permute(FFTContext *s, FFTComplex *z); > >> > >> /** > >> * Do a complex FFT with the parameters defined in av_fft_init(). The > >> * input data must be permuted before. No 1.0/sqrt(n) normalization is done. > >> + * @deprecated use AVTXContext from libavutil/tx.h > >> > > > > This deprecation warning is supposed to apply to av_fft_calc() and > > apparently AVTXContext is not a replacement for av_fft_calc(). Probably > > better to write something like: "Use the AVTX API from libavutil/tx.h." > > > > v2 attached. Added a comment to both APIchanges > on a replacement, and added comments to this file to > indicate how to replace each functionality. > > > >> > >> -#define LIBAVCODEC_VERSION_MINOR 22 > >> +#define LIBAVCODEC_VERSION_MINOR 23 > >> #define LIBAVCODEC_VERSION_MICRO 100 > >> > >> #define LIBAVCODEC_VERSION_INT AV_VERSION_INT(LIBAVCODEC_VERSION_MAJOR, \ > >> diff --git a/libavcodec/version_major.h b/libavcodec/version_major.h > >> index 95c5aec0c5..cfd0b5023e 100644 > >> --- a/libavcodec/version_major.h > >> +++ b/libavcodec/version_major.h > >> @@ -50,6 +50,8 @@ > >> #define FF_API_TICKS_PER_FRAME (LIBAVCODEC_VERSION_MAJOR < 61) > >> #define FF_API_DROPCHANGED (LIBAVCODEC_VERSION_MAJOR < 61) > >> > >> +#define FF_API_AVFFT (LIBAVCODEC_VERSION_MAJOR < 62) > >> + > >> // reminder to remove CrystalHD decoders on next major bump > >> #define FF_CODEC_CRYSTAL_HD (LIBAVCODEC_VERSION_MAJOR < 61) > >> > > > > This patch is very much orthogonal to my patch (as expected). > > > > This patch is - but my next series of patches will enable full removal of > all FFT code from lavc - which I would prefer to do all at once, if possible. > > > doc/APIchanges | 5 +++++ > libavcodec/avfft.h | 31 +++++++++++++++++++++++++++++++ > libavcodec/tests/fft.c | 4 ++++ > libavcodec/version.h | 2 +- > libavcodec/version_major.h | 2 ++ > 5 files changed, 43 insertions(+), 1 deletion(-) > 601e82db992b2dbebc61b09f460d703c488aa228 v2-0002-lavc-avfft-deprecate-the-API.patch > From fc7891b0cfeb296525919d1ff8baaa2c0b68bbeb Mon Sep 17 00:00:00 2001 > From: Lynne <dev@lynne.ee> > Date: Mon, 24 Jul 2023 23:55:55 +0200 > Subject: [PATCH v2 2/2] lavc/avfft: deprecate the API > > This deprecates the currently unused API. > --- > doc/APIchanges | 5 +++++ > libavcodec/avfft.h | 31 +++++++++++++++++++++++++++++++ > libavcodec/tests/fft.c | 4 ++++ > libavcodec/version.h | 2 +- > libavcodec/version_major.h | 2 ++ > 5 files changed, 43 insertions(+), 1 deletion(-) not sure i missed a patch but this breaks "make testprogs" here CC libavcodec/tests/avfft.o In file included from /usr/include/math.h:37:0, from libavcodec/tests/fft.c:34, from libavcodec/tests/avfft.c:25: /usr/include/x86_64-linux-gnu/bits/types.h:30:1: error: expected ‘=’, ‘,’, ‘;’, ‘asm’ or ‘__attribute__’ before ‘typedef’ typedef unsigned char __u_char; ^~~~~~~ thx [...] -- Michael GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB Take away the freedom of one citizen and you will be jailed, take away the freedom of all citizens and you will be congratulated by your peers in Parliament. [-- Attachment #1.2: signature.asc --] [-- Type: application/pgp-signature, Size: 195 bytes --] [-- Attachment #2: Type: text/plain, Size: 251 bytes --] _______________________________________________ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org https://ffmpeg.org/mailman/listinfo/ffmpeg-devel To unsubscribe, visit link above, or email ffmpeg-devel-request@ffmpeg.org with subject "unsubscribe". ^ permalink raw reply [flat|nested] 7+ messages in thread
* Re: [FFmpeg-devel] [PATCH 2/2] lavc/avfft: deprecate the API 2023-07-25 23:17 ` Michael Niedermayer @ 2023-07-26 7:28 ` Lynne 0 siblings, 0 replies; 7+ messages in thread From: Lynne @ 2023-07-26 7:28 UTC (permalink / raw) To: FFmpeg development discussions and patches [-- Attachment #1: Type: text/plain, Size: 935 bytes --] Jul 26, 2023, 01:17 by michael@niedermayer.cc: > On Tue, Jul 25, 2023 at 12:47:12AM +0200, Lynne wrote: > >> This deprecates the currently unused API. >> --- >> doc/APIchanges | 5 +++++ >> libavcodec/avfft.h | 31 +++++++++++++++++++++++++++++++ >> libavcodec/tests/fft.c | 4 ++++ >> libavcodec/version.h | 2 +- >> libavcodec/version_major.h | 2 ++ >> 5 files changed, 43 insertions(+), 1 deletion(-) >> > > not sure i missed a patch but this breaks > "make testprogs" here > > CC libavcodec/tests/avfft.o > In file included from /usr/include/math.h:37:0, > from libavcodec/tests/fft.c:34, > from libavcodec/tests/avfft.c:25: > /usr/include/x86_64-linux-gnu/bits/types.h:30:1: error: expected ‘=’, ‘,’, ‘;’, ‘asm’ or ‘__attribute__’ before ‘typedef’ > typedef unsigned char __u_char; > ^~~~~~~ > > thx > Thanks, fixed in attached v3. [-- Attachment #2: v3-0002-lavc-avfft-deprecate-the-API.patch --] [-- Type: text/x-diff, Size: 5513 bytes --] From 4573f20278b817e4a2d59f010b02f467171fbf42 Mon Sep 17 00:00:00 2001 From: Lynne <dev@lynne.ee> Date: Mon, 24 Jul 2023 23:55:55 +0200 Subject: [PATCH v3 2/2] lavc/avfft: deprecate the API This deprecates the currently unused API. --- doc/APIchanges | 5 +++++ libavcodec/avfft.h | 31 +++++++++++++++++++++++++++++++ libavcodec/tests/fft.c | 6 ++++++ libavcodec/version.h | 2 +- libavcodec/version_major.h | 2 ++ 5 files changed, 45 insertions(+), 1 deletion(-) diff --git a/doc/APIchanges b/doc/APIchanges index 5afe8bcb75..6f6c3b4aa6 100644 --- a/doc/APIchanges +++ b/doc/APIchanges @@ -2,6 +2,11 @@ The last version increases of all libraries were on 2023-02-09 API changes, most recent first: +2023-07-xx - xxxxxxxxxx - lavc 60.23.100 - avfft.h + The entire header will be deprecated and removed in two major bumps. + For a replacement to av_dct, av_rdft, av_fft and av_mdct, use + the new API from libavutil/tx.h. + 2023-07-xx - xxxxxxxxxx - lavc 60 - avcodec.h Deprecate AV_CODEC_FLAG_DROPCHANGED without replacement. diff --git a/libavcodec/avfft.h b/libavcodec/avfft.h index 0c0f9b8d8d..e3a0da1eb9 100644 --- a/libavcodec/avfft.h +++ b/libavcodec/avfft.h @@ -19,6 +19,10 @@ #ifndef AVCODEC_AVFFT_H #define AVCODEC_AVFFT_H +#include "libavutil/attributes.h" +#include "version_major.h" +#if FF_API_AVFFT + /** * @file * @ingroup lavc_fft @@ -44,26 +48,42 @@ typedef struct FFTContext FFTContext; * Set up a complex FFT. * @param nbits log2 of the length of the input array * @param inverse if 0 perform the forward transform, if 1 perform the inverse + * @deprecated use av_tx_init from libavutil/tx.h with a type of AV_TX_FLOAT_FFT */ +attribute_deprecated FFTContext *av_fft_init(int nbits, int inverse); /** * Do the permutation needed BEFORE calling ff_fft_calc(). + * @deprecated without replacement */ +attribute_deprecated void av_fft_permute(FFTContext *s, FFTComplex *z); /** * Do a complex FFT with the parameters defined in av_fft_init(). The * input data must be permuted before. No 1.0/sqrt(n) normalization is done. + * @deprecated use the av_tx_fn value returned by av_tx_init, which also does permutation */ +attribute_deprecated void av_fft_calc(FFTContext *s, FFTComplex *z); +attribute_deprecated void av_fft_end(FFTContext *s); +/** + * @deprecated use av_tx_init from libavutil/tx.h with a type of AV_TX_FLOAT_MDCT, + * with a flag of AV_TX_FULL_IMDCT for a replacement to av_imdct_calc. + */ +attribute_deprecated FFTContext *av_mdct_init(int nbits, int inverse, double scale); +attribute_deprecated void av_imdct_calc(FFTContext *s, FFTSample *output, const FFTSample *input); +attribute_deprecated void av_imdct_half(FFTContext *s, FFTSample *output, const FFTSample *input); +attribute_deprecated void av_mdct_calc(FFTContext *s, FFTSample *output, const FFTSample *input); +attribute_deprecated void av_mdct_end(FFTContext *s); /* Real Discrete Fourier Transform */ @@ -81,9 +101,14 @@ typedef struct RDFTContext RDFTContext; * Set up a real FFT. * @param nbits log2 of the length of the input array * @param trans the type of transform + * + * @deprecated use av_tx_init from libavutil/tx.h with a type of AV_TX_FLOAT_RDFT */ +attribute_deprecated RDFTContext *av_rdft_init(int nbits, enum RDFTransformType trans); +attribute_deprecated void av_rdft_calc(RDFTContext *s, FFTSample *data); +attribute_deprecated void av_rdft_end(RDFTContext *s); /* Discrete Cosine Transform */ @@ -106,13 +131,19 @@ enum DCTTransformType { * @param type the type of transform * * @note the first element of the input of DST-I is ignored + * + * @deprecated use av_tx_init from libavutil/tx.h with an appropriate type of AV_TX_FLOAT_DCT */ +attribute_deprecated DCTContext *av_dct_init(int nbits, enum DCTTransformType type); +attribute_deprecated void av_dct_calc(DCTContext *s, FFTSample *data); +attribute_deprecated void av_dct_end (DCTContext *s); /** * @} */ +#endif /* FF_API_AVFFT */ #endif /* AVCODEC_AVFFT_H */ diff --git a/libavcodec/tests/fft.c b/libavcodec/tests/fft.c index 163f3e89c4..0f03c9232d 100644 --- a/libavcodec/tests/fft.c +++ b/libavcodec/tests/fft.c @@ -18,6 +18,10 @@ * Foundation, Inc., 51 Franklin Street, Fifth Floor, Boston, MA 02110-1301 USA */ +#include "libavutil/internal.h" + +FF_DISABLE_DEPRECATION_WARNINGS + /** * @file * FFT and MDCT tests. @@ -675,3 +679,5 @@ cleanup: return !!err; } + +FF_ENABLE_DEPRECATION_WARNINGS diff --git a/libavcodec/version.h b/libavcodec/version.h index 9411511e04..728ab8839d 100644 --- a/libavcodec/version.h +++ b/libavcodec/version.h @@ -29,7 +29,7 @@ #include "version_major.h" -#define LIBAVCODEC_VERSION_MINOR 22 +#define LIBAVCODEC_VERSION_MINOR 23 #define LIBAVCODEC_VERSION_MICRO 100 #define LIBAVCODEC_VERSION_INT AV_VERSION_INT(LIBAVCODEC_VERSION_MAJOR, \ diff --git a/libavcodec/version_major.h b/libavcodec/version_major.h index 95c5aec0c5..cfd0b5023e 100644 --- a/libavcodec/version_major.h +++ b/libavcodec/version_major.h @@ -50,6 +50,8 @@ #define FF_API_TICKS_PER_FRAME (LIBAVCODEC_VERSION_MAJOR < 61) #define FF_API_DROPCHANGED (LIBAVCODEC_VERSION_MAJOR < 61) +#define FF_API_AVFFT (LIBAVCODEC_VERSION_MAJOR < 62) + // reminder to remove CrystalHD decoders on next major bump #define FF_CODEC_CRYSTAL_HD (LIBAVCODEC_VERSION_MAJOR < 61) -- 2.40.1 [-- Attachment #3: Type: text/plain, Size: 251 bytes --] _______________________________________________ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org https://ffmpeg.org/mailman/listinfo/ffmpeg-devel To unsubscribe, visit link above, or email ffmpeg-devel-request@ffmpeg.org with subject "unsubscribe". ^ permalink raw reply [flat|nested] 7+ messages in thread
end of thread, other threads:[~2023-07-26 7:28 UTC | newest] Thread overview: 7+ messages (download: mbox.gz / follow: Atom feed) -- links below jump to the message on this page -- 2023-07-24 21:58 [FFmpeg-devel] [PATCH 1/2] ffplay: port to lavu/tx Lynne [not found] ` <Na8uIYC--3-9@lynne.ee-Na8uLym----9> 2023-07-24 22:00 ` [FFmpeg-devel] [PATCH 2/2] lavc/avfft: deprecate the API Lynne 2023-07-24 22:20 ` Andreas Rheinhardt 2023-07-24 22:47 ` Lynne 2023-07-24 23:19 ` Andreas Rheinhardt 2023-07-25 23:17 ` Michael Niedermayer 2023-07-26 7:28 ` Lynne
Git Inbox Mirror of the ffmpeg-devel mailing list - see https://ffmpeg.org/mailman/listinfo/ffmpeg-devel This inbox may be cloned and mirrored by anyone: git clone --mirror https://master.gitmailbox.com/ffmpegdev/0 ffmpegdev/git/0.git # If you have public-inbox 1.1+ installed, you may # initialize and index your mirror using the following commands: public-inbox-init -V2 ffmpegdev ffmpegdev/ https://master.gitmailbox.com/ffmpegdev \ ffmpegdev@gitmailbox.com public-inbox-index ffmpegdev Example config snippet for mirrors. AGPL code for this site: git clone https://public-inbox.org/public-inbox.git