Re: [FFmpeg-devel] [PATCH] ipfsgateway: Remove default gateway

2022-08-22 Thread Tomas Härdin
fre 2022-08-19 klockan 14:52 +0200 skrev Mark Gaiser: > > > I believe we went over this in detail during those patch rounds when > this > was brought up (by you?). > I didn't go back in the archives to find it, but some reasons that > come to > mind: > - just handing the mere edge cases would mak

[FFmpeg-devel] [PATCH 1/2] avformat/ildadec: ILDA Image Data Transfer Format demuxer

2022-08-22 Thread Peter Ross
Signed-off-by: Peter Ross --- sample: 'ftp://ftp.photonlexicon.com/public/Laser Shows/ILDA Frames/wim.ild' doc/general_contents.texi | 1 + libavformat/Makefile | 1 + libavformat/allformats.c | 1 + libavformat/ildadec.c | 137 ++ 4 files chan

[FFmpeg-devel] [PATCH 2/2] avcodec/ildadec: ILDA Image Data Transfer Format decoder

2022-08-22 Thread Peter Ross
Signed-off-by: Peter Ross --- libavcodec/Makefile | 1 + libavcodec/allcodecs.c | 1 + libavcodec/codec_desc.c | 7 + libavcodec/codec_id.h | 1 + libavcodec/ildadec.c| 297 5 files changed, 307 insertions(+) create mode 100644 libavcod

[FFmpeg-devel] [PATCH] fftools/ffmpeg: stop accessing av_stream_get_parser() from the main thread

2022-08-22 Thread Anton Khirnov
It races with the demuxing thread. Instead, send the information along with the demuxed packets. Ideally, the code should stop using the stream-internal parsing completely, but that requires considerably more effort. Fixes races, e.g. in: - fate-h264-brokensps-2580 - fate-h264-extradata-reload -

Re: [FFmpeg-devel] [PATCH v5 00/25] Subtitle Filtering 2022

2022-08-22 Thread Anton Khirnov
Quoting Jean-Baptiste Kempf (2022-08-21 12:41:20) > > On Sun, 21 Aug 2022, at 11:41, Paul B Mahol wrote: > > We should more forward and merge this considerable subtitle work > > Are there parts of this work that have reach majority consensus? Almost exactly identical objections to the basic asp

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

2022-08-22 Thread 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 simplistic parser overhead. > Also, maybe I'm j

Re: [FFmpeg-devel] [PATCH 1/2] doc/plans: add a file to list approved projects

2022-08-22 Thread Nicolas George
Jean-Baptiste Kempf (12022-08-19): > Exactly. This is called the bus-factor and is important in all > volunteers open source communities. > > Which is why the norm is "show us the code" and not wait for future > things. > > Because as we are volunteers working when we can, shit happens in real >

Re: [FFmpeg-devel] [PATCH] ipfsgateway: Remove default gateway

2022-08-22 Thread Nicolas George
Tomas Härdin (12022-08-22): > I'd actually argue that in that case we should link a library that > implements IPFS, not split developer effort by trying to implement it > ourselves. Is FFmpeg meant to be just a convenient set of wrappers for existing libraries, then? If not, what is your criterio

Re: [FFmpeg-devel] [PATCH v5 00/25] Subtitle Filtering 2022

2022-08-22 Thread Jean-Baptiste Kempf
On Mon, 22 Aug 2022, at 14:18, Anton Khirnov wrote: > Almost exactly identical objections to the basic aspects of the API were > raised independently by me, Lynne, and Hendrik. > IIUC Soft Works still refuses to address them (though it's not so easy > to tell in a 200-email thread). OK. I lost the

Re: [FFmpeg-devel] [PATCH 1/2] doc/plans: add a file to list approved projects

2022-08-22 Thread Jean-Baptiste Kempf
On Mon, 22 Aug 2022, at 14:43, Nicolas George wrote: >> Therefore, I don't think those plans documents are useful: "show us >> the code!". > > Try this with your plumber: “Show me my shower fixed and I'll pay you if > I'm satisfied.” It does not work that way, you have to sign a sales This is a co

Re: [FFmpeg-devel] [PATCH 2/2] doc/plans: add AVWriter

2022-08-22 Thread Leo Izen
On 8/18/22 13:23, Jean-Baptiste Kempf wrote: On Wed, 17 Aug 2022, at 23:49, Nicolas George wrote: +An API that can be used everywhere a function needs to return a string, or +really an arbitrary buffer of bytes, and everywhere code needs to build a +string from parts. It needs to be fast and lig

[FFmpeg-devel] [PATCH 1/2] avcodec/m101: Move checks before ff_get_buffer()

2022-08-22 Thread Michael Niedermayer
Fixes: Timeout Fixes: 50109/clusterfuzz-testcase-minimized-ffmpeg_AV_CODEC_ID_M101_fuzzer-6553193986785280 Found-by: continuous fuzzing process https://github.com/google/oss-fuzz/tree/master/projects/ffmpeg Signed-off-by: Michael Niedermayer --- libavcodec/m101.c | 9 - 1 file changed,

[FFmpeg-devel] [PATCH 2/2] tools/target_dec_fuzzer: Adjust threshold for MTS2

2022-08-22 Thread Michael Niedermayer
Fixes: Timeout Fixes: 50030/clusterfuzz-testcase-minimized-ffmpeg_AV_CODEC_ID_MTS2_fuzzer-5767793731043328 Found-by: continuous fuzzing process https://github.com/google/oss-fuzz/tree/master/projects/ffmpeg Signed-off-by: Michael Niedermayer --- tools/target_dec_fuzzer.c | 1 + 1 file changed,

Re: [FFmpeg-devel] [PATCH 2/2] doc/plans: add AVWriter

2022-08-22 Thread Nicolas George
Leo Izen (12022-08-22): > > > +An API that can be used everywhere a function needs to return a > > > string, or > > > +really an arbitrary buffer of bytes, and everywhere code needs to > > > build a > > > +string from parts. It needs to be fast and lightweight enough to be > > > used in > > > +tigh

Re: [FFmpeg-devel] [PATCH 1/2] doc/plans: add a file to list approved projects

2022-08-22 Thread Nicolas George
Jean-Baptiste Kempf (12022-08-22): > This is a contract, with paid money and clear objectives. > FFmpeg is a volunteer open source project, not a contracting or plumbing > initiative. > If the SoW is not done, the money is not paid. This does not work for open > source communities. I was not as

[FFmpeg-devel] [PATCH 02/10] avcodec/wmalosslessdec: Remove unnecessary emms_c()

2022-08-22 Thread Andreas Rheinhardt
Possible since 6feea076e98512d78c8d735509ab6b5e9a71ca1c. Signed-off-by: Andreas Rheinhardt --- libavcodec/wmalosslessdec.c | 1 - 1 file changed, 1 deletion(-) diff --git a/libavcodec/wmalosslessdec.c b/libavcodec/wmalosslessdec.c index 614b6135f5..6ba9f04d25 100644 --- a/libavcodec/wmalossless

[FFmpeg-devel] [PATCH 03/10] avcodec/takdec: Remove unnecessary emms_c()

2022-08-22 Thread Andreas Rheinhardt
Possible since 3d716d38abdae1982e84e30becb57458244656bd. Signed-off-by: Andreas Rheinhardt --- libavcodec/takdec.c | 4 1 file changed, 4 deletions(-) diff --git a/libavcodec/takdec.c b/libavcodec/takdec.c index 3e53401328..6f4cc92e88 100644 --- a/libavcodec/takdec.c +++ b/libavcodec/takde

[FFmpeg-devel] [PATCH 04/10] avcodec/jpeglsenc: Remove unnecessary emms_c()

2022-08-22 Thread Andreas Rheinhardt
This encoder does not use any MMX anywhere. Signed-off-by: Andreas Rheinhardt --- libavcodec/jpeglsenc.c | 2 -- 1 file changed, 2 deletions(-) diff --git a/libavcodec/jpeglsenc.c b/libavcodec/jpeglsenc.c index ba9355225b..8f3197e687 100644 --- a/libavcodec/jpeglsenc.c +++ b/libavcodec/jpeglsen

[FFmpeg-devel] [PATCH 05/10] avcodec/ffv1(dec|enc): Remove unnecessary emms_c()

2022-08-22 Thread Andreas Rheinhardt
These codecs do not use MMX at all. Signed-off-by: Andreas Rheinhardt --- libavcodec/ffv1dec.c | 2 -- libavcodec/ffv1enc.c | 1 - 2 files changed, 3 deletions(-) diff --git a/libavcodec/ffv1dec.c b/libavcodec/ffv1dec.c index 4fe1738dea..fd549c7913 100644 --- a/libavcodec/ffv1dec.c +++ b/libavc

[FFmpeg-devel] [PATCH 06/10] avcodec/apedec: Remove unnecessary emms_c()

2022-08-22 Thread Andreas Rheinhardt
Possible since 6feea076e98512d78c8d735509ab6b5e9a71ca1c. Signed-off-by: Andreas Rheinhardt --- libavcodec/apedec.c | 1 - 1 file changed, 1 deletion(-) diff --git a/libavcodec/apedec.c b/libavcodec/apedec.c index 26183067a8..e7baa2e77f 100644 --- a/libavcodec/apedec.c +++ b/libavcodec/apedec.c

[FFmpeg-devel] [PATCH 07/10] avcodec/4xm: Remove unnecessary and redundat emms_c()

2022-08-22 Thread Andreas Rheinhardt
It is unnecessary since ee551a21ddcbf81afe183d9489c534ee80f263a0; but it was redundant even before that, because decode_simple_internal() calls emms_c(). Signed-off-by: Andreas Rheinhardt --- libavcodec/4xm.c | 2 -- 1 file changed, 2 deletions(-) diff --git a/libavcodec/4xm.c b/libavcodec/4xm.

[FFmpeg-devel] [PATCH 08/10] avcodec/loongarch/cabac, vp9dsp_loongarch: Add missing headers

2022-08-22 Thread Andreas Rheinhardt
Fixes checkheaders on loongarch. Signed-off-by: Andreas Rheinhardt --- libavcodec/loongarch/cabac.h| 1 + libavcodec/loongarch/vp9dsp_loongarch.h | 3 +++ 2 files changed, 4 insertions(+) diff --git a/libavcodec/loongarch/cabac.h b/libavcodec/loongarch/cabac.h index e1c946fe16..f896

[FFmpeg-devel] [PATCH 09/10] avformat/os_support: Include stdint.h for int64_t

2022-08-22 Thread Andreas Rheinhardt
Fixes checkheaders for Windows targets. Signed-off-by: Andreas Rheinhardt --- libavformat/os_support.h | 1 + 1 file changed, 1 insertion(+) diff --git a/libavformat/os_support.h b/libavformat/os_support.h index b419efa408..f2ff38e23b 100644 --- a/libavformat/os_support.h +++ b/libavformat/os_s

[FFmpeg-devel] [PATCH 10/10] avutil/mem_internal: Fix headers

2022-08-22 Thread Andreas Rheinhardt
Including avassert.h is unnecessary since commit 786be70e28fe739b8e49893fa13ae4652a68d1ea. Signed-off-by: Andreas Rheinhardt --- libavcodec/x86/h264_qpel.c | 1 + libavutil/mem_internal.h | 3 ++- libavutil/tx.c | 1 + 3 files changed, 4 insertions(+), 1 deletion(-) diff --git a/l

Re: [FFmpeg-devel] [PATCH 1/4] avcodec/pngdec: Fix APNG_DISPOSE_OP_BACKGROUND

2022-08-22 Thread Andreas Rheinhardt
Andreas Rheinhardt: > APNG works with a single reference frame and an output frame. > According to the spec, decoding APNG works by decoding > the current IDAT/fdAT chunks (which decodes to a rectangular > subregion of the whole image region), followed by either > overwriting the region of the outp

Re: [FFmpeg-devel] [PATCH] swscale/x86/rgb2_rgb: Empty MMX state in ff_shuffle_bytes_2103_mmxext

2022-08-22 Thread Andreas Rheinhardt
Andreas Rheinhardt: > Fixes FATE-failures with the the filter-2xbr filter-3xbr filter-4xbr > filter-ep2x filter-ep3x filter-hq2x filter-hq3x filter-hq4x > filter-paletteuse-bayer filter-paletteuse-bayer0 > filter-paletteuse-nodither and filter-paletteuse-sierra2_4a tests > when using 32bit x86 with

Re: [FFmpeg-devel] [PATCH v5 00/25] Subtitle Filtering 2022

2022-08-22 Thread Soft Works
From: ffmpeg-devel on behalf of Anton Khirnov Sent: Monday, August 22, 2022 2:18 PM To: ffmpeg-devel Subject: Re: [FFmpeg-devel] [PATCH v5 00/25] Subtitle Filtering 2022 Quoting Jean-Baptiste Kempf (2022-08-21 12:41:20) > > On Sun, 21 Aug 2022, at 11:

[FFmpeg-devel] [PATCH 3/4] tools/target_dec_fuzzer: Adjust threshold for MVDV

2022-08-22 Thread Michael Niedermayer
Fixes: Timeout Fixes: 50213/clusterfuzz-testcase-minimized-ffmpeg_AV_CODEC_ID_MVDV_fuzzer-5228284098510848 Found-by: continuous fuzzing process https://github.com/google/oss-fuzz/tree/master/projects/ffmpeg Signed-off-by: Michael Niedermayer --- tools/target_dec_fuzzer.c | 1 + 1 file changed,

[FFmpeg-devel] [PATCH 1/4] libavformat/iff: Check for overflow in body_end calculation

2022-08-22 Thread Michael Niedermayer
Fixes: signed integer overflow: -6322983228386819992 - 5557477266266529857 cannot be represented in type 'long' Fixes: 50112/clusterfuzz-testcase-minimized-ffmpeg_dem_IFF_fuzzer-6329186221948928 Found-by: continuous fuzzing process https://github.com/google/oss-fuzz/tree/master/projects/ffmpeg

[FFmpeg-devel] [PATCH 4/4] libavcodec/8bps: Check that line lengths fit within the buffer

2022-08-22 Thread Michael Niedermayer
Fixes: Timeout Fixes: undefined pointer arithmetic Fixes: 50330/clusterfuzz-testcase-minimized-ffmpeg_AV_CODEC_ID_EIGHTBPS_fuzzer-5436287485607936 Found-by: continuous fuzzing process https://github.com/google/oss-fuzz/tree/master/projects/ffmpeg Signed-off-by: Michael Niedermayer --- libavcod

[FFmpeg-devel] [PATCH 2/4] avcodec/midivid: Perform lzss_uncompress() before ff_reget_buffer()

2022-08-22 Thread Michael Niedermayer
This would avoid regeting the frame on lzss errors Signed-off-by: Michael Niedermayer --- libavcodec/midivid.c | 13 ++--- 1 file changed, 6 insertions(+), 7 deletions(-) diff --git a/libavcodec/midivid.c b/libavcodec/midivid.c index 7448c8c797..6b76d5bcad 100644 --- a/libavcodec/midivi

Re: [FFmpeg-devel] [PATCH v5 00/25] Subtitle Filtering 2022

2022-08-22 Thread Soft Works
From: ffmpeg-devel on behalf of Soft Works Sent: Tuesday, August 23, 2022 12:08 AM To: FFmpeg development discussions and patches Subject: Re: [FFmpeg-devel] [PATCH v5 00/25] Subtitle Filtering 2022 From: ffm

Re: [FFmpeg-devel] [PATCH] avformat/wavdec: fix the ID3 metadata obtained in WAV format's missing

2022-08-22 Thread Steven Liu
Wujian(Chin) 于2022年8月22日周一 22:00写道: > > Fixes ticket #9848. > > Signed-off-by: wujian_nanjing > --- > libavformat/wavdec.c | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/libavformat/wavdec.c b/libavformat/wavdec.c > index ca61b84..e3f790f 100644 > --- a/libavformat/wavd

Re: [FFmpeg-devel] [PATCH v2] lavf/dashdec: Multithreaded DASH initialization

2022-08-22 Thread Steven Liu
Lukas Fellechner 于2022年8月22日周一 03:27写道: > > Initializing DASH streams is currently slow, because each individual stream > is opened and probed sequentially. With DASH streams often having somewhere > between 10-20 streams, this can easily take up to half a minute. This patch > adds an "init-thr

[FFmpeg-devel] [PATCH v1 1/2] avformat/imfdec: use CPL start timecode if available

2022-08-22 Thread pal
From: Pierre-Anthony Lemieux The IMF CPL contains an optional timecode start address. This patch reads the latter, if present, into the context's timecode metadata parameter. This addresses https://trac.ffmpeg.org/ticket/9842. --- libavformat/imf.h | 2 + libavformat/imf_cpl.c | 109 +

[FFmpeg-devel] [PATCH v1 2/2] avformat/tests/imf: add CPL timecode test

2022-08-22 Thread pal
From: Pierre-Anthony Lemieux --- libavformat/tests/imf.c | 7 +++ tests/ref/fate/imf | 1 + 2 files changed, 8 insertions(+) diff --git a/libavformat/tests/imf.c b/libavformat/tests/imf.c index e65629ccbc..d42295dc29 100644 --- a/libavformat/tests/imf.c +++ b/libavformat/tests/imf.c @@

Re: [FFmpeg-devel] [PATCH] libavcodec/qsvenc: Add adaptive_i/b to hevc_qsv

2022-08-22 Thread Xiang, Haihao
On Wed, 2022-08-17 at 15:07 +0800, Wenbin Chen wrote: > Add adaptive_i/b feature to hevc_qsv. Adaptive_i allows changing of > frame type from P and B to I. Adaptive_b allows changing of frame type > frome B to P. > > Signed-off-by: Wenbin Chen > --- > doc/encoders.texi| 7 +++ > liba

Re: [FFmpeg-devel] [PATCH v2] qsvenc_{hevc, h264}: add scenario option

2022-08-22 Thread Xiang, Haihao
On Mon, 2022-08-22 at 14:29 +0800, Xiang, Haihao wrote: > From: Haihao Xiang > > User may get better experience after providing corresponding senario hint > to encoder for the encoding session. See [1] for scenario support in the > SDK > > [1] > https://github.com/Intel-Media-SDK/MediaSDK/blob/m

[FFmpeg-devel] [PATCH 1/2] lavc/qsvenc: ignore video signal info buffer for VP9

2022-08-22 Thread Xiang, Haihao
From: Haihao Xiang The SDK doesn't support this type of buffer for vp9 encoding Signed-off-by: Haihao Xiang --- libavcodec/qsvenc.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/libavcodec/qsvenc.c b/libavcodec/qsvenc.c index 202a645ea2..f1838ce597 100644 --- a/libavcodec

[FFmpeg-devel] [PATCH 2/2] lavc/qsvenc: enlarge the SPS buffer to retrieve larger header

2022-08-22 Thread Xiang, Haihao
From: gavin zhang Increase SPS header buffer to support larger header Signed-off-by: gavin zhang Signed-off-by: Haihao Xiang --- libavcodec/qsvenc.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/libavcodec/qsvenc.c b/libavcodec/qsvenc.c index f1838ce597..7ac5390f10 10064

Re: [FFmpeg-devel] [PATCH v2] libavfilter/x86/vf_convolution: add sobel filter optimization and unit test with intel AVX512 VNNI

2022-08-22 Thread Xiang, Haihao
On Mon, 2022-08-15 at 13:38 +0800, bin.wang-at-intel@ffmpeg.org wrote: > From: bwang30 > > This commit enabled assembly code with intel AVX512 VNNI and added unit test > for sobel filter > > sobel_c: 4537 > sobel_avx512icl 2470 > > Signed-off-by: bwang30 > --- > libavfilter/convolution.h