Re: [FFmpeg-devel] [PATCH v2 7/7] avcodec: add AV_CODEC_FLAG_CLEAR

2023-12-14 Thread Anton Khirnov
Quoting Marton Balint (2023-12-13 18:09:45) > On Wed, 13 Dec 2023, Anton Khirnov wrote: > > Quoting Marton Balint (2023-12-12 19:37:57) > >> > >> So for this flag, I'd rather make it clear it is not security-related, and > >> also that it has performance impact. > > > > So then maybe make a FF_EC

Re: [FFmpeg-devel] [PATCH v2 7/7] avcodec: add AV_CODEC_FLAG_CLEAR

2023-12-13 Thread Marton Balint
On Wed, 13 Dec 2023, Anton Khirnov wrote: Quoting Marton Balint (2023-12-12 19:37:57) On Tue, 12 Dec 2023, Anton Khirnov wrote: Quoting Marton Balint (2023-12-08 00:11:21) Wipe reminds me of the wipe effect. How about 'predecode_clear'? Fine with me I guess. + */ +#define

Re: [FFmpeg-devel] [PATCH v2 7/7] avcodec: add AV_CODEC_FLAG_CLEAR

2023-12-13 Thread Anton Khirnov
Quoting Marton Balint (2023-12-12 19:37:57) > > > On Tue, 12 Dec 2023, Anton Khirnov wrote: > > > Quoting Marton Balint (2023-12-08 00:11:21) > >> Wipe reminds me of the wipe effect. How about 'predecode_clear'? > > > > Fine with me I guess. > > > >>> > + */ > +#define

Re: [FFmpeg-devel] [PATCH v2 7/7] avcodec: add AV_CODEC_FLAG_CLEAR

2023-12-12 Thread Michael Niedermayer
On Tue, Dec 12, 2023 at 12:23:38PM +0100, Anton Khirnov wrote: > Quoting Marton Balint (2023-12-08 00:11:21) > > Wipe reminds me of the wipe effect. How about 'predecode_clear'? > > Fine with me I guess. > > > > > > >> + */ > > >> +#define AV_CODEC_FLAG_CLEAR (1 << 12) > > >> /** > >

Re: [FFmpeg-devel] [PATCH v2 7/7] avcodec: add AV_CODEC_FLAG_CLEAR

2023-12-12 Thread Marton Balint
On Tue, 12 Dec 2023, Anton Khirnov wrote: Quoting Marton Balint (2023-12-08 00:11:21) Wipe reminds me of the wipe effect. How about 'predecode_clear'? Fine with me I guess. + */ +#define AV_CODEC_FLAG_CLEAR (1 << 12) /** * Only decode/encode grayscale. */ diff --git

Re: [FFmpeg-devel] [PATCH v2 7/7] avcodec: add AV_CODEC_FLAG_CLEAR

2023-12-12 Thread Anton Khirnov
Quoting Marton Balint (2023-12-08 00:11:21) > Wipe reminds me of the wipe effect. How about 'predecode_clear'? Fine with me I guess. > > > >> + */ > >> +#define AV_CODEC_FLAG_CLEAR (1 << 12) > >> /** > >> * Only decode/encode grayscale. > >> */ > >> diff --git

Re: [FFmpeg-devel] [PATCH v2 7/7] avcodec: add AV_CODEC_FLAG_CLEAR

2023-12-11 Thread Mark Thompson
On 07/12/2023 23:11, Marton Balint wrote: On Thu, 7 Dec 2023, Anton Khirnov wrote: Quoting Marton Balint (2023-12-06 09:22:20) Signed-off-by: Marton Balint ---  doc/APIchanges |  3 +++  doc/codecs.texi    | 14 ++  libavcodec/avcodec.h   |  4  

Re: [FFmpeg-devel] [PATCH v2 7/7] avcodec: add AV_CODEC_FLAG_CLEAR

2023-12-07 Thread Rémi Denis-Courmont
Hi, Le 8 décembre 2023 00:47:13 GMT+02:00, Marton Balint a écrit : > > >On Thu, 7 Dec 2023, Anton Khirnov wrote: > >> Quoting Ronald S. Bultje (2023-12-07 02:44:36) >>> Hi, >>> >>> On Wed, Dec 6, 2023 at 3:23 AM Marton Balint wrote: >>> >>> > Signed-off-by: Marton Balint >>> > --- >>> >

Re: [FFmpeg-devel] [PATCH v2 7/7] avcodec: add AV_CODEC_FLAG_CLEAR

2023-12-07 Thread Marton Balint
On Thu, 7 Dec 2023, Anton Khirnov wrote: Quoting Marton Balint (2023-12-06 09:22:20) Signed-off-by: Marton Balint --- doc/APIchanges | 3 +++ doc/codecs.texi| 14 ++ libavcodec/avcodec.h | 4 libavcodec/decode.c| 6 ++

Re: [FFmpeg-devel] [PATCH v2 7/7] avcodec: add AV_CODEC_FLAG_CLEAR

2023-12-07 Thread Marton Balint
On Thu, 7 Dec 2023, Anton Khirnov wrote: Quoting Ronald S. Bultje (2023-12-07 02:44:36) Hi, On Wed, Dec 6, 2023 at 3:23 AM Marton Balint wrote: > Signed-off-by: Marton Balint > --- > doc/APIchanges | 3 +++ > doc/codecs.texi| 14 ++ >

Re: [FFmpeg-devel] [PATCH v2 7/7] avcodec: add AV_CODEC_FLAG_CLEAR

2023-12-07 Thread Anton Khirnov
Quoting Marton Balint (2023-12-06 09:22:20) > Signed-off-by: Marton Balint > --- > doc/APIchanges | 3 +++ > doc/codecs.texi| 14 ++ > libavcodec/avcodec.h | 4 > libavcodec/decode.c| 6 ++ > libavcodec/options_table.h | 1 + >

Re: [FFmpeg-devel] [PATCH v2 7/7] avcodec: add AV_CODEC_FLAG_CLEAR

2023-12-07 Thread Anton Khirnov
Quoting Ronald S. Bultje (2023-12-07 02:44:36) > Hi, > > On Wed, Dec 6, 2023 at 3:23 AM Marton Balint wrote: > > > Signed-off-by: Marton Balint > > --- > > doc/APIchanges | 3 +++ > > doc/codecs.texi| 14 ++ > > libavcodec/avcodec.h | 4 > >

Re: [FFmpeg-devel] [PATCH v2 7/7] avcodec: add AV_CODEC_FLAG_CLEAR

2023-12-06 Thread Vittorio Giovara
On Wed, Dec 6, 2023 at 3:23 AM Marton Balint wrote: > diff --git a/libavcodec/decode.c b/libavcodec/decode.c > index 2cfb3fcf97..f9b18a2c35 100644 > --- a/libavcodec/decode.c > +++ b/libavcodec/decode.c > @@ -1675,6 +1675,12 @@ FF_ENABLE_DEPRECATION_WARNINGS > >

Re: [FFmpeg-devel] [PATCH v2 7/7] avcodec: add AV_CODEC_FLAG_CLEAR

2023-12-06 Thread Ronald S. Bultje
Hi, On Wed, Dec 6, 2023 at 3:23 AM Marton Balint wrote: > Signed-off-by: Marton Balint > --- > doc/APIchanges | 3 +++ > doc/codecs.texi| 14 ++ > libavcodec/avcodec.h | 4 > libavcodec/decode.c| 6 ++ > libavcodec/options_table.h

Re: [FFmpeg-devel] [PATCH v2 7/7] avcodec: add AV_CODEC_FLAG_CLEAR

2023-12-06 Thread Stefano Sabatini
On date Wednesday 2023-12-06 09:22:20 +0100, Marton Balint wrote: > Signed-off-by: Marton Balint > --- > doc/APIchanges | 3 +++ > doc/codecs.texi| 14 ++ > libavcodec/avcodec.h | 4 > libavcodec/decode.c| 6 ++ >

[FFmpeg-devel] [PATCH v2 7/7] avcodec: add AV_CODEC_FLAG_CLEAR

2023-12-06 Thread Marton Balint
Signed-off-by: Marton Balint --- doc/APIchanges | 3 +++ doc/codecs.texi| 14 ++ libavcodec/avcodec.h | 4 libavcodec/decode.c| 6 ++ libavcodec/options_table.h | 1 + libavcodec/version.h | 2 +- 6 files changed, 29