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

2023-03-02 Thread xufuji456
--- 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 +569,13 @@ static void ffmpeg_cleanup(int ret) av_freep(_files);

[FFmpeg-devel] [PATCH 2/2] avutil: add HDR10+ dynamic metadata serialization function

2023-03-02 Thread Raphaël Zumer
Fixed brace style and moved inline buffer size calculation comments to a single block at the top. Signed-off-by: Raphaël Zumer --- libavutil/hdr_dynamic_metadata.c | 142 +++ libavutil/hdr_dynamic_metadata.h | 11 +++ libavutil/version.h | 2 +- 3

Re: [FFmpeg-devel] [PATCH 1/2] avcodec/libvpxdec: remove unnecessary init_static_data function

2023-03-02 Thread James Zern
On Tue, Feb 28, 2023 at 1:20 PM James Zern wrote: > > On Tue, Feb 28, 2023 at 12:49 PM James Almer wrote: > > > > On 2/28/2023 5:40 PM, James Zern wrote: > > > On Tue, Feb 28, 2023 at 4:07 AM James Almer wrote: > > >> > > >> It's used only by the encoder. > > >> > > > > > > It sets

Re: [FFmpeg-devel] [PATCH 2/2] avutil: add HDR10+ dynamic metadata serialization function

2023-03-02 Thread Raphaël Zumer
On 3/2/23 15:24, Leo Izen wrote: > On 3/2/23 14:25, Raphaël Zumer wrote: >> Signed-off-by: Raphaël Zumer >> --- >> libavutil/hdr_dynamic_metadata.c | 146 +++ >> libavutil/hdr_dynamic_metadata.h | 11 +++ >> libavutil/version.h | 2 +- >> 3 files

Re: [FFmpeg-devel] [PATCH 2/2] avutil: add HDR10+ dynamic metadata serialization function

2023-03-02 Thread Derek Buitenhuis
On 3/2/2023 8:24 PM, Leo Izen wrote: > Why not put this in avcodec/dynamic_hdr10_plus.c? You reference > put_bits.h which is in avcodec, so that can possibly be an issue, even > though it is inlined (i.e. it sends the wrong message since avutil is > supposed to not depend on avcodec). put_bits

Re: [FFmpeg-devel] [PATCH 2/2] avutil: add HDR10+ dynamic metadata serialization function

2023-03-02 Thread Leo Izen
On 3/2/23 14:25, Raphaël Zumer wrote: Signed-off-by: Raphaël Zumer --- libavutil/hdr_dynamic_metadata.c | 146 +++ libavutil/hdr_dynamic_metadata.h | 11 +++ libavutil/version.h | 2 +- 3 files changed, 158 insertions(+), 1 deletion(-) Why not

[FFmpeg-devel] [PATCH 2/2] avutil: add HDR10+ dynamic metadata serialization function

2023-03-02 Thread Raphaël Zumer
Signed-off-by: Raphaël Zumer --- libavutil/hdr_dynamic_metadata.c | 146 +++ libavutil/hdr_dynamic_metadata.h | 11 +++ libavutil/version.h | 2 +- 3 files changed, 158 insertions(+), 1 deletion(-) diff --git a/libavutil/hdr_dynamic_metadata.c

[FFmpeg-devel] Question: about "igndts" flag, and how to handle DTS output problems.

2023-03-02 Thread Daniel Cantarín
I'm having trouble with DTS values in a single dvbsub stream while receiving LIVE MPEGTS with muxed video, audio, subs, and other data (like scte35) over multicast UDP. I'm doing transcoding tasks, and then output that to another MPEGTS stream that feeds other services. This are the problems

Re: [FFmpeg-devel] [PATCH 2/2] avutil: add HDR10+ dynamic metadata serialization function

2023-03-02 Thread Raphaël Zumer
On 3/2/23 13:57, James Almer wrote: > > The SEI for HEVC does keep the country code in the payload, but not AV1. > Are you sure about HEVC? I just checked a sample and trace_headers > reported this: > > [trace_headers] Prefix Supplemental Enhancement Information > [trace_headers] 0

Re: [FFmpeg-devel] [PATCH 2/2] avutil: add HDR10+ dynamic metadata serialization function

2023-03-02 Thread James Almer
On 3/2/2023 3:33 PM, quietvoid wrote:> On 27/02/2023 12.34, Raphaël Zumer wrote: > >> Resent due to my mail client incorrectly re-wrapping lines in the >> version I sent earlier. >> See the previous patch for context. >> >> Signed-off-by: Raphaël Zumer >> --- >>

Re: [FFmpeg-devel] [PATCH 2/2] avutil: add HDR10+ dynamic metadata serialization function

2023-03-02 Thread Raphaël Zumer
On 3/2/23 13:33, quietvoid wrote: > This patch series mentions that it would be used for AV1, however the AV1 > specification is clear that the payload does not contain the country code. > https://aomediacodec.github.io/av1-hdr10plus/#hdr10plus-metadata, Figure 1. > > So far all the AV1 HDR10+

Re: [FFmpeg-devel] [PATCH] doc/filters: update SITI description

2023-03-02 Thread Werner Robitza
On Thu, Mar 2, 2023 at 7:02 PM Thilo Borgmann wrote: > > Am 01.03.23 um 08:46 schrieb Werner Robitza: > > On Wed, Mar 1, 2023 at 1:15 AM Thilo Borgmann > > wrote: > >> > >> Am 28.02.23 um 18:12 schrieb Werner Robitza: > >>> On Tue, Feb 28, 2023 at 2:16 PM Thilo Borgmann > >>> wrote: > >

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

2023-03-02 Thread Nicolas George
Rémi Denis-Courmont (12023-03-02): > That needs to match the socket protocol family, or bind() will fail. Oh, right! The proper way to work with getaddrinfo is to create the socket afterwards based on its return, but staying within a family is a good second best choice. Thanks for noticing.

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

2023-03-02 Thread Rémi Denis-Courmont
Le keskiviikkona 1. maaliskuuta 2023, 17.44.56 EET jackarain a écrit : > Signed-off-by: jackarain > --- > doc/protocols.texi| 6 + > libavformat/network.c | 14 > libavformat/network.h | 2 +- > libavformat/tcp.c | 53 ++- > 4

Re: [FFmpeg-devel] [PATCH 2/2] avutil: add HDR10+ dynamic metadata serialization function

2023-03-02 Thread quietvoid
On 27/02/2023 12.34, Raphaël Zumer wrote: Resent due to my mail client incorrectly re-wrapping lines in the version I sent earlier. See the previous patch for context. Signed-off-by: Raphaël Zumer --- libavutil/hdr_dynamic_metadata.c | 147 +++

Re: [FFmpeg-devel] [PATCH] doc/filters: update SITI description

2023-03-02 Thread Thilo Borgmann
Am 01.03.23 um 08:46 schrieb Werner Robitza: On Wed, Mar 1, 2023 at 1:15 AM Thilo Borgmann wrote: Am 28.02.23 um 18:12 schrieb Werner Robitza: On Tue, Feb 28, 2023 at 2:16 PM Thilo Borgmann wrote: Am 28.02.23 um 14:13 schrieb Thilo Borgmann: Am 28.02.23 um 12:42 schrieb Werner Robitza:

[FFmpeg-devel] [PATCH] lavu/vulkan: fix handle type for 32-bit targets

2023-03-02 Thread Kacper Michajłow
Fixes compilation with clang which errors out on Wint-conversion. Signed-off-by: Kacper Michajłow --- libavutil/hwcontext_vulkan.c | 2 +- libavutil/vulkan.h | 4 2 files changed, 5 insertions(+), 1 deletion(-) diff --git a/libavutil/hwcontext_vulkan.c

Re: [FFmpeg-devel] [PATCH] libavcodec/h264dec: avoid arithmetic on null pointers

2023-03-02 Thread Jeremy Dorfman
On Thu, Mar 2, 2023 at 6:37 AM James Almer wrote: > > On 3/2/2023 8:33 AM, James Almer wrote: > > On 3/2/2023 6:05 AM, Anton Khirnov wrote: > >> Quoting Jeremy Dorfman (2023-03-01 19:50:08) > >>> null pointer arithmetic is undefined behavior in C. > >>> --- > >>> libavcodec/h264dec.c | 4 ++-- >

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

2023-03-02 Thread liuyizhuo
From: "yizhuo.liu...@gmail.com" --- libavfilter/vf_signalstats.c | 219 +- .../fate/filter-metadata-signalstats-yuv420p | 2 +- .../filter-metadata-signalstats-yuv420p10 | 2 +- 3 files changed, 220 insertions(+), 3 deletions(-) diff --git

[FFmpeg-devel] [PATCH] avfilter/graphparser: fix filter instance name when an id is provided

2023-03-02 Thread James Almer
Restores the behavior of naming the instance filter@id, which was accidentally changed to simpy id in commit f17051eaae. Fixes ticket #10226. Signed-off-by: James Almer --- libavfilter/graphparser.c | 7 --- 1 file changed, 4 insertions(+), 3 deletions(-) diff --git

Re: [FFmpeg-devel] [PATCH] libavcodec/h264dec: avoid arithmetic on null pointers

2023-03-02 Thread James Almer
On 3/2/2023 8:33 AM, James Almer wrote: On 3/2/2023 6:05 AM, Anton Khirnov wrote: Quoting Jeremy Dorfman (2023-03-01 19:50:08) null pointer arithmetic is undefined behavior in C. ---   libavcodec/h264dec.c | 4 ++--   1 file changed, 2 insertions(+), 2 deletions(-) diff --git

Re: [FFmpeg-devel] [PATCH] libavcodec/h264dec: avoid arithmetic on null pointers

2023-03-02 Thread James Almer
On 3/2/2023 6:05 AM, Anton Khirnov wrote: Quoting Jeremy Dorfman (2023-03-01 19:50:08) null pointer arithmetic is undefined behavior in C. --- libavcodec/h264dec.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/libavcodec/h264dec.c b/libavcodec/h264dec.c index

Re: [FFmpeg-devel] [PATCH v2 0/4] Add hevc alpha layer decoding basic support

2023-03-02 Thread zhilizhao(赵志立)
> On Mar 2, 2023, at 18:11, Anton Khirnov wrote: > > Quoting zhilizhao(赵志立) (2023-02-28 10:00:15) >> >>> On Jan 6, 2023, at 23:52, Zhao Zhili wrote: >>> >>> From: Zhao Zhili >>> >>> v2: >>> 1. Check vps_max_layers and vps_max_layer_id inside hevc decoder >>> 2. Add hevc_extract_layer bsf

Re: [FFmpeg-devel] [PATCH v2 0/4] Add hevc alpha layer decoding basic support

2023-03-02 Thread Anton Khirnov
Quoting zhilizhao(赵志立) (2023-02-28 10:00:15) > > > On Jan 6, 2023, at 23:52, Zhao Zhili wrote: > > > > From: Zhao Zhili > > > > v2: > > 1. Check vps_max_layers and vps_max_layer_id inside hevc decoder > > 2. Add hevc_extract_layer bsf to do the job rather than modifing > > hevc_metadata, > >

Re: [FFmpeg-devel] [PATCH] avcodec/ac3: Remove unused fields

2023-03-02 Thread Anton Khirnov
Quoting Nicolas Gaullier (2023-02-21 11:12:32) > Signed-off-by: Nicolas Gaullier > --- > libavcodec/ac3dec.h | 2 -- > 1 file changed, 2 deletions(-) Looks ok, will push. -- Anton Khirnov ___ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org

Re: [FFmpeg-devel] [PATCH] libavcodec/h264dec: avoid arithmetic on null pointers

2023-03-02 Thread Anton Khirnov
Quoting Jeremy Dorfman (2023-03-01 19:50:08) > null pointer arithmetic is undefined behavior in C. > --- > libavcodec/h264dec.c | 4 ++-- > 1 file changed, 2 insertions(+), 2 deletions(-) > > diff --git a/libavcodec/h264dec.c b/libavcodec/h264dec.c > index 2d691731c5..ef698f2630 100644 > ---