* [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