Re: [FFmpeg-devel] [PATCH] libavfilter/vf_signalstats.c: add new hsl(Hue, Saturation, Lightness) for filter signalstats v2

2023-03-03 Thread Michael Niedermayer
On Thu, Mar 02, 2023 at 09:19:18PM +0800, liuyizhuo wrote: > From: "yizhuo.liu...@gmail.com" > > --- > libavfilter/vf_signalstats.c | 219 +- > .../fate/filter-metadata-signalstats-yuv420p | 2 +- > .../filter-metadata-signalstats-yuv420p10 | 2 +- > 3

[FFmpeg-devel] RFC: Preserving original demux timestamps in AVPackets

2023-03-03 Thread Devin Heitmueller
Hello all, I am working on some SCTE-35 stuff, which embeds PTS timestamps deep within the section payload. In order to pass through packets to an MPEG-TS output, I need the original PTS value that was used when the packet was received to calculate the correct offsets. The way a typical

[FFmpeg-devel] [PATCH v2 3/3] decklink: Fix unused variable compiler warnings

2023-03-03 Thread Devin Heitmueller
Due to refactoring, the ctx/cctx variables are never actually used in ff_decklink_write_packet(), so just remove them. Signed-off-by: Devin Heitmueller --- libavdevice/decklink_enc.cpp | 2 -- 1 file changed, 2 deletions(-) diff --git a/libavdevice/decklink_enc.cpp

[FFmpeg-devel] [PATCH v2 2/3] decklink: Fix setting of last_pts to only be set for video

2023-03-03 Thread Devin Heitmueller
The ff_decklink_write_packet() was always caching the last pts received, to be used when calling StopScheduledPlayback(). However because audio and video are on different timebases and the call to StopScheduledPlayback() expects the video timebase, we'll end up sending a weird value to the stop

[FFmpeg-devel] [PATCH v2 1/3] decklink: Don't take for granted that first frame to decklink output will be PTS 0

2023-03-03 Thread Devin Heitmueller
The existing code assumed that the first frame received by the decklink output would always be PTS zero. However if running in other timing modes than the default of CBR, items such as frame dropping at the beginning may result in starting at a non-zero PTS. For example, in our setup because we

[FFmpeg-devel] [PATCH 2/2] avformat/jpegxl_anim_dec: add animated JPEG XL demuxer

2023-03-03 Thread Leo Izen
Animated JPEG XL files requires a separate demuxer than image2, because the timebase information is set by the demuxer. Should the timebase of an animated JPEG XL file be incompatible with the timebase set by the image2pipe demuxer (usually 1/25 unless set otherwise), rescaling will fail. Adding a

[FFmpeg-devel] [PATCH 1/2] avcodec/libjxldec: add animated decode support

2023-03-03 Thread Leo Izen
Migrate the libjxl decoder wrapper from the decode_frame method to the receive_frame method, which allows sending more than one frame from a single packet. This allows the libjxl decoder to decode JPEG XL files that are animated, and emit every frame of the animation. Now, clients that feed the

[FFmpeg-devel] [PATCH 0/2] JPEG XL Animation Support

2023-03-03 Thread Leo Izen
This patch adds support for animated JPEG XL files in both the libjxl decoder wrapper and a separate demuxer to properly set the timebase for it. Leo Izen (2): avcodec/libjxldec: add animated decode support avformat/jpegxl_anim_dec: add animated JPEG XL demuxer MAINTAINERS

[FFmpeg-devel] [PATCH 2/2] avcodec/libdav1d: reindent after previous commit

2023-03-03 Thread James Almer
Signed-off-by: James Almer --- libavcodec/libdav1d.c | 34 +- 1 file changed, 17 insertions(+), 17 deletions(-) diff --git a/libavcodec/libdav1d.c b/libavcodec/libdav1d.c index dcd7cd3f05..aacbc9a59e 100644 --- a/libavcodec/libdav1d.c +++ b/libavcodec/libdav1d.c

[FFmpeg-devel] [PATCH 1/2] avcodec/libdav1d: export Dynamic HDR10+ frame metadata

2023-03-03 Thread James Almer
As defined in https://aomediacodec.github.io/av1-hdr10plus/ Signed-off-by: James Almer --- A sample is available in https://github.com/AOMediaCodec/av1-hdr10plus/wiki libavcodec/libdav1d.c | 35 +++ 1 file changed, 31 insertions(+), 4 deletions(-) diff --git

[FFmpeg-devel] [PATCH] libavformat/http.c: cookie path attribute should be optional not compulsory

2023-03-03 Thread Michael J. Walsh
The path attribute in the Set-Cookie header is optional but treated by ffmpeg as being compulsory. Signed-off-by: Michael J. Walsh —-- libavformat/http.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/libavformat/http.c b/libavformat/http.c index 7bce821535..dde22a7e09

[FFmpeg-devel] [PATCH] libavforamt/tls: Memory allocation failure check

2023-03-03 Thread jackarain
Signed-off-by: jackarain --- libavformat/tls.c | 26 -- 1 file changed, 20 insertions(+), 6 deletions(-) diff --git a/libavformat/tls.c b/libavformat/tls.c index e42d399862..b9d3426331 100644 --- a/libavformat/tls.c +++ b/libavformat/tls.c @@ -30,15 +30,18 @@ #include

[FFmpeg-devel] [PATCH] libavforamt/udp: Memory allocation failure check

2023-03-03 Thread jackarain
Signed-off-by: jackarain --- libavformat/udp.c | 4 1 file changed, 4 insertions(+) diff --git a/libavformat/udp.c b/libavformat/udp.c index e8980b29d8..909213a467 100644 --- a/libavformat/udp.c +++ b/libavformat/udp.c @@ -740,6 +740,10 @@ static int udp_open(URLContext *h, const char

[FFmpeg-devel] ?????? [PATCH] fftools/ffmpeg: reset pointer, which could be reused by Android and iOS

2023-03-03 Thread ??????
Thank you for your reminding, George. I really ignore av_freep() above. Please ignore this patch. // xufuji456 ---- ??:

[FFmpeg-devel] RFC: opinions on making fftools work like a library

2023-03-03 Thread Zhao Zhili
I'd like to know what do you think about making fftools work like a library, like what ffmpeg-kit already did, but with built-in support. https://github.com/arthenica/ffmpeg-kit FFmpegKit is a collection of tools to use FFmpeg in Android, iOS, Linux, macOS, tvOS, Flutter and React Native

Re: [FFmpeg-devel] [PATCH v7 0/4] add ARIB caption decoder using libaribcaption

2023-03-03 Thread TADANO Tokumei
On 2023/02/28 9:07, Ridley Combs wrote: On Feb 23, 2023, at 04:17, TADANO Tokumei wrote: Ping with rebased patch set. Some chages are added to reflect review results from outside of this ML. --- This patch set add another ARIB caption decoder using libaribcaption external library:

[FFmpeg-devel] [PATCH] libavformat/tcp: add local_addr/local_port for network option

2023-03-03 Thread jackarain
Signed-off-by: jackarain --- doc/protocols.texi| 6 + libavformat/network.c | 14 libavformat/network.h | 2 +- libavformat/tcp.c | 53 +-- 4 files changed, 68 insertions(+), 7 deletions(-) diff --git a/doc/protocols.texi

Re: [FFmpeg-devel] [PATCH] fftools/ffmpeg: reset pointer, which could be reused by Android and iOS

2023-03-03 Thread Nicolas George
"zhilizhao(赵志立)" (12023-03-03): > > av_freep(_files); > > av_freep(_files); > > +input_files = NULL; > > +output_files = NULL; > Until we decided to make ffmpeg cmd works as a library, > it doesn’t matter. It is worse than that: the patch completely ignores the semantic

Re: [FFmpeg-devel] [PATCH] fftools/ffmpeg: reset pointer, which could be reused by Android and iOS

2023-03-03 Thread zhilizhao(赵志立)
> On Mar 3, 2023, at 14:51, xufuji456 <839789...@qq.com> wrote: > > --- > fftools/ffmpeg.c | 7 +++ > 1 file changed, 7 insertions(+) > > diff --git a/fftools/ffmpeg.c b/fftools/ffmpeg.c > index 743bc0c6b6..0ce9531235 100644 > --- a/fftools/ffmpeg.c > +++ b/fftools/ffmpeg.c > @@ -569,6