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