[FFmpeg-devel] [PATCH] doc: Merge doc/dev_community/* and Code of Conduct into a seperate file

2023-02-08 Thread Thilo Borgmann
Remove doc/dev_communit markup files completely as they are at the wrong place. Create a new community page, merging all of doc/dev_community and subsection Code of Conduct into a common place. The corresponding patch to ffmpeg-web puts the Organisation & Code of Conduct into a seperate community

Re: [FFmpeg-devel] Hardware purchase request Apple M2

2023-02-07 Thread Thilo Borgmann
Am 07.02.23 um 21:19 schrieb Devin Heitmueller: On Mon, Feb 6, 2023 at 6:24 AM Thilo Borgmann wrote: Martin pointed out we don't need an M2 for FATE as the instruction sets are equal on both machines. It may be worth noting that while the M1 and M2 have the same instruction set, there

Re: [FFmpeg-devel] [PATCH 2/2] fftools/ffmpeg: add an option for writing pre-muxing stats

2023-02-07 Thread Thilo Borgmann
Am 07.02.23 um 10:41 schrieb Anton Khirnov: Also, I'm wondering if we shouldn't rename the options to -stats_{mux,enc_pre,enc_post}[_fmt] for consistency. They were added just recently, so it should be fine if done before the release. Opinions? I'd like that. Can't comment on the code itself

Re: [FFmpeg-devel] Hardware purchase request Apple M2

2023-02-06 Thread Thilo Borgmann
Hi, the M2 chip recently made it from the laptops into the mini as well. It's the generation of the Chip, I think we should buy one and have that running FATE as well. Kieran agreed on hosting this one like he does for the existing M1 machines. There are currently two M1 machines running at Ki

[FFmpeg-devel] Hardware purchase request Apple M2

2023-02-05 Thread Thilo Borgmann
Hi, the M2 chip recently made it from the laptops into the mini as well. It's the generation of the Chip, I think we should buy one and have that running FATE as well. Kieran agreed on hosting this one like he does for the existing M1 machines. There are currently two M1 machines running at Kie

[FFmpeg-devel] Code of Conduct & Mailing list rules & Community Committee

2023-02-05 Thread Thilo Borgmann
Hi folks, unfortunately the latest escalation on the user mailing list [1][2][3][4] let's me think that it is necessary to emphasize some things and update the mind of many of us. A) We have imposed upon us our Code of Conduct [5]. We ask everyone to (read and) follow that in our Mailing list

[FFmpeg-devel] [CLT2023] FFmpeg at Chemnitzer Linux-Tage

2023-02-02 Thread Thilo Borgmann
ntly registered for the CLT team, more attendees appreciated: Carl Eugen Hoyos Thomas Volkert Alexander Strasser Thilo Borgmann We would like to encourage everyone visiting the CLT to bring us sample files and/or command lines that show suspicious or buggy behavior - this will be your change to get

Re: [FFmpeg-devel] Let us review and collebrate on vvc native decoder.

2023-01-26 Thread Thilo Borgmann
Am 14.01.23 um 14:32 schrieb Nuo Mi: On Sat, Jan 14, 2023 at 9:13 PM Nuo Mi wrote: Hi Experts: I am happy to send out the first draft of vvc decoder. It's not ready for upstream yet, but it's a good base ground for review and future improvement. It has the following features: * C only + Fast.

Re: [FFmpeg-devel] GSoC 2023

2023-01-23 Thread Thilo Borgmann
Hi, sorry for delay, some thoughts: Description: VVC is the successor to the well-known codec HEVC. The whole video industry has high anticipation for VVC. The goal of the ffvvc project is to provide a VVC decoder for FFmpeg. We invite you to join us in> improving ffvvc. Depending on your bac

Re: [FFmpeg-devel] GSoC 2023

2023-01-15 Thread Thilo Borgmann
Am 15.01.23 um 15:28 schrieb Nuo Mi: On Sun, Jan 15, 2023 at 9:27 PM Ronald S. Bultje wrote: Hi, On Sun, Jan 15, 2023 at 4:20 AM Nuo Mi wrote: Is it finalized? Any chance to add some items in https://github.com/ffvvc/FFmpeg/issues to the GSOC 2023? It all depends on the student. And ba

Re: [FFmpeg-devel] GSoC 2023

2022-12-18 Thread Thilo Borgmann
both of you wouldn't be aware that there shall be no top-posting here... -Thilo On Thu, 15 Dec 2022, at 15:42, Thilo Borgmann wrote: Hi, as we figured during the last dev meeting on December 2nd we want to apply again for GSoC 2023. I told mentors often enough already that it'

[FFmpeg-devel] GSoC 2023

2022-12-15 Thread Thilo Borgmann
Hi, as we figured during the last dev meeting on December 2nd we want to apply again for GSoC 2023. I told mentors often enough already that it's a vital point in our applications to have a properly filled results page from the last GSoC we did. Last time no mentor cared about this - we can be

Re: [FFmpeg-devel] [REFUND-REQUEST] Travel to Developer Meeting

2022-12-14 Thread Thilo Borgmann
Am 14.12.22 um 15:06 schrieb Derek Buitenhuis: Hello, I travelled to Barcelona for the developer meeting. I am only requesting for my flight, as I chose to stay in non-VideoLAN hotel. Flight: £111.98 - receipt available, of course. As I am not super active currently, understandable if I canno

Re: [FFmpeg-devel] [PATCH] lavfi/vf_dnn_processing.c: Fix missing AV_PIX_FMT_GRAY8

2022-11-15 Thread Thilo Borgmann
Am 10.11.22 um 12:22 schrieb Thilo Borgmann: Am 02.11.22 um 16:07 schrieb Thilo Borgmann: From: Cosmin Stejerean Has been removed by mistake in 2003e32f62d94ba75b59d70632c9f2862b383591, readd it to the switch cases. Signed-off-by: Thilo Borgmann ---   libavfilter/vf_dnn_processing.c | 1

Re: [FFmpeg-devel] [PATCH] lavfi/vf_dnn_processing.c: Fix missing AV_PIX_FMT_GRAY8

2022-11-10 Thread Thilo Borgmann
Am 02.11.22 um 16:07 schrieb Thilo Borgmann: From: Cosmin Stejerean Has been removed by mistake in 2003e32f62d94ba75b59d70632c9f2862b383591, readd it to the switch cases. Signed-off-by: Thilo Borgmann --- libavfilter/vf_dnn_processing.c | 1 + 1 file changed, 1 insertion(+) Ping

[FFmpeg-devel] [PATCH] lavfi/vf_dnn_processing.c: Fix missing AV_PIX_FMT_GRAY8

2022-11-02 Thread Thilo Borgmann
From: Cosmin Stejerean Has been removed by mistake in 2003e32f62d94ba75b59d70632c9f2862b383591, readd it to the switch cases. Signed-off-by: Thilo Borgmann --- libavfilter/vf_dnn_processing.c | 1 + 1 file changed, 1 insertion(+) diff --git a/libavfilter/vf_dnn_processing.c b/libavfilter

Re: [FFmpeg-devel] [PATCH v7 0/2] ffmpeg: Add display_{rotation, hflip, vflip} options

2022-10-19 Thread Thilo Borgmann
Am 19.10.22 um 12:04 schrieb Thilo Borgmann: Am 15.10.22 um 12:25 schrieb Thilo Borgmann: Hi, this is an almost identical rebased version of Jan's patchset discussed in [1], now v7... Should fix #8329 and #6370. Fixed with two more nits, OK'd by Anton via Jitsi. Pushed with..

Re: [FFmpeg-devel] [PATCH v7 0/2] ffmpeg: Add display_{rotation, hflip, vflip} options

2022-10-19 Thread Thilo Borgmann
Am 15.10.22 um 12:25 schrieb Thilo Borgmann: Hi, this is an almost identical rebased version of Jan's patchset discussed in [1], now v7... Should fix #8329 and #6370. Fixed with two more nits, OK'd by Anton via Jitsi. Thanks, Thilo

Re: [FFmpeg-devel] [PATCH v6 1/2] ffmpeg: Add display_{rotation, hflip, vflip} options

2022-10-15 Thread Thilo Borgmann
Am 04.10.22 um 10:44 schrieb Anton Khirnov: Quoting Thilo Borgmann (2022-09-26 07:57:52) From: Jan Ekström This enables overriding the rotation as well as horizontal/vertical flip state of a specific video stream on the input side. Additionally, switch the singular test that was utilizing

[FFmpeg-devel] [PATCH v7 2/2] ffmpeg: Deprecate display rotation override with a metadata key

2022-10-15 Thread Thilo Borgmann
From: Jan Ekström Now that we have proper options for defining display matrix overrides, this should no longer be required. fftools does not have its own versioning, so for now the define is just set to 1 and disables the functionality if set to zero. --- fftools/ffmpeg.c | 2 ++ fftools/f

[FFmpeg-devel] [PATCH v7 1/2] ffmpeg: Add display_{rotation, hflip, vflip} options

2022-10-15 Thread Thilo Borgmann
From: Jan Ekström This enables overriding the rotation as well as horizontal/vertical flip state of a specific video stream on the input side. Additionally, switch the singular test that was utilizing the rotation metadata to instead override the input display rotation, thus leading to the same

[FFmpeg-devel] [PATCH v7 0/2] ffmpeg: Add display_{rotation, hflip, vflip} options

2022-10-15 Thread Thilo Borgmann
Hi, this is an almost identical rebased version of Jan's patchset discussed in [1], now v7... Should fix #8329 and #6370. Thanks, Thilo [1] https://lists.ffmpeg.org/pipermail/ffmpeg-devel/2022-May/296553.html Jan Ekström (2): ffmpeg: Add display_{rotation, hflip, vflip} options ffmpeg: De

[FFmpeg-devel] [PATCH v6 2/2] ffmpeg: Deprecate display rotation override with a metadata key

2022-09-25 Thread Thilo Borgmann
From: Jan Ekström Now that we have proper options for defining display matrix overrides, this should no longer be required. fftools does not have its own versioning, so for now the define is just set to 1 and disables the functionality if set to zero. --- fftools/ffmpeg.c | 2 ++ fftools/f

[FFmpeg-devel] [PATCH v6 1/2] ffmpeg: Add display_{rotation, hflip, vflip} options

2022-09-25 Thread Thilo Borgmann
From: Jan Ekström This enables overriding the rotation as well as horizontal/vertical flip state of a specific video stream on the input side. Additionally, switch the singular test that was utilizing the rotation metadata to instead override the input display rotation, thus leading to the same

[FFmpeg-devel] [PATCH v6 0/2] ffmpeg: Add display_{rotation, hflip, vflip} options

2022-09-25 Thread Thilo Borgmann
Hi, this is an almost identical rebased version of Jan's patchset discussed in [1], now v6... Should fix #8329 and #6370. Thanks, Thilo [1] https://lists.ffmpeg.org/pipermail/ffmpeg-devel/2022-May/296553.html Jan Ekström (2): ffmpeg: Add display_{rotation, hflip, vflip} options ffmpeg: De

[FFmpeg-devel] [PATCH v5 3/3] ffmpeg: Deprecate display rotation override with a metadata key

2022-09-25 Thread Thilo Borgmann
From: Jan Ekström Now that we have proper options for defining display matrix overrides, this should no longer be required. fftools does not have its own versioning, so for now the define is just set to 1 and disables the functionality if set to zero. --- fftools/ffmpeg.c | 2 ++ fftools/f

[FFmpeg-devel] [PATCH v5 2/3] ffmpeg: Add display_{rotation, hflip, vflip, scale} options

2022-09-25 Thread Thilo Borgmann
From: Jan Ekström This enables overriding the scale, rotation as well as horizontal/vertical flip state of a specific video stream on the input side. Additionally, switch the singular test that was utilizing the rotation metadata to instead override the input display rotation, thus leading to th

[FFmpeg-devel] [PATCH v5 1/3] lavu/display: Add scaling functions to the display matrix

2022-09-25 Thread Thilo Borgmann
--- doc/APIchanges | 3 +++ libavutil/display.c | 21 + libavutil/display.h | 15 +++ libavutil/version.h | 4 ++-- 4 files changed, 41 insertions(+), 2 deletions(-) diff --git a/doc/APIchanges b/doc/APIchanges index 729f56be7b..42e3391b72 100644 --- a/doc/A

[FFmpeg-devel] [PATCH v5 0/3] ffmpeg: Add display_{rotation, hflip, vflip, scale} options

2022-09-25 Thread Thilo Borgmann
Hi, this is an updated and cleaned-up version of Jan's patchset discussed in [1], now v5... Should fix #8329 and #6370. Thanks, Thilo Jan Ekström (2): ffmpeg: Add display_{rotation, hflip, vflip, scale} options ffmpeg: Deprecate display rotation override with a metadata key

Re: [FFmpeg-devel] [PATCH] lavc/videotoolboxenc: Fix crash by uninitialized value

2022-09-21 Thread Thilo Borgmann
Am 15.09.22 um 20:42 schrieb Thilo Borgmann: If create_cv_pixel_buffer() fails, pixel_buffer_info might get into CFRelease() containing an arbitrary value. --- libavcodec/videotoolboxenc.c | 4 +--- 1 file changed, 1 insertion(+), 3 deletions(-) diff --git a/libavcodec/videotoolboxenc.c b

[FFmpeg-devel] [PATCH v4 5/6] ffmpeg: Add display_matrix option

2022-09-19 Thread Thilo Borgmann
From: Jan Ekström This enables overriding the rotation as well as horizontal/vertical flip state of a specific video stream on the input side. Additionally, switch the singular test that was utilizing the rotation metadata to instead override the input display rotation, thus leading to the same

[FFmpeg-devel] [PATCH v4 4/6] lavu/display: Add horizontal and vertical scaling to the display matrix

2022-09-19 Thread Thilo Borgmann
--- doc/APIchanges | 3 +++ libavutil/display.c | 35 +++ libavutil/display.h | 26 ++ libavutil/version.h | 2 +- 4 files changed, 65 insertions(+), 1 deletion(-) diff --git a/doc/APIchanges b/doc/APIchanges index e4b2fc8799..9dc393d

[FFmpeg-devel] [PATCH v4 6/6] ffmpeg: Deprecate display rotation override with a metadata key

2022-09-19 Thread Thilo Borgmann
From: Jan Ekström Now that we have proper options for defining display matrix overrides, this should no longer be required. fftools does not have its own versioning, so for now the define is just set to 1 and disables the functionality if set to zero. --- fftools/ffmpeg.c | 2 ++ fftools/f

[FFmpeg-devel] [PATCH v4 3/6] fftools: Add support for dictionary options

2022-09-19 Thread Thilo Borgmann
From: Jan Ekström --- fftools/cmdutils.c | 17 + fftools/cmdutils.h | 2 ++ fftools/ffmpeg_opt.c | 29 ++--- 3 files changed, 45 insertions(+), 3 deletions(-) diff --git a/fftools/cmdutils.c b/fftools/cmdutils.c index b6dd73902b..f918d83854 100644 --

[FFmpeg-devel] [PATCH v4 2/6] fftools/cmdutils: Print arguments of options

2022-09-19 Thread Thilo Borgmann
--- fftools/cmdutils.c | 5 + fftools/cmdutils.h | 1 + 2 files changed, 6 insertions(+) diff --git a/fftools/cmdutils.c b/fftools/cmdutils.c index f911c52be2..b6dd73902b 100644 --- a/fftools/cmdutils.c +++ b/fftools/cmdutils.c @@ -162,6 +162,11 @@ void show_help_options(const OptionDef *opti

[FFmpeg-devel] [PATCH v4 1/6] lavu/opt: Allow options to be arguments of other options

2022-09-19 Thread Thilo Borgmann
Adds AV_OPT_FLAG_ARGUMENT for options being used as arguments for other options. Adds av_arg_show() to allow printing of such arguments in help output. --- doc/APIchanges | 4 libavutil/opt.c | 14 +- libavutil/opt.h | 8 libavutil/version.h | 4 ++-- 4 fi

[FFmpeg-devel] [PATCH v4 0/6] Add display_matrix option

2022-09-19 Thread Thilo Borgmann
022-August/300076.html [3] https://lists.ffmpeg.org/pipermail/ffmpeg-devel/2022-August/300122.html Jan Ekström (3): fftools: Add support for dictionary options ffmpeg: Add display_matrix option ffmpeg: Deprecate display rotation override with a metadata key Thilo Borgmann (3): lavu/opt: All

Re: [FFmpeg-devel] [PATCH 1/2] lavc/videotoolbox: do not pass AVCodecContext to decoder output callback

2022-09-15 Thread Thilo Borgmann
Am 15.09.22 um 15:56 schrieb Anton Khirnov: The opaque parameter for the callback is set in videotoolbox_start(), called when the hwaccel is initialized. When frame threading is used, avctx will be the context corresponding to the frame thread currently doing the decoding. Using this same codec c

[FFmpeg-devel] [PATCH] lavc/videotoolboxenc: Fix crash by uninitialized value

2022-09-15 Thread Thilo Borgmann
If create_cv_pixel_buffer() fails, pixel_buffer_info might get into CFRelease() containing an arbitrary value. --- libavcodec/videotoolboxenc.c | 4 +--- 1 file changed, 1 insertion(+), 3 deletions(-) diff --git a/libavcodec/videotoolboxenc.c b/libavcodec/videotoolboxenc.c index bb3065d1d5..dc9e

Re: [FFmpeg-devel] [PATCH v2 2/4] ffmpeg: Add display_matrix option

2022-09-07 Thread Thilo Borgmann
Am 22.08.22 um 14:30 schrieb Nicolas George: Thilo Borgman (12022-08-20): My two cents about it that the a=b:c=d syntax from AVDict is at least known and used in filters already. The function style a(b,c,d) thing from SVG would be completely new. Instead of the AVDict overhead, it adds a simplis

Re: [FFmpeg-devel] [PATCH v2 2/4] ffmpeg: Add display_matrix option

2022-08-20 Thread Thilo Borgmann
Am 20.08.22 um 15:39 schrieb Nicolas George: Thilo Borgman (12022-08-20): suggestion with a parser for SVG-style (new syntax) is not backup up by s.o. else. I feel it was somewhat drowned by the rest of the discussion. Do YOU like it? My two cents about it that the a=b:c=d syntax from AVDic

Re: [FFmpeg-devel] [PATCH v2 2/4] ffmpeg: Add display_matrix option

2022-08-20 Thread Thilo Borgmann
On 18 Aug 2022, at 12:58, Gyan Doshi wrote: On 2022-08-17 05:55 pm, Anton Khirnov wrote: Quoting Gyan Doshi (2022-08-17 12:53:11) On 2022-08-17 02:35 pm, Anton Khirnov wrote: Quoting Gyan Doshi (2022-08-17 10:50:43) On 2022-08-17 01:48 pm, Anton Khirnov wrote: Quoting Thilo Borgmann

Re: [FFmpeg-devel] [PATCH v2 2/4] ffmpeg: Add display_matrix option

2022-08-16 Thread Thilo Borgmann
Am 16.08.22 um 16:10 schrieb Anton Khirnov: Quoting Thilo Borgmann (2022-08-15 22:02:09) $subject -Thilo From fe2ff114cb004f897c7774753d9cf28298eba82d Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?Jan=20Ekstr=C3=B6m?= Date: Mon, 15 Aug 2022 21:09:27 +0200 Subject: [PATCH v2 2/4] ffmpeg: Add

Re: [FFmpeg-devel] [PATCH 5/5] ffmpeg: Add {h, v}scale argument to display_matrix option to allow for scaling via the display matrix

2022-08-16 Thread Thilo Borgmann
Am 16.08.22 um 15:30 schrieb Thilo Borgmann: --- doc/ffmpeg.texi | 4 fftools/ffmpeg_filter.c | 15 +++ fftools/ffmpeg_opt.c| 10 ++ libavutil/display.c | 21 + libavutil/display.h | 28 5 files

[FFmpeg-devel] [PATCH 5/5] ffmpeg: Add {h, v}scale argument to display_matrix option to allow for scaling via the display matrix

2022-08-16 Thread Thilo Borgmann
--- doc/ffmpeg.texi | 4 fftools/ffmpeg_filter.c | 15 +++ fftools/ffmpeg_opt.c| 10 ++ libavutil/display.c | 21 + libavutil/display.h | 28 5 files changed, 78 insertions(+) diff --git a/doc/ffmpeg.te

[FFmpeg-devel] [PATCH 4/5] ffmpeg: Allow printing of option arguments in help output

2022-08-16 Thread Thilo Borgmann
--- fftools/cmdutils.c | 5 + libavutil/opt.c| 14 +- libavutil/opt.h| 8 3 files changed, 26 insertions(+), 1 deletion(-) diff --git a/fftools/cmdutils.c b/fftools/cmdutils.c index 22ba654bb0..dae018f83a 100644 --- a/fftools/cmdutils.c +++ b/fftools/cmdutils.c

[FFmpeg-devel] [PATCH 3/5] ffmpeg: Deprecate display rotation override with a metadata key

2022-08-16 Thread Thilo Borgmann
From: Jan Ekström Now that we have proper options for defining display matrix overrides, this should no longer be required. fftools does not have its own versioning, so for now the define is just set to 1 and disables the functionality if set to zero. --- fftools/ffmpeg.c | 2 ++ fftools/f

[FFmpeg-devel] [PATCH 2/5] ffmpeg: Add display_matrix option

2022-08-16 Thread Thilo Borgmann
From: Jan Ekström This enables overriding the rotation as well as horizontal/vertical flip state of a specific video stream on the input side. Additionally, switch the singular test that was utilizing the rotation metadata to instead override the input display rotation, thus leading to the same

[FFmpeg-devel] [PATCH 1/5] fftools: Add support for dictionary options

2022-08-16 Thread Thilo Borgmann
From: Jan Ekström --- fftools/cmdutils.c | 18 ++ fftools/cmdutils.h | 2 ++ fftools/ffmpeg_opt.c | 11 +-- 3 files changed, 29 insertions(+), 2 deletions(-) diff --git a/fftools/cmdutils.c b/fftools/cmdutils.c index 18e768b386..22ba654bb0 100644 --- a/fftools/cmdut

[FFmpeg-devel] [PATCH v2 4/4] ffmpeg: Allow printing of option arguments in help output

2022-08-15 Thread Thilo Borgmann
$subject -ThiloFrom 33abe03ac137bd1e4bf4af90731ec177d34298a8 Mon Sep 17 00:00:00 2001 From: Thilo Borgmann Date: Mon, 15 Aug 2022 21:00:18 +0200 Subject: [PATCH v2 4/4] ffmpeg: Allow printing of option arguments in help output --- fftools/cmdutils.c | 5 + libavutil/opt.c| 14

[FFmpeg-devel] [PATCH v2 3/4] ffmpeg: Deprecate display rotation override with a metadata key

2022-08-15 Thread Thilo Borgmann
$subject -ThiloFrom 2c556c126b77b7bc90749096f858daa6124cf097 Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?Jan=20Ekstr=C3=B6m?= Date: Mon, 15 Aug 2022 20:58:05 +0200 Subject: [PATCH v2 3/4] ffmpeg: Deprecate display rotation override with a metadata key Now that we have proper options for defining

[FFmpeg-devel] [PATCH v2 2/4] ffmpeg: Add display_matrix option

2022-08-15 Thread Thilo Borgmann
$subject -ThiloFrom fe2ff114cb004f897c7774753d9cf28298eba82d Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?Jan=20Ekstr=C3=B6m?= Date: Mon, 15 Aug 2022 21:09:27 +0200 Subject: [PATCH v2 2/4] ffmpeg: Add display_matrix option This enables overriding the rotation as well as horizontal/vertical flip sta

[FFmpeg-devel] [PATCH v2 1/4] fftools: Add support for dictionary options

2022-08-15 Thread Thilo Borgmann
Hi, this is an updated and cleaned-up version of Jan's patchset discussed in [1], therefore v2... Comments from the previous thread are in and left-overs from the several options version removed. I'd especially appreciate any comments on 2/4, ffmpeg_opt.c:112ff which is pretty ugly as-is. Sho

Re: [FFmpeg-devel] [PATCH] Document the addresses of the committees (was: Contact of the Technical Committee)

2022-08-02 Thread Thilo Borgmann
On 2 Aug 2022, at 20:16, Thilo Borgmann wrote: On 2 Aug 2022, at 19:34, Nicolas George wrote: Thilo Borgman (12022-08-02): Of course, thought it would already be there together with CC. There’s a typo in the mail address, otherwise LGTM. Indeed, locally fixed: +The TC can be contacted

Re: [FFmpeg-devel] [PATCH] Document the addresses of the committees (was: Contact of the Technical Committee)

2022-08-02 Thread Thilo Borgmann
On 2 Aug 2022, at 19:34, Nicolas George wrote: > Thilo Borgman (12022-08-02): >> Of course, thought it would already be there together with CC. >> >> There’s a typo in the mail address, otherwise LGTM. > > Indeed, locally fixed: > > +The TC can be contacted at . > +The CC can be contacted at .

Re: [FFmpeg-devel] [PATCH] avutil/opt: Combine multiple av_log statements

2022-08-02 Thread Thilo Borgmann
On 2 Aug 2022, at 19:28, Andreas Rheinhardt wrote: > Signed-off-by: Andreas Rheinhardt > --- > libavutil/opt.c | 23 --- > 1 file changed, 12 insertions(+), 11 deletions(-) > > diff --git a/libavutil/opt.c b/libavutil/opt.c > index 8ffb10449b..da3a65c041 100644 > --- a/libavu

Re: [FFmpeg-devel] [PATCH 1/2] ffmpeg: add display_{rotation, hflip, vflip} options

2022-08-02 Thread Thilo Borgmann
On 2 Aug 2022, at 19:10, Andreas Rheinhardt wrote: Thilo Borgmann: Hi, On 2022-05-18 11:34 pm, Jan Ekström wrote: This enables overriding the rotation as well as horizontal/vertical flip state of a specific video stream on either the input or output side. Since rotation, flip and

Re: [FFmpeg-devel] [PATCH] Document the addresses of the committees (was: Contact of the Technical Committee)

2022-08-02 Thread Thilo Borgmann
On 2 Aug 2022, at 19:10, Nicolas George wrote: > Jean-Baptiste Kempf (12022-07-24): >> Have you tried tc@ffmpeg.o.. ? > > Thanks. > > Would this patch be a good idea? Of course, thought it would already be there together with CC. There’s a typo in the mail address, otherwise LGTM. -Thilo

Re: [FFmpeg-devel] [PATCH 1/2] ffmpeg: add display_{rotation, hflip, vflip} options

2022-08-02 Thread Thilo Borgmann
Am 02.08.22 um 18:48 schrieb Thilo Borgmann: Hi, On 2022-05-18 11:34 pm, Jan Ekström wrote: This enables overriding the rotation as well as horizontal/vertical flip state of a specific video stream on either the input or output side. Since rotation, flip and scale are stored in a single

Re: [FFmpeg-devel] [PATCH 1/2] ffmpeg: add display_{rotation, hflip, vflip} options

2022-08-02 Thread Thilo Borgmann
/ffmpeg_c_display_matrix_with_dicts From 4fae59de38c93a4cdd079535cc3631f9ccadced1 Mon Sep 17 00:00:00 2001 From: Thilo Borgmann Date: Tue, 2 Aug 2022 18:39:18 +0200 Subject: [PATCH] ffmpeg: Allow printing of option arguments in help output --- fftools/cmdutils.c | 5 ++ fftools/cmdutils

Re: [FFmpeg-devel] [PATCH v2 1/2] lavfi/edge_common: Add 16bit versions of gaussian_blur and sobel

2022-07-30 Thread Thilo Borgmann
Am 29.07.22 um 15:11 schrieb Thilo Borgmann: Hi, 1/2 adds 16 bit versions of ff_gaussian_blur and ff_sobel. 2/2 adds new mode to cropdetect. v3 does it the template way for 1/2 as requested on IRC. v4 fixed bug in gaussian_blur. Otherwise identical. v5 fixes minor things mentioned on IRC

Re: [FFmpeg-devel] [PATCH v2 1/2] lavfi/edge_common: Add 16bit versions of gaussian_blur and sobel

2022-07-29 Thread Thilo Borgmann
Hi, 1/2 adds 16 bit versions of ff_gaussian_blur and ff_sobel. 2/2 adds new mode to cropdetect. >>> >>> v3 does it the template way for 1/2 as requested on IRC. >> >> v4 fixed bug in gaussian_blur. Otherwise identical. > > v5 fixes minor things mentioned on IRC and another bug found on t

Re: [FFmpeg-devel] [PATCH v2 2/2] lavfi/cropdetect: Add new mode to detect crop-area based on motion vectors and edges

2022-07-18 Thread Thilo Borgmann
Am 17.07.22 um 09:54 schrieb Thilo Borgmann: Am 16.07.22 um 23:09 schrieb Thilo Borgmann: Am 11.07.22 um 10:54 schrieb Thilo Borgmann: $subject v3. v4. v5 now with fixed docs. -ThiloFrom c5962d580dd103d0b9cc08724a6863e62383da7b Mon Sep 17 00:00:00 2001 From: Thilo Borgmann Date: Mon

Re: [FFmpeg-devel] [PATCH v2 1/2] lavfi/edge_common: Add 16bit versions of gaussian_blur and sobel

2022-07-18 Thread Thilo Borgmann
Am 17.07.22 um 09:54 schrieb Thilo Borgmann: Am 16.07.22 um 23:07 schrieb Thilo Borgmann: Hi, 1/2 adds 16 bit versions of ff_gaussian_blur and ff_sobel. 2/2 adds new mode to cropdetect. v3 does it the template way for 1/2 as requested on IRC. v4 fixed bug in gaussian_blur. Otherwise

Re: [FFmpeg-devel] [PATCH 1/2] ffmpeg: add display_{rotation, hflip, vflip} options

2022-07-17 Thread Thilo Borgmann
Hi, Am 26.05.22 um 13:31 schrieb Jan Ekström: On Thu, May 19, 2022 at 9:27 AM Jan Ekström wrote: On Thu, May 19, 2022, 07:21 Gyan Doshi wrote: On 2022-05-18 11:34 pm, Jan Ekström wrote: This enables overriding the rotation as well as horizontal/vertical flip state of a specific video st

Re: [FFmpeg-devel] [PATCH v2 2/2] lavfi/cropdetect: Add new mode to detect crop-area based on motion vectors and edges

2022-07-17 Thread Thilo Borgmann
Am 16.07.22 um 23:09 schrieb Thilo Borgmann: Am 11.07.22 um 10:54 schrieb Thilo Borgmann: $subject v3. v4. -ThiloFrom 9933d7d69781e1922b4a2ddc22777fdef588dbb2 Mon Sep 17 00:00:00 2001 From: Thilo Borgmann Date: Sun, 17 Jul 2022 09:52:01 +0200 Subject: [PATCH v4 2/2] lavfi/cropdetect: Add

Re: [FFmpeg-devel] [PATCH v2 1/2] lavfi/edge_common: Add 16bit versions of gaussian_blur and sobel

2022-07-17 Thread Thilo Borgmann
Am 16.07.22 um 23:07 schrieb Thilo Borgmann: Hi, 1/2 adds 16 bit versions of ff_gaussian_blur and ff_sobel. 2/2 adds new mode to cropdetect. v3 does it the template way for 1/2 as requested on IRC. v4 fixed bug in gaussian_blur. Otherwise identical. -ThiloFrom

Re: [FFmpeg-devel] [PATCH v2 2/2] lavfi/cropdetect: Add new mode to detect crop-area based on motion vectors and edges

2022-07-16 Thread Thilo Borgmann
Am 11.07.22 um 10:54 schrieb Thilo Borgmann: $subject v3. -ThiloFrom 763c169d82395ec3fd59fa66ebc78c676f0f186d Mon Sep 17 00:00:00 2001 From: Thilo Borgmann Date: Sat, 16 Jul 2022 23:01:14 +0200 Subject: [PATCH v3 2/2] lavfi/cropdetect: Add new mode to detect crop-area based on motion

Re: [FFmpeg-devel] [PATCH v2 1/2] lavfi/edge_common: Add 16bit versions of gaussian_blur and sobel

2022-07-16 Thread Thilo Borgmann
Hi, 1/2 adds 16 bit versions of ff_gaussian_blur and ff_sobel. 2/2 adds new mode to cropdetect. v3 does it the template way for 1/2 as requested on IRC. -ThiloFrom 5453c0e27cd2c54931b012d663178a7c0b5a9f5f Mon Sep 17 00:00:00 2001 From: Thilo Borgmann Date: Sat, 16 Jul 2022 22:59:57 +0200

[FFmpeg-devel] [PATCH v2 2/2] lavfi/cropdetect: Add new mode to detect crop-area based on motion vectors and edges

2022-07-11 Thread Thilo Borgmann
$subject -ThiloFrom ccc2b5ab29c4ca00c0a59af318fc865d37832377 Mon Sep 17 00:00:00 2001 From: Thilo Borgmann Date: Mon, 11 Jul 2022 10:48:53 +0200 Subject: [PATCH v2 2/2] lavfi/cropdetect: Add new mode to detect crop-area based on motion vectors and edges This filter allows crop detection even

[FFmpeg-devel] [PATCH v2 1/2] lavfi/edge_common: Add 16bit versions of gaussian_blur and sobel

2022-07-11 Thread Thilo Borgmann
Hi, 1/2 adds 16 bit versions of ff_gaussian_blur and ff_sobel. 2/2 adds new mode to cropdetect. Thanks, ThiloFrom fc8c179e2de4dee3d32d2e02684f3e3215af63c6 Mon Sep 17 00:00:00 2001 From: Thilo Borgmann Date: Sun, 10 Jul 2022 12:40:27 +0200 Subject: [PATCH v2 1/2] lavfi/edge_common: Add 16bit

Re: [FFmpeg-devel] [PATCH 1/3] avcodec/alsdec: No channels cannot be accessed

2022-07-02 Thread Thilo Borgmann
Am 02.07.22 um 14:22 schrieb Michael Niedermayer: Fixes: out of array access Fixes: 48145/clusterfuzz-testcase-minimized-ffmpeg_AV_CODEC_ID_ALS_fuzzer-5054524173189120 Found-by: continuous fuzzing process https://github.com/google/oss-fuzz/tree/master/projects/ffmpeg Signed-off-by: Michael Nie

[FFmpeg-devel] [PATCH] lavfi: Add cropdetect_video filter

2022-07-02 Thread Thilo Borgmann
/vf_cropdetect.c - if we'd like to add this as LGPL for some reason it might go into its own file without sharing (useful) logic with the existing cropdetect. Thanks, ThiloFrom 9050d15c2f1bcb3b2a628c8b6f04ea3a5f7e69d1 Mon Sep 17 00:00:00 2001 From: Thilo Borgmann Date: Sat, 2 Jul 2022 11:42:47

Re: [FFmpeg-devel] [PATCH] lavc/dovi_rpu: Fix UB for possible left shift of negative values

2022-06-21 Thread Thilo Borgmann
Am 20.06.22 um 11:06 schrieb Thilo Borgmann: Am 16.06.22 um 13:19 schrieb Thilo Borgmann: Am 16.06.22 um 12:40 schrieb Andreas Rheinhardt: Thilo Borgmann: Am 16.06.22 um 12:13 schrieb Andreas Rheinhardt: Thilo Borgmann: diff --git a/libavcodec/dovi_rpu.c b/libavcodec/dovi_rpu.c index

Re: [FFmpeg-devel] [PATCH] lavc/libx264.c: Fix possible UB by NULL pointer LHS

2022-06-20 Thread Thilo Borgmann
Am 16.06.22 um 22:01 schrieb Michael Niedermayer: On Thu, Jun 16, 2022 at 05:58:09PM +0200, Thilo Borgmann wrote: Hi, the LHS pointer might be NULL so that += would be UB. Thanks, Thilo libx264.c |4 +++- 1 file changed, 3 insertions(+), 1 deletion

Re: [FFmpeg-devel] [PATCH] tests/checkasm/sw_scale: Fix alignment for movdqa

2022-06-20 Thread Thilo Borgmann
Am 18.06.22 um 23:50 schrieb Martin Storsjö: On Thu, 16 Jun 2022, Thilo Borgmann wrote: Hi, movdqa in ff_yuv2yuvX_sse3() expects a 16-byte alignment according to its documentation causing segfaults in fate-checkasm-sw_scale. LGTM, thanks! Pushed, Thanks! -Thilo

Re: [FFmpeg-devel] [PATCH] lavc/dovi_rpu: Fix UB for possible left shift of negative values

2022-06-20 Thread Thilo Borgmann
Am 16.06.22 um 13:19 schrieb Thilo Borgmann: Am 16.06.22 um 12:40 schrieb Andreas Rheinhardt: Thilo Borgmann: Am 16.06.22 um 12:13 schrieb Andreas Rheinhardt: Thilo Borgmann: diff --git a/libavcodec/dovi_rpu.c b/libavcodec/dovi_rpu.c index a87562c8a3..833ce9e705 100644 --- a/libavcodec

[FFmpeg-devel] [PATCH] lavc/libx264.c: Fix possible UB by NULL pointer LHS

2022-06-16 Thread Thilo Borgmann
Hi, the LHS pointer might be NULL so that += would be UB. Thanks, ThiloFrom cfb7ce8092c34436fae3120645aa96fe082af4ea Mon Sep 17 00:00:00 2001 From: Michael Goulet Date: Thu, 16 Jun 2022 17:52:56 +0200 Subject: [PATCH] lavc/libx264.c: Fix possible UB by NULL pointer LHS It is UB to attempt to d

Re: [FFmpeg-devel] [PATCH] lavc/dovi_rpu: Fix UB for possible left shift of negative values

2022-06-16 Thread Thilo Borgmann
Am 16.06.22 um 12:40 schrieb Andreas Rheinhardt: Thilo Borgmann: Am 16.06.22 um 12:13 schrieb Andreas Rheinhardt: Thilo Borgmann: diff --git a/libavcodec/dovi_rpu.c b/libavcodec/dovi_rpu.c index a87562c8a3..833ce9e705 100644 --- a/libavcodec/dovi_rpu.c +++ b/libavcodec/dovi_rpu.c @@ -153,7

Re: [FFmpeg-devel] [PATCH] lavc/dovi_rpu: Fix UB for possible left shift of negative values

2022-06-16 Thread Thilo Borgmann
Am 16.06.22 um 12:13 schrieb Andreas Rheinhardt: Thilo Borgmann: diff --git a/libavcodec/dovi_rpu.c b/libavcodec/dovi_rpu.c index a87562c8a3..833ce9e705 100644 --- a/libavcodec/dovi_rpu.c +++ b/libavcodec/dovi_rpu.c @@ -153,7 +153,7 @@ static inline uint64_t get_ue_coef(GetBitContext *gb, const

Re: [FFmpeg-devel] [PATCH] lavc/dovi_rpu: Fix UB for possible left shift of negative values

2022-06-16 Thread Thilo Borgmann
Am 16.06.22 um 11:44 schrieb Andreas Rheinhardt: Thilo Borgmann: Hi, avoid left-shifting possibly negative values in case of RPU_COEFF_FLOAT. Changed RPU_COEFF_FIXED as well for consistency. This is wrong: Both of your changes only affect RPU_COEFF_FIXED. Indeed I'm sorry, got con

[FFmpeg-devel] [PATCH] tests/checkasm/sw_scale: Fix alignment for movdqa

2022-06-16 Thread Thilo Borgmann
Hi, movdqa in ff_yuv2yuvX_sse3() expects a 16-byte alignment according to its documentation causing segfaults in fate-checkasm-sw_scale. -ThiloFrom ed84410b2371a758dad03d3830bfb4f3d86cd4ed Mon Sep 17 00:00:00 2001 From: Michael Goulet Date: Thu, 16 Jun 2022 10:14:50 +0200 Subject: [PATCH] test

[FFmpeg-devel] [PATCH] lavc/dovi_rpu: Fix UB for possible left shift of negative values

2022-06-16 Thread Thilo Borgmann
Hi, avoid left-shifting possibly negative values in case of RPU_COEFF_FLOAT. Changed RPU_COEFF_FIXED as well for consistency. -ThiloFrom 7eca2659b588c404f058eccef478c889555957fd Mon Sep 17 00:00:00 2001 From: Michael Goulet Date: Thu, 16 Jun 2022 10:25:28 +0200 Subject: [PATCH] lavc/dovi_rpu:

Re: [FFmpeg-devel] [PATCH] avcodec/exif: Move tag_list out of header

2022-06-13 Thread Thilo Borgmann
 > Am 12.06.2022 um 20:46 schrieb Andreas Rheinhardt > : > It is only used by exif.c (and e.g. EXIF_TAG_NAME_LENGTH > is an implementation detail anyway). > Also remove the sentinel, as it is used in conjunction > with FF_ARRAY_ELEMS. > > Signed-off-by: Andreas Rheinhardt > --- LGTM if test

[FFmpeg-devel] [RFC] User-defined default enc/dec/mux/dem/etc

2022-06-07 Thread Thilo Borgmann
690543d2d295a5edcc39b74899 Mon Sep 17 00:00:00 2001 From: Thilo Borgmann Date: Tue, 7 Jun 2022 11:33:28 +0200 Subject: [PATCH] defaults --- configure | 48 ++ libavcodec/allcodecs.c | 34 ++ libavcodec/codec.h |

Re: [FFmpeg-devel] [PATCH 1/2] lavfi/blockdetect: Fix possible div by 0

2022-06-06 Thread Thilo Borgmann
Am 04.06.22 um 13:57 schrieb Thilo Borgmann: Now with actual patch… Pushed, thanks! -Thilo ___ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org https://ffmpeg.org/mailman/listinfo/ffmpeg-devel To unsubscribe, visit link above, or email ffmpeg

Re: [FFmpeg-devel] [PATCH 1/2] lavfi/blockdetect: Fix possible div by 0

2022-06-04 Thread Thilo Borgmann
On 4 Jun 2022, at 13:56, Andreas Rheinhardt wrote: > Thilo Borgmann: >> Hi, >> >> Coverity complained about $subject. >> >> Close to trivial, will apply both soonish if nobody screams. >> > > You seem to have forgotten to attach your patches. Than

Re: [FFmpeg-devel] [PATCH 1/2] lavfi/blockdetect: Fix possible div by 0

2022-06-04 Thread Thilo Borgmann
$subject -ThiloFrom 11bc0a429d35815da7950f85b5f48f90758dc9ef Mon Sep 17 00:00:00 2001 From: Thilo Borgmann Date: Sat, 4 Jun 2022 13:47:31 +0200 Subject: [PATCH 2/2] lavfi/blockdetect: Reindent after last commit --- libavfilter/vf_blockdetect.c | 8 1 file changed, 4 insertions(+), 4

Re: [FFmpeg-devel] [PATCH 1/2] lavfi/blockdetect: Fix possible div by 0

2022-06-04 Thread Thilo Borgmann
Now with actual patch… -ThiloFrom 25e875c330e76c2c6be30231e9730fa3a2f30115 Mon Sep 17 00:00:00 2001 From: Thilo Borgmann Date: Sat, 4 Jun 2022 13:45:30 +0200 Subject: [PATCH 1/2] lavfi/blockdetect: Fix possible div by 0 Found-by Covererity, CID 1504568, 1504569 --- libavfilter/vf_blockdetect.c

[FFmpeg-devel] [PATCH 1/2] lavfi/blockdetect: Fix possible div by 0

2022-06-04 Thread Thilo Borgmann
Hi, Coverity complained about $subject. Close to trivial, will apply both soonish if nobody screams. Thanks, Thilo ___ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org https://ffmpeg.org/mailman/listinfo/ffmpeg-devel To unsubscribe, visit link above,

Re: [FFmpeg-devel] GSoC 2022

2022-05-29 Thread Thilo Borgmann
Hi! FFmpeg has been accepted for GSoC this year again! Thanks to everyone who wants to mentor and proposed a project! https://summerofcode.withgoogle.com/programs/2022/organizations/ffmpeg https://trac.ffmpeg.org/wiki/SponsoringPrograms/GSoC/2022 We've been granted two slots this year! One

Re: [FFmpeg-devel] [PATCH] avfilter: Add blockdetect filter

2022-05-24 Thread Thilo Borgmann
Am 18.05.22 um 15:42 schrieb Thilo Borgmann: Am 09.05.22 um 14:24 schrieb Thilo Borgmann: Hi, $subject based on http://www.eurasip.org/Proceedings/Eusipco/Eusipco2005/defevent/papers/cr1042.pdf v2 with minimal changes from IRC. Applying soon. Pushed, thanks! -Thilo

Re: [FFmpeg-devel] [PATCH] avfilter: Add blockdetect filter

2022-05-18 Thread Thilo Borgmann
Am 09.05.22 um 14:24 schrieb Thilo Borgmann: Hi, $subject based on http://www.eurasip.org/Proceedings/Eusipco/Eusipco2005/defevent/papers/cr1042.pdf v2 with minimal changes from IRC. Applying soon. Thanks, Thilo ___ ffmpeg-devel mailing list

Re: [FFmpeg-devel] [PATCH] avfilter: Add blockdetect filter

2022-05-09 Thread Thilo Borgmann
Hi, $subject based on http://www.eurasip.org/Proceedings/Eusipco/Eusipco2005/defevent/papers/cr1042.pdf v2 with minimal changes from IRC. -Thilo From 1a15eac35b30477e5c6d31f277819071851331c1 Mon Sep 17 00:00:00 2001 From: Thilo Borgmann Date: Mon, 9 May 2022 14:21:52 +0200 Subject: [PATCH

[FFmpeg-devel] [PATCH] avfilter: Add blockdetect filter

2022-04-29 Thread Thilo Borgmann
Hi, $subject based on http://www.eurasip.org/Proceedings/Eusipco/Eusipco2005/defevent/papers/cr1042.pdf -ThiloFrom e69095b17619454e671e401ec5c01f130c308a76 Mon Sep 17 00:00:00 2001 From: Thilo Borgmann Date: Fri, 29 Apr 2022 15:58:11 +0200 Subject: [PATCH] avfilter: Add blockdetect filter

Re: [FFmpeg-devel] [PATCH 2/2] lavfi: Add blurriness filter

2022-04-25 Thread Thilo Borgmann
Am 25.04.22 um 00:08 schrieb Thilo Borgmann: Am 25.04.22 um 00:03 schrieb Thilo Borgmann: Am 24.04.22 um 19:28 schrieb Thilo Borgmann: Am 23.04.22 um 15:32 schrieb Thilo Borgmann: Hi, v3 updated to current HEAD. Named blurdetect filter now. Minor fixes on allocation and removed -f option

Re: [FFmpeg-devel] [PATCH 2/2] lavfi: Add blurriness filter

2022-04-24 Thread Thilo Borgmann
Am 25.04.22 um 00:03 schrieb Thilo Borgmann: Am 24.04.22 um 19:28 schrieb Thilo Borgmann: Am 23.04.22 um 15:32 schrieb Thilo Borgmann: Hi, v3 updated to current HEAD. Named blurdetect filter now. Minor fixes on allocation and removed -f option. Please make this per plane filtering, with

Re: [FFmpeg-devel] [PATCH 2/2] lavfi: Add blurriness filter

2022-04-24 Thread Thilo Borgmann
Am 24.04.22 um 19:28 schrieb Thilo Borgmann: Am 23.04.22 um 15:32 schrieb Thilo Borgmann: Hi, v3 updated to current HEAD. Named blurdetect filter now. Minor fixes on allocation and removed -f option. Please make this per plane filtering, with default to measure only first plane. done in

Re: [FFmpeg-devel] [PATCH 2/2] lavfi: Add blurriness filter

2022-04-24 Thread Thilo Borgmann
Am 23.04.22 um 15:32 schrieb Thilo Borgmann: Hi, v3 updated to current HEAD. Named blurdetect filter now. Minor fixes on allocation and removed -f option. Please make this per plane filtering, with default to measure only first plane. done in v4. (Will add Changelog, version.h and fate

Re: [FFmpeg-devel] [PATCH] libavcodec/videotoolboxenc.c: add option to hevc encoder to prioritize speed.

2022-04-24 Thread Thilo Borgmann
Am 23.04.22 um 22:51 schrieb Simone Karin Lehmann: Am 23.04.2022 um 21:56 schrieb Thilo Borgmann : Am 23.04.22 um 17:42 schrieb Simone Karin Lehmann: Am 23.04.2022 um 17:07 schrieb Thilo Borgmann : If that works I'd be happy. Does not apply for me anymore (on HEAD), though? hhmm

<    1   2   3   4   5   6   7   8   9   10   >