Re: [FFmpeg-devel] [FFmpeg-cvslog] avcodec/magicyuv: add vlc multi support

2023-10-20 Thread Michael Niedermayer
On Wed, Sep 06, 2023 at 10:19:27PM +, Paul B Mahol wrote: > ffmpeg | branch: master | Paul B Mahol | Mon Aug 28 > 12:20:15 2023 +0200| [8b7391cb5ff94ce94612fda69392a95d7ab1ffd0] | committer: > Paul B Mahol > > avcodec/magicyuv: add vlc multi support > > Gives nice speed boost, depending

Re: [FFmpeg-devel] [PATCH v2 00/13] YUVJ removal

2023-10-20 Thread Ronald S. Bultje
Hi, On Fri, Oct 20, 2023 at 12:14 PM Vittorio Giovara < vittorio.giov...@gmail.com> wrote: > tbh while it's great that this functionality exists *somewhere* in ffmpeg, > this is really something that should belong to the scaler filter (whichever > implementation) > I agree. The reason this was

Re: [FFmpeg-devel] trac spam

2023-10-20 Thread Cosmin Stejerean via ffmpeg-devel
> On Oct 20, 2023, at 10:53 AM, Michael Koch > wrote: > >> should i add /pipermail/ffmpeg-trac to robots.txt ? > is there anyone who googles in these ffmpeg trac archives ? > > I don't. I'd imagine anyone wanting to search would search trac itself rather than the mailing list archive. If

Re: [FFmpeg-devel] trac spam

2023-10-20 Thread Michael Koch
strings that are certain spam, you can just add that twice for double the points lost done. should i add /pipermail/ffmpeg-trac to robots.txt ? is there anyone who googles in these ffmpeg trac archives ? I don't. Michael ___ ffmpeg-devel

Re: [FFmpeg-devel] [PATCH] ffprobe: add -output_format as an alias of -of

2023-10-20 Thread Stefano Sabatini
On date Wednesday 2023-10-18 23:45:11 +0200, Stefano Sabatini wrote: > On date Sunday 2023-10-15 11:29:10 +0200, Stefano Sabatini wrote: > > Currently we have -of and -print_format, which is a bit confusing. Add > > -output_format as an alias of -of to match the short name. > > --- > > Changelog

Re: [FFmpeg-devel] [PATCH] ffprobe: fix XML rendering, review XML layout

2023-10-20 Thread Stefano Sabatini
On date Wednesday 2023-10-18 23:44:45 +0200, Stefano Sabatini wrote: > On date Tuesday 2023-10-17 11:56:35 +0200, Stefano Sabatini wrote: > > On date Sunday 2023-10-15 19:15:16 +0200, Michael Niedermayer wrote: > > > On Sat, Oct 14, 2023 at 09:50:24PM +0200, Stefano Sabatini wrote: > > > > On date

Re: [FFmpeg-devel] [PATCH 1/2] avcodec/pthread_frame: Remove ff_thread_release_buffer()

2023-10-20 Thread Anton Khirnov
Quoting Andreas Rheinhardt (2023-10-20 17:41:17) > Anton Khirnov: > > Quoting Andreas Rheinhardt (2023-10-20 16:33:06) > >> diff --git a/libavcodec/av1dec.c b/libavcodec/av1dec.c > >> index c02408548c..1a4339b346 100644 > >> --- a/libavcodec/av1dec.c > >> +++ b/libavcodec/av1dec.c > >> @@ -636,9

Re: [FFmpeg-devel] [PATCH v2 00/13] YUVJ removal

2023-10-20 Thread Vittorio Giovara
On Fri, Oct 20, 2023 at 7:30 AM Niklas Haas wrote: > On Fri, 13 Oct 2023 14:33:11 -0400 Vittorio Giovara < > vittorio.giov...@gmail.com> wrote: > > On Fri, Oct 13, 2023 at 10:27 AM Niklas Haas wrote: > > > > > Changes since v1: > > > > > > - Remove unneeded patch (AVCodecContext.colorspace

Re: [FFmpeg-devel] [PATCH v2 00/13] YUVJ removal

2023-10-20 Thread Vittorio Giovara
On Fri, Oct 20, 2023 at 8:14 AM Ronald S. Bultje wrote: > Hi, > > On Sat, Oct 14, 2023 at 11:16 AM Vittorio Giovara < > vittorio.giov...@gmail.com> wrote: > > > On Sat, Oct 14, 2023 at 9:11 AM Lynne wrote: > > > colorspace doesn't make it impossible to introduce all that is needed. > > > It's a

[FFmpeg-devel] [PATCH] web: move 5.0 to olddownloads

2023-10-20 Thread Michael Niedermayer
our downstream list shows no maintained users of that release https://trac.ffmpeg.org/wiki/Downstreams Signed-off-by: Michael Niedermayer --- src/download| 36 src/olddownload | 36 2 files changed, 36 insertions(+),

Re: [FFmpeg-devel] [PATCH v2 1/3] avutil/dict: add av_dict_pop

2023-10-20 Thread Andreas Rheinhardt
Anton Khirnov: > Quoting Andreas Rheinhardt (2023-10-20 16:00:45) >> Anton Khirnov: >>> Quoting Andreas Rheinhardt (2023-07-03 20:02:25) Marvin Scholz: > I honestly can't think of a sensible API design for this, > if you have any idea feel free to share. > The only way I

Re: [FFmpeg-devel] [PATCH 1/2] avcodec/pthread_frame: Remove ff_thread_release_buffer()

2023-10-20 Thread Andreas Rheinhardt
Anton Khirnov: > Quoting Andreas Rheinhardt (2023-10-20 16:33:06) >> diff --git a/libavcodec/av1dec.c b/libavcodec/av1dec.c >> index c02408548c..1a4339b346 100644 >> --- a/libavcodec/av1dec.c >> +++ b/libavcodec/av1dec.c >> @@ -636,9 +636,9 @@ static int get_pixel_format(AVCodecContext *avctx) >>

Re: [FFmpeg-devel] [PATCH 1/2] avcodec/pthread_frame: Remove ff_thread_release_buffer()

2023-10-20 Thread Anton Khirnov
Quoting Andreas Rheinhardt (2023-10-20 16:33:06) > diff --git a/libavcodec/av1dec.c b/libavcodec/av1dec.c > index c02408548c..1a4339b346 100644 > --- a/libavcodec/av1dec.c > +++ b/libavcodec/av1dec.c > @@ -636,9 +636,9 @@ static int get_pixel_format(AVCodecContext *avctx) > return 0; > } >

Re: [FFmpeg-devel] [PATCH v2 1/3] avutil/dict: add av_dict_pop

2023-10-20 Thread Anton Khirnov
Quoting Andreas Rheinhardt (2023-10-20 16:00:45) > Anton Khirnov: > > Quoting Andreas Rheinhardt (2023-07-03 20:02:25) > >> Marvin Scholz: > >>> I honestly can't think of a sensible API design for this, > >>> if you have any idea feel free to share. > >>> > >> > >> The only way I can think of that

[FFmpeg-devel] [PATCH 2/2] avcodec/av1dec, vaapi_av1: Remove excessive logmessages

2023-10-20 Thread Andreas Rheinhardt
Signed-off-by: Andreas Rheinhardt --- libavcodec/av1dec.c| 10 ++ libavcodec/vaapi_av1.c | 10 ++ 2 files changed, 4 insertions(+), 16 deletions(-) diff --git a/libavcodec/av1dec.c b/libavcodec/av1dec.c index 1a4339b346..6114cb78e6 100644 --- a/libavcodec/av1dec.c +++

[FFmpeg-devel] [PATCH 1/2] avcodec/pthread_frame: Remove ff_thread_release_buffer()

2023-10-20 Thread Andreas Rheinhardt
It is unnecessary since the removal of non-thread-safe callbacks in e0786a8eeb9e7c8feb057e83f284491f0a87e463. Since then, the AVCodecContext has only been used as logcontext. Removing ff_thread_release_buffer() allowed to remove AVCodecContext* parameters from several other functions (not only

Re: [FFmpeg-devel] [PATCH v2 02/13] avcodec: add extended AVCodec color metadata

2023-10-20 Thread Michael Niedermayer
On Fri, Oct 20, 2023 at 03:54:16PM +0200, Anton Khirnov wrote: > Quoting Niklas Haas (2023-10-14 13:46:34) > > > 3. I don't see how the MJPEG encoder behaviour where the valid formats > > > de facto depend upon strictness can be encoded in this way; isn't the > > > aim to get rid of the necessity

Re: [FFmpeg-devel] [PATCH avcodec/amfenc: 10 bit support, v4, 1/3] avcodec/amfenc: Fixes the color information in the output.

2023-10-20 Thread Evgeny Pavlov
On Wed, Oct 18, 2023 at 10:48 PM Mark Thompson wrote: > On 17/10/2023 19:00, Evgeny Pavlov wrote: > > On Mon, Oct 16, 2023 at 11:41 PM Mark Thompson wrote: > > ... > >>> @@ -785,6 +787,41 @@ int ff_amf_receive_packet(AVCodecContext *avctx, > >> AVPacket *avpkt) > >>>return ret; > >>>

Re: [FFmpeg-devel] [PATCH v2 02/13] avcodec: add extended AVCodec color metadata

2023-10-20 Thread Anton Khirnov
Quoting Andreas Rheinhardt (2023-10-13 19:10:33) > Niklas Haas: > > From: Niklas Haas > > > > This is motivated primarily by a desire for YUVJ removal, which will > > require signalling the supported color ranges as part of the codec > > capabilities. But since we're here anyway, we might as

Re: [FFmpeg-devel] [PATCH v2 1/3] avutil/dict: add av_dict_pop

2023-10-20 Thread Andreas Rheinhardt
Anton Khirnov: > Quoting Andreas Rheinhardt (2023-07-03 20:02:25) >> Marvin Scholz: >>> I honestly can't think of a sensible API design for this, >>> if you have any idea feel free to share. >>> >> >> The only way I can think of that allows this is for the user to pass a >> pointer to a (const)

Re: [FFmpeg-devel] [PATCH v2 02/13] avcodec: add extended AVCodec color metadata

2023-10-20 Thread Anton Khirnov
Quoting Niklas Haas (2023-10-14 13:46:34) > > 3. I don't see how the MJPEG encoder behaviour where the valid formats > > de facto depend upon strictness can be encoded in this way; isn't the > > aim to get rid of the necessity of the workaround in ffmpeg cli? > > Note that ffmpeg cli presently

Re: [FFmpeg-devel] trac spam

2023-10-20 Thread Michael Niedermayer
On Thu, Oct 19, 2023 at 09:05:07PM +0200, Michael Koch wrote: > > IIRC each regex give -10 so multiple matches give more. > > So far, the search strings contain only combinations of keywords, for example > "concretevictoria". > False positives are almost impossible. strings that are certain

Re: [FFmpeg-devel] [PATCH v2 00/13] YUVJ removal

2023-10-20 Thread Ronald S. Bultje
Hi, On Sat, Oct 14, 2023 at 11:16 AM Vittorio Giovara < vittorio.giov...@gmail.com> wrote: > On Sat, Oct 14, 2023 at 9:11 AM Lynne wrote: > > colorspace doesn't make it impossible to introduce all that is needed. > > It's a cleaner codebase that we can extend. > > * that only works on a subset

Re: [FFmpeg-devel] [PATCH v2 00/13] YUVJ removal

2023-10-20 Thread Niklas Haas
On Fri, 13 Oct 2023 14:33:11 -0400 Vittorio Giovara wrote: > On Fri, Oct 13, 2023 at 10:27 AM Niklas Haas wrote: > > > Changes since v1: > > > > - Remove unneeded patch (AVCodecContext.colorspace init) > > - Merge auto-range conversion into auto-scale filter > > - Replace vf_zscale by

Re: [FFmpeg-devel] [PATCH] avformat/mov: The iloc test is not redundant

2023-10-20 Thread Anton Khirnov
Quoting Michael Niedermayer (2023-10-19 20:53:05) > On Thu, Oct 19, 2023 at 07:42:30PM +0200, Anton Khirnov wrote: > > Quoting Michael Niedermayer (2023-10-19 18:33:13) > > > On Thu, Oct 19, 2023 at 01:10:18PM +0200, Anton Khirnov wrote: > > > > Quoting Michael Niedermayer (2023-10-15 02:13:23) >

Re: [FFmpeg-devel] [PATCH] libavcodec/amfenc: Add more pixel formats support

2023-10-20 Thread Evgeny Pavlov
On Tue, Jul 18, 2023 at 10:32 AM Evgeny Pavlov wrote: > This commit adds BGRA, RGBA and ARGB pixel formats for AMF encoders > > Signed-off-by: Evgeny Pavlov > --- > libavcodec/amfenc.c | 3 +++ > 1 file changed, 3 insertions(+) > > diff --git a/libavcodec/amfenc.c b/libavcodec/amfenc.c > index

Re: [FFmpeg-devel] [PATCH v2 1/3] avutil/dict: add av_dict_pop

2023-10-20 Thread Anton Khirnov
Quoting Andreas Rheinhardt (2023-07-03 20:02:25) > Marvin Scholz: > > I honestly can't think of a sensible API design for this, > > if you have any idea feel free to share. > > > > The only way I can think of that allows this is for the user to pass a > pointer to a (const) AVDictionaryEntry,