Re: [FFmpeg-devel] [PATCH] doc/developer: Better {} style rule

2025-02-27 Thread Nicolas George
Michael Niedermayer (HE12025-02-28): > we have tools/patcheck :) > I have the feeling we started forgeting about it You cannot click on it. Therefore, the coders who are somehow capable of producing C code at quality level for FFmpeg but unable to use a command line and unable to learn will not be

[FFmpeg-devel] [PATCH] libavutil/vulkan: Expose ff_vk_set_descriptor_image

2025-02-27 Thread IndecisiveTurtle
From: IndecisiveTurtle <47210458+raphaelthegr...@users.noreply.github.com> Useful when creating a descriptor array of separate images --- libavutil/vulkan.c | 12 ++-- libavutil/vulkan.h | 8 2 files changed, 14 insertions(+), 6 deletions(-) diff --git a/libavutil/vulkan.c b/li

[FFmpeg-devel] [PATCH v2 4/8] ffprobe/avtextformat: Rename flags and enums

2025-02-27 Thread softworkz
From: softworkz Signed-off-by: softworkz --- fftools/ffprobe.c | 156 ++-- libavutil/avtextformat.h| 24 ++--- libavutil/textformat/avtextformat.c | 26 ++--- 3 files changed, 103 insertions(+), 103 deletions(-) diff --git a/fftools/ffpro

[FFmpeg-devel] [PATCH v2 8/8] ffprobe/avtextformat: Split out text writers as independent classes

2025-02-27 Thread softworkz
From: softworkz Signed-off-by: softworkz --- fftools/ffprobe.c | 20 +++- libavutil/Makefile | 4 + libavutil/avtextformat.h| 13 +-- libavutil/avtextwriters.h | 68 + libavutil/textformat/avtextformat.c | 145 +++

[FFmpeg-devel] [PATCH v2 7/8] ffprobe/avtextformat: Move formatters to avutil

2025-02-27 Thread softworkz
From: softworkz Signed-off-by: softworkz --- fftools/ffprobe.c | 936 +- libavutil/Makefile| 6 + libavutil/avtextformat.h | 8 + libavutil/textformat/tf_compact.c | 285 + libavutil/textformat/tf_default.c | 150 +

[FFmpeg-devel] [PATCH v2 6/8] ffprobe/avtextformat: Rename writer to formatter

2025-02-27 Thread softworkz
From: softworkz ..except for the actrual writers Signed-off-by: softworkz --- fftools/ffprobe.c | 116 ++-- libavutil/avtextformat.h| 14 ++-- libavutil/textformat/avtextformat.c | 48 ++-- 3 files changed, 89 insertions(+), 89 de

[FFmpeg-devel] [PATCH v2 3/8] ffprobe/avtextformat: Generalize ffprobe specifics

2025-02-27 Thread softworkz
From: softworkz Signed-off-by: softworkz --- fftools/ffprobe.c | 34 +++-- libavutil/avtextformat.h| 13 +-- libavutil/textformat/avtextformat.c | 30 ++--- 3 files changed, 35 insertions(+), 42 deletions(-) diff

[FFmpeg-devel] [PATCH v2 5/8] ffprobe/avtextformat: Move flags

2025-02-27 Thread softworkz
From: softworkz Signed-off-by: softworkz --- libavutil/avtextformat.h| 4 libavutil/textformat/avtextformat.c | 4 2 files changed, 4 insertions(+), 4 deletions(-) diff --git a/libavutil/avtextformat.h b/libavutil/avtextformat.h index e40aca1de7..68db9445c9 100644 --- a/l

[FFmpeg-devel] [PATCH v2 2/8] ffprobe/avtextformat: Rename and move writer functions and options

2025-02-27 Thread softworkz
From: softworkz Signed-off-by: softworkz --- fftools/ffprobe.c | 894 ++-- libavutil/Makefile | 1 + libavutil/avtextformat.h| 50 ++ libavutil/textformat/avtextformat.c | 621 +++ 4 files changed, 841 ins

[FFmpeg-devel] [PATCH v2 1/8] ffprobe/avtextformat: Rename and move writer structs

2025-02-27 Thread softworkz
From: softworkz Signed-off-by: softworkz --- fftools/ffprobe.c| 370 +++ libavutil/Makefile | 1 + libavutil/avtextformat.h | 120 + 3 files changed, 264 insertions(+), 227 deletions(-) create mode 100644 libavutil/avtextformat.h

[FFmpeg-devel] [PATCH v2 0/8] [RFC] avtextformat: Transform text writing into an independent API

2025-02-27 Thread ffmpegagent
Hello everybody, this is an effort to pull out the text writing code from ffprobe and shape it as an independent API. To avoid any other big refacoring in the future, this patchset is building it up as a public API in avutil, but it's not that I would be insisting this to happen (or maybe not yet)

Re: [FFmpeg-devel] [PATCH v2 1/3] avformat/movenc: Add AVS3 support

2025-02-27 Thread Zhao Zhili
> On Feb 25, 2025, at 10:38, Zhao Zhili wrote: > > > >> On Feb 25, 2025, at 01:52, Michael Niedermayer >> wrote: >> >> On Mon, Feb 24, 2025 at 09:51:32PM +0800, Zhao Zhili wrote: >>> From: Zhao Zhili >>> >>> 'avs3' is registered at mp4ra.org. The Avs3ConfigurationBox 'av3c' >>> inside 'a

[FFmpeg-devel] [PATCH] fftools/ffmpeg_filter: also remove display matrix side data from buffered frames

2025-02-27 Thread James Almer
Some frames may be buffered before a complex filtergraph can be configured. This change ensures the side data removal in the cases where autorotation is enabled also applies to them. Fixes ticket #11487 Signed-off-by: James Almer --- fftools/ffmpeg_filter.c | 2 ++ 1 file changed, 2 insertions(

Re: [FFmpeg-devel] [PATCH 2/8] ffprobe/avtextformat: Rename and move writer functions and options

2025-02-27 Thread Michael Niedermayer
On Fri, Feb 28, 2025 at 01:47:05AM +, Soft Works wrote: > > > > -Original Message- > > From: ffmpeg-devel On Behalf Of Soft > > Works > > Sent: Freitag, 28. Februar 2025 02:31 > > To: FFmpeg development discussions and patches > > Subject: Re: [FFmpeg-devel] [PATCH 2/8] ffprobe/avte

Re: [FFmpeg-devel] [PATCH] doc/developer: Better {} style rule

2025-02-27 Thread Michael Niedermayer
On Fri, Feb 28, 2025 at 12:25:43AM +0100, epira...@gmail.com wrote: > > > On 28 Feb 2025, at 0:14, Michael Niedermayer wrote: > > > Hi > > > > On Thu, Feb 27, 2025 at 11:46:54PM +0100, epira...@gmail.com wrote: > >> On 27 Feb 2025, at 2:10, Michael Niedermayer wrote: > >> > >>> Signed-off-by: Mi

Re: [FFmpeg-devel] [PATCH 2/8] ffprobe/avtextformat: Rename and move writer functions and options

2025-02-27 Thread Soft Works
> -Original Message- > From: ffmpeg-devel On Behalf Of > Michael Niedermayer > Sent: Freitag, 28. Februar 2025 03:29 > To: FFmpeg development discussions and patches > Subject: Re: [FFmpeg-devel] [PATCH 2/8] ffprobe/avtextformat: Rename and > move writer functions and options > > On F

Re: [FFmpeg-devel] [PATCH] doc/developer: Better {} style rule

2025-02-27 Thread Michael Niedermayer
On Fri, Feb 28, 2025 at 03:25:06AM +0100, Lynne wrote: > On 27/02/2025 02:10, Michael Niedermayer wrote: > > Signed-off-by: Michael Niedermayer > > --- > > doc/developer.texi | 11 +-- > > 1 file changed, 5 insertions(+), 6 deletions(-) > > > > diff --git a/doc/developer.texi b/doc/dev

Re: [FFmpeg-devel] [PATCH] swscale/aarch64: dotprod implementation of rgba32_to_Y

2025-02-27 Thread Zhao Zhili
Cc haasn. Libswscale in under refactor. Does current asm works after refactor, or they need to be refactored or rewrite after? If it’s the second case, maybe we should hold on to do more asm with libswscale before hassn work done. > On Feb 28, 2025, at 06:44, Krzysztof Pyrkosz via ffmpeg-devel

Re: [FFmpeg-devel] [PATCH] Mark C globals with small code model

2025-02-27 Thread Lynne
On 25/02/2025 22:37, Pranav Kant via ffmpeg-devel wrote: By default, all globals in C/C++ compiled by clang are allocated in non-large data sections. See [1] for background on code models. For PIC (Position independent code), this is fine as long as binary is small but as binary size increases, u

Re: [FFmpeg-devel] [PATCH] doc/developer: Better {} style rule

2025-02-27 Thread Lynne
On 27/02/2025 02:10, Michael Niedermayer wrote: Signed-off-by: Michael Niedermayer --- doc/developer.texi | 11 +-- 1 file changed, 5 insertions(+), 6 deletions(-) diff --git a/doc/developer.texi b/doc/developer.texi index a1bfe180c9b..6a753f99da6 100644 --- a/doc/developer.texi +++

Re: [FFmpeg-devel] I've written a filter in Rust

2025-02-27 Thread Pavel Koshevoy
On Thu, Feb 27, 2025 at 2:02 PM Michael Niedermayer wrote: > On Wed, Feb 26, 2025 at 03:11:13PM +0100, Tomas Härdin wrote: > > sön 2025-02-23 klockan 22:51 +0100 skrev Michael Niedermayer: > > > Hi > > > > > > On Sun, Feb 23, 2025 at 10:30:03PM +0100, Tomas Härdin wrote: > > > > lör 2025-02-22 kl

Re: [FFmpeg-devel] [PATCH 2/8] ffprobe/avtextformat: Rename and move writer functions and options

2025-02-27 Thread Soft Works
> -Original Message- > From: ffmpeg-devel On Behalf Of Soft > Works > Sent: Freitag, 28. Februar 2025 02:31 > To: FFmpeg development discussions and patches > Subject: Re: [FFmpeg-devel] [PATCH 2/8] ffprobe/avtextformat: Rename and > move writer functions and options > > > > > -

Re: [FFmpeg-devel] [PATCH 2/8] ffprobe/avtextformat: Rename and move writer functions and options

2025-02-27 Thread Michael Niedermayer
On Thu, Feb 27, 2025 at 02:01:34PM +, softworkz wrote: > From: softworkz > > Signed-off-by: softworkz > --- > fftools/ffprobe.c | 910 ++-- > libavutil/Makefile | 1 + > libavutil/avtextformat.h| 50 ++ > libavutil/te

Re: [FFmpeg-devel] [PATCH] avformat/hls: Revert "reduce default max reload to 3"

2025-02-27 Thread Soft Works
> -Original Message- > From: ffmpeg-devel On Behalf Of > Michael Niedermayer > Sent: Donnerstag, 27. Februar 2025 23:58 > To: FFmpeg development discussions and patches > Subject: Re: [FFmpeg-devel] [PATCH] avformat/hls: Revert "reduce default > max reload to 3" > > On Wed, Feb 26, 202

Re: [FFmpeg-devel] [PATCH 2/8] ffprobe/avtextformat: Rename and move writer functions and options

2025-02-27 Thread Soft Works
> -Original Message- > From: ffmpeg-devel On Behalf Of > Michael Niedermayer > Sent: Freitag, 28. Februar 2025 02:23 > To: FFmpeg development discussions and patches > Subject: Re: [FFmpeg-devel] [PATCH 2/8] ffprobe/avtextformat: Rename and > move writer functions and options > > On T

Re: [FFmpeg-devel] [PATCH 1/8] ffprobe/avtextformat: Rename and move writer structs

2025-02-27 Thread Michael Niedermayer
On Thu, Feb 27, 2025 at 02:01:33PM +, softworkz wrote: > From: softworkz > > Signed-off-by: softworkz > --- > fftools/ffprobe.c| 356 +++ > libavutil/Makefile | 1 + > libavutil/avtextformat.h | 120 + > 3 files changed, 257 in

Re: [FFmpeg-devel] [PATCH] Mark C globals with small code model

2025-02-27 Thread Michael Niedermayer
On Wed, Feb 26, 2025 at 07:44:37PM +, Pranav Kant via ffmpeg-devel wrote: > By default, all globals in C/C++ compiled by clang are allocated > in non-large data sections. See [1] for background on code models. > For PIC (Position independent code), this is fine as long as binary is > small but

Re: [FFmpeg-devel] [PATCH] doc/developer: Better {} style rule

2025-02-27 Thread Soft Works
> -Original Message- > From: ffmpeg-devel On Behalf Of Devin > Heitmueller > Sent: Freitag, 28. Februar 2025 01:40 > To: FFmpeg development discussions and patches > Subject: Re: [FFmpeg-devel] [PATCH] doc/developer: Better {} style rule > > On Thu, Feb 27, 2025 at 5:57 PM James Almer

Re: [FFmpeg-devel] [PATCH] doc/developer: Better {} style rule

2025-02-27 Thread Michael Niedermayer
On Thu, Feb 27, 2025 at 07:40:14PM -0500, Devin Heitmueller wrote: > On Thu, Feb 27, 2025 at 5:57 PM James Almer wrote: > > > would lve a format defining script with a prehook that formats your > > > patches before sending <3 > > > > That can be done automatically as one of the many jobs CI ru

Re: [FFmpeg-devel] [PATCH] doc/developer: Better {} style rule

2025-02-27 Thread Devin Heitmueller
On Thu, Feb 27, 2025 at 5:57 PM James Almer wrote: > > would lve a format defining script with a prehook that formats your > > patches before sending <3 > > That can be done automatically as one of the many jobs CI runs once we > move to forgejo/gitlab. Same with every other check in patcheck.

Re: [FFmpeg-devel] [PATCH] doc/developer: Better {} style rule

2025-02-27 Thread epirat07
On 28 Feb 2025, at 0:14, Michael Niedermayer wrote: > Hi > > On Thu, Feb 27, 2025 at 11:46:54PM +0100, epira...@gmail.com wrote: >> On 27 Feb 2025, at 2:10, Michael Niedermayer wrote: >> >>> Signed-off-by: Michael Niedermayer >>> --- >>> doc/developer.texi | 11 +-- >>> 1 file changed

Re: [FFmpeg-devel] [PATCH] doc/developer: Better {} style rule

2025-02-27 Thread Michael Niedermayer
Hi On Thu, Feb 27, 2025 at 11:46:54PM +0100, epira...@gmail.com wrote: > On 27 Feb 2025, at 2:10, Michael Niedermayer wrote: > > > Signed-off-by: Michael Niedermayer > > --- > > doc/developer.texi | 11 +-- > > 1 file changed, 5 insertions(+), 6 deletions(-) > > > > diff --git a/doc/dev

Re: [FFmpeg-devel] [PATCH v2] avformat/hls: Partially revert "reduce default max reload to 3"

2025-02-27 Thread Michael Niedermayer
On Thu, Feb 27, 2025 at 04:36:56PM +, softworkz wrote: > From: softworkz > > (setting to 100 as a reasonable compromise) > > The change has caused regressions for many users and consumers. > Playlist reloads only happen when a playlist doesn't indicate that it > has ended (via #EXT-X-ENDLIST

Re: [FFmpeg-devel] FFmpeg Community Committee – Updates & Next Steps

2025-02-27 Thread Marth64
Hi Michael, Thanks for the reply & thoughts. To clarify on "internal panel" and "structured issue resolution", this is with regards to unresolved complaints sent to CC, such as publicly or privately reported interpersonal/CoC issues. Some high ticket items directed to the CC from last year remain

Re: [FFmpeg-devel] [PATCH] avformat/hls: Revert "reduce default max reload to 3"

2025-02-27 Thread Michael Niedermayer
On Wed, Feb 26, 2025 at 11:38:29PM +, Soft Works wrote: > > > > -Original Message- > > From: ffmpeg-devel On Behalf Of > > Michael Niedermayer > > Sent: Mittwoch, 26. Februar 2025 01:39 > > To: FFmpeg development discussions and patches > de...@ffmpeg.org> > > Subject: Re: [FFmpeg-d

Re: [FFmpeg-devel] [PATCH] doc/developer: Better {} style rule

2025-02-27 Thread James Almer
On 2/27/2025 7:53 PM, Vittorio Giovara wrote: On Thu, Feb 27, 2025 at 11:47 PM wrote: On 27 Feb 2025, at 2:10, Michael Niedermayer wrote: Signed-off-by: Michael Niedermayer --- doc/developer.texi | 11 +-- 1 file changed, 5 insertions(+), 6 deletions(-) diff --git a/doc/develope

Re: [FFmpeg-devel] [PATCH] doc/developer: Better {} style rule

2025-02-27 Thread Vittorio Giovara
On Thu, Feb 27, 2025 at 11:47 PM wrote: > On 27 Feb 2025, at 2:10, Michael Niedermayer wrote: > > > Signed-off-by: Michael Niedermayer > > --- > > doc/developer.texi | 11 +-- > > 1 file changed, 5 insertions(+), 6 deletions(-) > > > > diff --git a/doc/developer.texi b/doc/developer.tex

[FFmpeg-devel] [PATCH] swscale/aarch64: dotprod implementation of rgba32_to_Y

2025-02-27 Thread Krzysztof Pyrkosz via ffmpeg-devel
--- I was curious whether it's possible to implement this function without any widening, and it turns out it not only is, but it's quite performant at the same time! The idea is to split the 16 bit coefficients into lower and upper half, invoke udot for the lower half, shift by 8, and follow by ud

Re: [FFmpeg-devel] [PATCH] doc/developer: Better {} style rule

2025-02-27 Thread epirat07
On 27 Feb 2025, at 2:10, Michael Niedermayer wrote: > Signed-off-by: Michael Niedermayer > --- > doc/developer.texi | 11 +-- > 1 file changed, 5 insertions(+), 6 deletions(-) > > diff --git a/doc/developer.texi b/doc/developer.texi > index a1bfe180c9b..6a753f99da6 100644 > --- a/doc/dev

Re: [FFmpeg-devel] FFmpeg Community Committee – Updates & Next Steps

2025-02-27 Thread Vittorio Giovara
On Thu, Feb 27, 2025 at 7:18 PM Michael Niedermayer wrote: > Hi jb > > On Thu, Feb 27, 2025 at 12:11:39AM +0100, Jean-Baptiste Kempf wrote: > > On Wed, 26 Feb 2025, at 02:51, Michael Niedermayer wrote: > [...] > > > 2. The CC is overstepping its authority. > > > > Literally, the email just says t

Re: [FFmpeg-devel] I've written a filter in Rust

2025-02-27 Thread Michael Niedermayer
Hi On Wed, Feb 26, 2025 at 04:35:52PM +, Soft Works wrote: [...] > > > What we could do is make it easier to build from source with extra > > components: > > [..] > > > > - Build scripts that detect available libraries. Possibly build scripts > > that install devel packages for these lib

Re: [FFmpeg-devel] Please do proper Release Candidates

2025-02-27 Thread James Almer
On 2/27/2025 6:05 PM, Andrew Randrianasulu wrote: Otherwise it just feels like random snapshot everyone downstream must adapt to. Does it make a difference? Every release is going to be a snapshot at some point in development. And how much testing is a release candidate going to get? Distros

Re: [FFmpeg-devel] [PATCH v2 1/4] avfilter/graphdump: implement options parsing

2025-02-27 Thread Soft Works
> -Original Message- > From: ffmpeg-devel On Behalf Of > Niklas Haas > Sent: Donnerstag, 27. Februar 2025 18:53 > To: ffmpeg-devel@ffmpeg.org > Cc: Niklas Haas > Subject: Re: [FFmpeg-devel] [PATCH v2 1/4] avfilter/graphdump: implement > options parsing > > On Tue, 18 Feb 2025 13:46:00

[FFmpeg-devel] Please do proper Release Candidates

2025-02-27 Thread Andrew Randrianasulu
Otherwise it just feels like random snapshot everyone downstream must adapt to. Also, PLEASE add API changelog in future release announcements on website, I want to see what broken in each release. Right now it looks like only "positive" end of news openly publicized. Be honest, post other end of

Re: [FFmpeg-devel] I've written a filter in Rust

2025-02-27 Thread Michael Niedermayer
On Wed, Feb 26, 2025 at 03:11:13PM +0100, Tomas Härdin wrote: > sön 2025-02-23 klockan 22:51 +0100 skrev Michael Niedermayer: > > Hi > > > > On Sun, Feb 23, 2025 at 10:30:03PM +0100, Tomas Härdin wrote: > > > lör 2025-02-22 klockan 14:57 +0200 skrev Rémi Denis-Courmont: > > > > Le perjantaina 21.

Re: [FFmpeg-devel] FFmpeg Community Committee – Updates & Next Steps

2025-02-27 Thread Michael Niedermayer
Hi jb On Thu, Feb 27, 2025 at 12:11:39AM +0100, Jean-Baptiste Kempf wrote: > On Wed, 26 Feb 2025, at 02:51, Michael Niedermayer wrote: [...] > > 2. The CC is overstepping its authority. > > Literally, the email just says that the CC is now meeting regularly; what > authority can it be oversteppi

Re: [FFmpeg-devel] [PATCH v2 1/4] avfilter/graphdump: implement options parsing

2025-02-27 Thread Niklas Haas
On Tue, 18 Feb 2025 13:46:00 +0100 Niklas Haas wrote: > From: Niklas Haas > > And use it to make the output format configurable. > > I also added a "none" option to explicitly disable the behavior (in case a > previous command line argument was used to enable it). This has the added > benefit of

Re: [FFmpeg-devel] [PATCH 01/12] avutil/frame: add AVFrame.alpha_mode

2025-02-27 Thread Niklas Haas
On Wed, 19 Feb 2025 21:45:37 +0100 Niklas Haas wrote: > From: Niklas Haas > > FFmpeg currently handles alpha in a quasi-arbitrary way. Some filters/codecs > assume alpha is premultiplied, others assume it is independent. If there is > to be any hope for order in this chaos, we need to start by de

Re: [FFmpeg-devel] CVE-2023-6602 fixes backporting

2025-02-27 Thread Michael Niedermayer
Hi On Thu, Feb 27, 2025 at 02:36:48AM +0100, Michael Niedermayer wrote: > On Wed, Feb 26, 2025 at 12:03:12AM -0300, James Almer wrote: > > On 2/25/2025 11:28 PM, Michael Niedermayer wrote: > > > Hi all > > > > > > As i backported the fixes for $subj, i noticed they depend on > > > ff_match_url_ex

Re: [FFmpeg-devel] [BASIC QUESTION] Profiling tool for ffmpeg

2025-02-27 Thread Leandro Santiago
I've been using Hotspot, a Qt based GUI for Linux perf, and am quite happy with the results: https://github.com/KDAB/hotspot Valgrind massif, callgrind and cachegrind also have some GUIs for Linux (massif-visualizer and Kcachegrind). Cheers, Leandro On 2/24/25 15:43, Cesar Matheus wrote: > Hi

[FFmpeg-devel] [PATCH v2] avformat/hls: Partially revert "reduce default max reload to 3"

2025-02-27 Thread softworkz
From: softworkz (setting to 100 as a reasonable compromise) The change has caused regressions for many users and consumers. Playlist reloads only happen when a playlist doesn't indicate that it has ended (via #EXT-X-ENDLIST), which means that the addition of future segments is still expected. It

Re: [FFmpeg-devel] [PATCH v4] avformat: add AV1 RTP depacketizer and packetizer

2025-02-27 Thread Chris Hodges
On 2/26/25 19:26, Ronald S. Bultje wrote: Added code to support this and enabled the extra scan for the sequence header OBU. This is great work, thanks for the additions and LGTM after some testing. Merged. Thanks a lot for reviewing and valuable suggestions to all involved! Will stay on

[FFmpeg-devel] [PATCH 2/8] ffprobe/avtextformat: Rename and move writer functions and options

2025-02-27 Thread softworkz
From: softworkz Signed-off-by: softworkz --- fftools/ffprobe.c | 910 ++-- libavutil/Makefile | 1 + libavutil/avtextformat.h| 50 ++ libavutil/textformat/avtextformat.c | 620 +++ 4 files changed, 848 ins

[FFmpeg-devel] [PATCH 8/8] ffprobe/avtextformat: Split out text writers as independent classes

2025-02-27 Thread softworkz
From: softworkz Signed-off-by: softworkz --- fftools/ffprobe.c | 20 +++- libavutil/Makefile | 4 + libavutil/avtextformat.h| 13 +-- libavutil/avtextwriters.h | 68 + libavutil/textformat/avtextformat.c | 145 +++

[FFmpeg-devel] [PATCH 7/8] ffprobe/avtextformat: Move formatters to avutil

2025-02-27 Thread softworkz
From: softworkz Signed-off-by: softworkz --- fftools/ffprobe.c | 936 +- libavutil/Makefile| 6 + libavutil/avtextformat.h | 8 + libavutil/textformat/tf_compact.c | 285 + libavutil/textformat/tf_default.c | 150 +

[FFmpeg-devel] [PATCH 6/8] ffprobe/avtextformat: Rename writer to formatter

2025-02-27 Thread softworkz
From: softworkz ..except for the actrual writers Signed-off-by: softworkz --- fftools/ffprobe.c | 100 ++-- libavutil/avtextformat.h| 14 ++-- libavutil/textformat/avtextformat.c | 48 ++--- 3 files changed, 81 insertions(+), 81 d

[FFmpeg-devel] [PATCH 5/8] ffprobe/avtextformat: Move flags

2025-02-27 Thread softworkz
From: softworkz Signed-off-by: softworkz --- libavutil/avtextformat.h| 4 libavutil/textformat/avtextformat.c | 4 2 files changed, 4 insertions(+), 4 deletions(-) diff --git a/libavutil/avtextformat.h b/libavutil/avtextformat.h index e40aca1de7..68db9445c9 100644 --- a/l

[FFmpeg-devel] [PATCH 4/8] ffprobe/avtextformat: Rename flags and enums

2025-02-27 Thread softworkz
From: softworkz Signed-off-by: softworkz --- fftools/ffprobe.c | 156 ++-- libavutil/avtextformat.h| 24 ++--- libavutil/textformat/avtextformat.c | 26 ++--- 3 files changed, 103 insertions(+), 103 deletions(-) diff --git a/fftools/ffpro

[FFmpeg-devel] [PATCH 3/8] ffprobe/avtextformat: Generalize ffprobe specifics

2025-02-27 Thread softworkz
From: softworkz Signed-off-by: softworkz --- fftools/ffprobe.c | 34 +++-- libavutil/avtextformat.h| 13 +-- libavutil/textformat/avtextformat.c | 30 ++--- 3 files changed, 35 insertions(+), 42 deletions(-) diff

[FFmpeg-devel] [PATCH 1/8] ffprobe/avtextformat: Rename and move writer structs

2025-02-27 Thread softworkz
From: softworkz Signed-off-by: softworkz --- fftools/ffprobe.c| 356 +++ libavutil/Makefile | 1 + libavutil/avtextformat.h | 120 + 3 files changed, 257 insertions(+), 220 deletions(-) create mode 100644 libavutil/avtextformat.h

[FFmpeg-devel] [PATCH 0/8] [RFC] avtextformat: Transform text writing into an independent API

2025-02-27 Thread ffmpegagent
Hello everybody, this is an effort to pull out the text writing code from ffprobe and shape it as an independent API. To avoid any other big refacoring in the future, this patchset is building it up as a public API in avutil, but it's not that I would be insisting this to happen (or maybe not yet)

Re: [FFmpeg-devel] [PATCH 2/3] fftools/ffmpeg_graphprint: Add options for filtergraph printing

2025-02-27 Thread Soft Works
> -Original Message- > From: ffmpeg-devel On Behalf Of > Nicolas George > Sent: Mittwoch, 26. Februar 2025 15:43 > To: FFmpeg development discussions and patches de...@ffmpeg.org> > Subject: Re: [FFmpeg-devel] [PATCH 2/3] fftools/ffmpeg_graphprint: Add > options for filtergraph printin

Re: [FFmpeg-devel] [PATCH] avformat/mov: (v4) fix get_eia608_packet

2025-02-27 Thread Andreas Rheinhardt
Pavel Koshevoy: > On Wed, Feb 26, 2025 at 8:03 PM Zhao Zhili < > quinkblack-at-foxmail@ffmpeg.org> wrote: > >> >> >>> On Feb 24, 2025, at 00:47, Pavel Koshevoy wrote: >>> >>> On Fri, Feb 21, 2025 at 9:49 AM Pavel Koshevoy >> wrote: >>> If there are no further constructive review comment