Re: [FFmpeg-devel] [PATCH v3 1/2] avcodec/aac_ac3_parser: simplify

2025-07-17 Thread Nicolas Gaullier
On 7/11/25 11:54, Nicolas Gaullier wrote: Remove unused USAC/ADTS code: ff_adts_header_parse() parse the object_type from a 2 bits field. See also 696ea1c2236842572df88d573e24a39be3f19c98. Use ff_adts_header_parse_buf() wrapper to simplify as GetBitContext is no longer needed (it was introduced

Re: [FFmpeg-devel] [PATCH v3 1/2] avcodec/aac_ac3_parser: simplify

2025-07-15 Thread Nicolas Gaullier
On 7/12/25 12:52, Michael Niedermayer wrote: On Fri, Jul 11, 2025 at 11:54:29AM +0200, Nicolas Gaullier wrote: Remove unused USAC/ADTS code: ff_adts_header_parse() parse the object_type from a 2 bits field. See also 696ea1c2236842572df88d573e24a39be3f19c98. Use ff_adts_header_parse_buf

[FFmpeg-devel] [PATCH v3 2/2] avcodec/aac_ac3_parser: do not override the profile set by the decoder

2025-07-11 Thread Nicolas Gaullier
Parsing the ADTS header is not enough to detect HE-AAC v1/v2. Regression since 64bb91fd3b5a00a8849531c7e8dd207f2a626096. Fixes #11600 Signed-off-by: Nicolas Gaullier --- libavcodec/aac_ac3_parser.c | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/libavcodec

[FFmpeg-devel] [PATCH v3 0/2] fix HE-AAC profile probing and simplify

2025-07-11 Thread Nicolas Gaullier
v3: complete overhaul of the commit message of the first patch Code is unchanged. Second patch is unchanged ("should be ok" by James). Nicolas Gaullier (2): avcodec/aac_ac3_parser: simplify avcodec/aac_ac3_parser: do not override the profile set by the decoder libavcodec/aac_ac

[FFmpeg-devel] [PATCH v3 1/2] avcodec/aac_ac3_parser: simplify

2025-07-11 Thread Nicolas Gaullier
64bb91fd3b5a00a8849531c7e8dd207f2a626096. Signed-off-by: Nicolas Gaullier --- libavcodec/aac_ac3_parser.c | 7 +++ 1 file changed, 3 insertions(+), 4 deletions(-) diff --git a/libavcodec/aac_ac3_parser.c b/libavcodec/aac_ac3_parser.c index e10ce13a3b..b49ce0277c 100644 --- a/libavcodec/aac_ac3_parser.c

Re: [FFmpeg-devel] [PATCH v2 1/2] avcodec/aac_ac3_parser: remove dead code

2025-07-07 Thread Nicolas Gaullier
On 7/7/25 15:40, James Almer wrote: On 7/7/2025 10:19 AM, Nicolas Gaullier wrote: ff_adts_header_parse() gets the object_type from only 2 bits. See also 696ea1c2236842572df88d573e24a39be3f19c98. Introduced in 64bb91fd3b5a00a8849531c7e8dd207f2a626096. Signed-off-by: Nicolas Gaullier

[FFmpeg-devel] [PATCH v2 2/2] avcodec/aac_ac3_parser: do not override the profile set by the decoder

2025-07-07 Thread Nicolas Gaullier
Parsing the ADTS header is not enough to detect HE-AAC v1/v2. Regression since 64bb91fd3b5a00a8849531c7e8dd207f2a626096. Fixes #11600 Signed-off-by: Nicolas Gaullier --- libavcodec/aac_ac3_parser.c | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/libavcodec

[FFmpeg-devel] [PATCH v2 1/2] avcodec/aac_ac3_parser: remove dead code

2025-07-07 Thread Nicolas Gaullier
ff_adts_header_parse() gets the object_type from only 2 bits. See also 696ea1c2236842572df88d573e24a39be3f19c98. Introduced in 64bb91fd3b5a00a8849531c7e8dd207f2a626096. Signed-off-by: Nicolas Gaullier --- libavcodec/aac_ac3_parser.c | 7 +++ 1 file changed, 3 insertions(+), 4 deletions

Re: [FFmpeg-devel] [PATCH] avcodec/aac_parser: fix aac profile probing

2025-07-01 Thread Nicolas Gaullier
On 6/30/25 21:03, Baptiste Coudurier wrote: On Jun 30, 2025, at 11:08 AM, Nicolas Gaullier wrote: On 6/28/25 03:54, Baptiste Coudurier wrote: On Jun 27, 2025, at 5:01 AM, Nicolas Gaullier wrote: On 6/26/25 19:30, Baptiste Coudurier wrote: Hi Nicolas On Jun 26, 2025, at 4:06 AM, Nicolas

Re: [FFmpeg-devel] [EXTERNE] Re: [PATCH] avcodec/aac_parser: fix aac profile probing

2025-06-30 Thread Nicolas Gaullier
On 6/28/25 03:54, Baptiste Coudurier wrote: On Jun 27, 2025, at 5:01 AM, Nicolas Gaullier wrote: On 6/26/25 19:30, Baptiste Coudurier wrote: Hi Nicolas On Jun 26, 2025, at 4:06 AM, Nicolas Gaullier wrote: On 6/17/25 13:41, Nicolas Gaullier wrote: Reverts

[FFmpeg-devel] [PATCH v3 21/21] fate/seek: fix missing mpeg4video parser dependency

2025-06-30 Thread Nicolas Gaullier
Signed-off-by: Nicolas Gaullier --- tests/fate/seek.mak | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/tests/fate/seek.mak b/tests/fate/seek.mak index aa1f856f7b..5504e331e6 100644 --- a/tests/fate/seek.mak +++ b/tests/fate/seek.mak @@ -62,7 +62,6 @@ FATE_SEEK_VSYNTH_LENA

[FFmpeg-devel] [PATCH v3 20/21] fate/all: fix missing some mov demuxer dependencies

2025-06-30 Thread Nicolas Gaullier
In aac/aac-fixed, also remove unnecessary aac demuxer dependency. Signed-off-by: Nicolas Gaullier --- tests/fate/aac.mak| 2 +- tests/fate/h264.mak | 2 +- tests/fate/lavf-container.mak | 8 tests/fate/mov.mak| 14 +++--- 4 files changed, 13

[FFmpeg-devel] [PATCH v3 19/21] fate/all: add missing crc/framecrc/md5/framemd5/pipe dependencies

2025-06-30 Thread Nicolas Gaullier
When CMD is crc/framecrc, always use the macros CRC/FRAMECRC, even if it includes unnecessary requirements for rawvideo/pcm_s16le encoders (as actually noticed in a comment of the Makefile). Signed-off-by: Nicolas Gaullier --- tests/Makefile| 3 ++ tests/fate/aac.mak

[FFmpeg-devel] [PATCH v3 13/21] fate/hlsenc: fix multiple dependencies

2025-06-30 Thread Nicolas Gaullier
Signed-off-by: Nicolas Gaullier --- tests/fate/hlsenc.mak | 18 +- 1 file changed, 9 insertions(+), 9 deletions(-) diff --git a/tests/fate/hlsenc.mak b/tests/fate/hlsenc.mak index 98341b2b81..72c75bec3a 100644 --- a/tests/fate/hlsenc.mak +++ b/tests/fate/hlsenc.mak @@ -5,7 +5,7

[FFmpeg-devel] [PATCH v3 18/21] fate/all: add missing file protocol dependencies

2025-06-30 Thread Nicolas Gaullier
First, always require file protocol when FATE suite is used. Then, add missing dependencies while removing duplicates. Signed-off-by: Nicolas Gaullier --- tests/Makefile | 1 + tests/fate/cbs.mak | 4 ++-- tests/fate/cover-art.mak| 8 tests/fate/demux.mak

[FFmpeg-devel] [PATCH v3 08/21] fate/vorbis: fix multiple dependencies

2025-06-30 Thread Nicolas Gaullier
Signed-off-by: Nicolas Gaullier --- tests/fate/vorbis.mak | 24 +--- 1 file changed, 13 insertions(+), 11 deletions(-) diff --git a/tests/fate/vorbis.mak b/tests/fate/vorbis.mak index 781430bcd3..73e837f5a6 100644 --- a/tests/fate/vorbis.mak +++ b/tests/fate/vorbis.mak

[FFmpeg-devel] [PATCH v3 17/21] fate/all: fix multiple dependencies

2025-06-30 Thread Nicolas Gaullier
Signed-off-by: Nicolas Gaullier --- tests/fate/api.mak | 2 +- tests/fate/cbs.mak | 2 +- tests/fate/ffmpeg.mak | 6 +++--- tests/fate/filter-audio.mak | 8 tests/fate/filter-video.mak | 2 +- tests/fate/fits.mak | 4 ++-- tests/fate/gif.mak | 2

[FFmpeg-devel] [PATCH v3 16/21] fate/hevc: fix multiple dependencies

2025-06-30 Thread Nicolas Gaullier
Signed-off-by: Nicolas Gaullier --- tests/fate/hevc.mak | 8 1 file changed, 4 insertions(+), 4 deletions(-) diff --git a/tests/fate/hevc.mak b/tests/fate/hevc.mak index 3ad6e255a9..7b22e36a3d 100644 --- a/tests/fate/hevc.mak +++ b/tests/fate/hevc.mak @@ -219,7 +219,7 @@ FATE_HEVC

[FFmpeg-devel] [PATCH v3 15/21] fate/pcm: fix multiple dependencies

2025-06-30 Thread Nicolas Gaullier
Signed-off-by: Nicolas Gaullier --- tests/fate/pcm.mak | 12 ++-- 1 file changed, 6 insertions(+), 6 deletions(-) diff --git a/tests/fate/pcm.mak b/tests/fate/pcm.mak index b10233a9eb..75636118b2 100644 --- a/tests/fate/pcm.mak +++ b/tests/fate/pcm.mak @@ -1,4 +1,4 @@ -FATE_SAMPLES_PCM

[FFmpeg-devel] [PATCH v3 14/21] fate/mpeg4: fix multiple dependencies

2025-06-30 Thread Nicolas Gaullier
Signed-off-by: Nicolas Gaullier --- tests/fate/mpeg4.mak | 11 +-- 1 file changed, 5 insertions(+), 6 deletions(-) diff --git a/tests/fate/mpeg4.mak b/tests/fate/mpeg4.mak index 9a3cd4e2fb..d6cce97a70 100644 --- a/tests/fate/mpeg4.mak +++ b/tests/fate/mpeg4.mak @@ -3,23 +3,22

[FFmpeg-devel] [PATCH v3 11/21] fate/ac3: fix multiple dependencies

2025-06-30 Thread Nicolas Gaullier
Signed-off-by: Nicolas Gaullier --- tests/fate/ac3.mak | 10 +- 1 file changed, 5 insertions(+), 5 deletions(-) diff --git a/tests/fate/ac3.mak b/tests/fate/ac3.mak index e52678a2fd..30f8b98ff1 100644 --- a/tests/fate/ac3.mak +++ b/tests/fate/ac3.mak @@ -65,17 +65,17 @@ fate-eac3-5: REF

[FFmpeg-devel] [PATCH v3 12/21] fate/cover-art: fix multiple dependencies

2025-06-30 Thread Nicolas Gaullier
Signed-off-by: Nicolas Gaullier --- tests/fate/cover-art.mak | 16 1 file changed, 8 insertions(+), 8 deletions(-) diff --git a/tests/fate/cover-art.mak b/tests/fate/cover-art.mak index faa2182745..7ee9443864 100644 --- a/tests/fate/cover-art.mak +++ b/tests/fate/cover-art.mak

[FFmpeg-devel] [PATCH v3 09/21] fate/aac: fix multiple dependencies

2025-06-30 Thread Nicolas Gaullier
Signed-off-by: Nicolas Gaullier --- tests/fate/aac.mak | 14 +++--- 1 file changed, 7 insertions(+), 7 deletions(-) diff --git a/tests/fate/aac.mak b/tests/fate/aac.mak index f5a41458e6..48a4410d6c 100644 --- a/tests/fate/aac.mak +++ b/tests/fate/aac.mak @@ -241,21 +241,21 @@ fate-aac

[FFmpeg-devel] [PATCH v3 10/21] fate/audio: fix multiple dependencies

2025-06-30 Thread Nicolas Gaullier
Signed-off-by: Nicolas Gaullier --- tests/fate/audio.mak | 12 ++-- 1 file changed, 6 insertions(+), 6 deletions(-) diff --git a/tests/fate/audio.mak b/tests/fate/audio.mak index 6e091760ea..6d3cae8f52 100644 --- a/tests/fate/audio.mak +++ b/tests/fate/audio.mak @@ -5,12 +5,12

[FFmpeg-devel] [PATCH v3 06/21] fate/gapless: fix multiple dependencies

2025-06-30 Thread Nicolas Gaullier
Signed-off-by: Nicolas Gaullier --- tests/fate/gapless.mak | 76 +- 1 file changed, 38 insertions(+), 38 deletions(-) diff --git a/tests/fate/gapless.mak b/tests/fate/gapless.mak index dfae964662..e6fba1436b 100644 --- a/tests/fate/gapless.mak +++ b/tests

[FFmpeg-devel] [PATCH v3 07/21] fate/lavf-container: fix multiple dependencies

2025-06-30 Thread Nicolas Gaullier
Signed-off-by: Nicolas Gaullier --- tests/fate/lavf-container.mak | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/tests/fate/lavf-container.mak b/tests/fate/lavf-container.mak index 6a46f98b5d..395b97b492 100644 --- a/tests/fate/lavf-container.mak +++ b/tests/fate/lavf

[FFmpeg-devel] [PATCH v3 05/21] fate/mov: fix multiple dependencies

2025-06-30 Thread Nicolas Gaullier
Signed-off-by: Nicolas Gaullier --- tests/fate/mov.mak | 38 ++ 1 file changed, 22 insertions(+), 16 deletions(-) diff --git a/tests/fate/mov.mak b/tests/fate/mov.mak index 73deb210cd..8ad8d26144 100644 --- a/tests/fate/mov.mak +++ b/tests/fate/mov.mak

[FFmpeg-devel] [PATCH v3 04/21] fate/all: switch-fix mov muxer dependency to mp4 muxer dependency

2025-06-30 Thread Nicolas Gaullier
Signed-off-by: Nicolas Gaullier --- tests/fate/enc_external.mak | 6 +++--- tests/fate/lavf-container.mak | 2 +- tests/fate/mov.mak| 14 +++--- tests/fate/subtitles.mak | 2 +- 4 files changed, 12 insertions(+), 12 deletions(-) diff --git a/tests/fate

[FFmpeg-devel] [PATCH v3 03/21] fate/demux: fix multiple dependencies

2025-06-30 Thread Nicolas Gaullier
Signed-off-by: Nicolas Gaullier --- tests/fate/demux.mak | 46 ++-- 1 file changed, 23 insertions(+), 23 deletions(-) diff --git a/tests/fate/demux.mak b/tests/fate/demux.mak index a654cdcbbf..0f87abadad 100644 --- a/tests/fate/demux.mak +++ b/tests/fate

[FFmpeg-devel] [PATCH v3 02/21] fate/all: add missing dependencies for extradata bsf

2025-06-30 Thread Nicolas Gaullier
--- tests/fate/concatdec.mak | 2 +- tests/fate/demux.mak | 16 tests/fate/flvenc.mak | 2 +- tests/fate/h264.mak | 8 +--- tests/fate/hevc.mak | 4 ++-- tests/fate/lavf-container.mak | 8 tests/fate/segment.mak|

[FFmpeg-devel] [PATCH v3 01/21] tests/Makefile: make easier to check for multiple dependencies

2025-06-30 Thread Nicolas Gaullier
--- tests/Makefile | 16 1 file changed, 8 insertions(+), 8 deletions(-) diff --git a/tests/Makefile b/tests/Makefile index 505d7f9c6d..7187b744cb 100644 --- a/tests/Makefile +++ b/tests/Makefile @@ -97,21 +97,21 @@ TRANSCODE = $(call ALLYES, $(firstword $(1))_ENCODER $(lastword

[FFmpeg-devel] [PATCH v3 00/21] fate: fix several dependencies

2025-06-30 Thread Nicolas Gaullier
Since v2: - rebased to current master - removed "libavcodec/tests: fix gitignore hashtable" which was applied recently - added two new patches at the end of the serie Nicolas Gaullier (21): tests/Makefile: make easier to check for multiple dependencies fate/all: add missing depend

Re: [FFmpeg-devel] [EXTERNE] Re: [PATCH v2 00/20] fate: fix several dependencies

2025-06-30 Thread Nicolas Gaullier
On 6/27/25 23:25, Michael Niedermayer wrote: Hi On Mon, Jun 23, 2025 at 07:41:32PM +0200, Nicolas Gaullier wrote: For remembering https://ffmpeg.org/pipermail/ffmpeg-devel/2024-October/335422.html This v2 is a "full rebase", which means some problems have been fixed in the while

Re: [FFmpeg-devel] [PATCH] avcodec/aac_parser: fix aac profile probing

2025-06-27 Thread Nicolas Gaullier
On 6/26/25 19:30, Baptiste Coudurier wrote: Hi Nicolas On Jun 26, 2025, at 4:06 AM, Nicolas Gaullier wrote: On 6/17/25 13:41, Nicolas Gaullier wrote: Reverts 64bb91fd3b5a00a8849531c7e8dd207f2a626096 except the key_frame setting for aac. See also 696ea1c2236842572df88d573e24a39be3f19c98

[FFmpeg-devel] [PATCH] avcodec/aac_parser: fix aac profile probing

2025-06-26 Thread Nicolas Gaullier
On 6/17/25 13:41, Nicolas Gaullier wrote: Reverts 64bb91fd3b5a00a8849531c7e8dd207f2a626096 except the key_frame setting for aac. See also 696ea1c2236842572df88d573e24a39be3f19c98. Don't force the profile as it may override the correct value set by the decoder (HE/HEv2). Fixes #11600 Signe

[FFmpeg-devel] [PATCH v2 03/20] fate/demux: fix multiple dependencies

2025-06-23 Thread Nicolas Gaullier
Signed-off-by: Nicolas Gaullier --- tests/fate/demux.mak | 46 ++-- 1 file changed, 23 insertions(+), 23 deletions(-) diff --git a/tests/fate/demux.mak b/tests/fate/demux.mak index a654cdcbbf..0f87abadad 100644 --- a/tests/fate/demux.mak +++ b/tests/fate

[FFmpeg-devel] [PATCH v2 18/20] fate/all: add missing file protocol dependencies

2025-06-23 Thread Nicolas Gaullier
First, always require file protocol when FATE suite is used. Then, add missing dependencies while removing duplicates. Signed-off-by: Nicolas Gaullier --- tests/Makefile | 1 + tests/fate/cbs.mak | 4 ++-- tests/fate/cover-art.mak| 8 tests/fate/demux.mak

[FFmpeg-devel] [PATCH v2 19/20] fate/all: add missing crc/framecrc/md5/framemd5/pipe dependencies

2025-06-23 Thread Nicolas Gaullier
When CMD is crc/framecrc, always use the macros CRC/FRAMECRC, even if it includes unnecessary requirements for rawvideo/pcm_s16le encoders (as actually noticed in a comment of the Makefile). Signed-off-by: Nicolas Gaullier --- tests/Makefile| 3 ++ tests/fate/aac.mak

[FFmpeg-devel] [PATCH v2 10/20] fate/audio: fix multiple dependencies

2025-06-23 Thread Nicolas Gaullier
Signed-off-by: Nicolas Gaullier --- tests/fate/audio.mak | 12 ++-- 1 file changed, 6 insertions(+), 6 deletions(-) diff --git a/tests/fate/audio.mak b/tests/fate/audio.mak index 6e091760ea..6d3cae8f52 100644 --- a/tests/fate/audio.mak +++ b/tests/fate/audio.mak @@ -5,12 +5,12

[FFmpeg-devel] [PATCH v2 20/20] libavcodec/tests: fix gitignore hashtable

2025-06-23 Thread Nicolas Gaullier
Signed-off-by: Nicolas Gaullier --- libavcodec/tests/.gitignore | 1 + 1 file changed, 1 insertion(+) diff --git a/libavcodec/tests/.gitignore b/libavcodec/tests/.gitignore index 29020a5945..6bd9b31b56 100644 --- a/libavcodec/tests/.gitignore +++ b/libavcodec/tests/.gitignore @@ -9,6 +9,7

[FFmpeg-devel] [PATCH v2 09/20] fate/aac: fix multiple dependencies

2025-06-23 Thread Nicolas Gaullier
Signed-off-by: Nicolas Gaullier --- tests/fate/aac.mak | 14 +++--- 1 file changed, 7 insertions(+), 7 deletions(-) diff --git a/tests/fate/aac.mak b/tests/fate/aac.mak index f5a41458e6..48a4410d6c 100644 --- a/tests/fate/aac.mak +++ b/tests/fate/aac.mak @@ -241,21 +241,21 @@ fate-aac

[FFmpeg-devel] [PATCH v2 08/20] fate/vorbis: fix multiple dependencies

2025-06-23 Thread Nicolas Gaullier
Signed-off-by: Nicolas Gaullier --- tests/fate/vorbis.mak | 24 +--- 1 file changed, 13 insertions(+), 11 deletions(-) diff --git a/tests/fate/vorbis.mak b/tests/fate/vorbis.mak index 781430bcd3..73e837f5a6 100644 --- a/tests/fate/vorbis.mak +++ b/tests/fate/vorbis.mak

[FFmpeg-devel] [PATCH v2 06/20] fate/gapless: fix multiple dependencies

2025-06-23 Thread Nicolas Gaullier
Signed-off-by: Nicolas Gaullier --- tests/fate/gapless.mak | 76 +- 1 file changed, 38 insertions(+), 38 deletions(-) diff --git a/tests/fate/gapless.mak b/tests/fate/gapless.mak index dfae964662..e6fba1436b 100644 --- a/tests/fate/gapless.mak +++ b/tests

[FFmpeg-devel] [PATCH v2 05/20] fate/mov: fix multiple dependencies

2025-06-23 Thread Nicolas Gaullier
Signed-off-by: Nicolas Gaullier --- tests/fate/mov.mak | 38 ++ 1 file changed, 22 insertions(+), 16 deletions(-) diff --git a/tests/fate/mov.mak b/tests/fate/mov.mak index a21b024015..8d4077da66 100644 --- a/tests/fate/mov.mak +++ b/tests/fate/mov.mak

[FFmpeg-devel] [PATCH v2 11/20] fate/ac3: fix multiple dependencies

2025-06-23 Thread Nicolas Gaullier
Signed-off-by: Nicolas Gaullier --- tests/fate/ac3.mak | 10 +- 1 file changed, 5 insertions(+), 5 deletions(-) diff --git a/tests/fate/ac3.mak b/tests/fate/ac3.mak index e52678a2fd..30f8b98ff1 100644 --- a/tests/fate/ac3.mak +++ b/tests/fate/ac3.mak @@ -65,17 +65,17 @@ fate-eac3-5: REF

[FFmpeg-devel] [PATCH v2 17/20] fate/all: fix multiple dependencies

2025-06-23 Thread Nicolas Gaullier
Signed-off-by: Nicolas Gaullier --- tests/fate/api.mak | 2 +- tests/fate/cbs.mak | 2 +- tests/fate/ffmpeg.mak | 6 +++--- tests/fate/filter-audio.mak | 8 tests/fate/filter-video.mak | 2 +- tests/fate/fits.mak | 4 ++-- tests/fate/gif.mak | 2

[FFmpeg-devel] [PATCH v2 15/20] fate/pcm: fix multiple dependencies

2025-06-23 Thread Nicolas Gaullier
Signed-off-by: Nicolas Gaullier --- tests/fate/pcm.mak | 12 ++-- 1 file changed, 6 insertions(+), 6 deletions(-) diff --git a/tests/fate/pcm.mak b/tests/fate/pcm.mak index b10233a9eb..75636118b2 100644 --- a/tests/fate/pcm.mak +++ b/tests/fate/pcm.mak @@ -1,4 +1,4 @@ -FATE_SAMPLES_PCM

[FFmpeg-devel] [PATCH v2 16/20] fate/hevc: fix multiple dependencies

2025-06-23 Thread Nicolas Gaullier
Signed-off-by: Nicolas Gaullier --- tests/fate/hevc.mak | 8 1 file changed, 4 insertions(+), 4 deletions(-) diff --git a/tests/fate/hevc.mak b/tests/fate/hevc.mak index 3ad6e255a9..7b22e36a3d 100644 --- a/tests/fate/hevc.mak +++ b/tests/fate/hevc.mak @@ -219,7 +219,7 @@ FATE_HEVC

[FFmpeg-devel] [PATCH v2 14/20] fate/mpeg4: fix multiple dependencies

2025-06-23 Thread Nicolas Gaullier
Signed-off-by: Nicolas Gaullier --- tests/fate/mpeg4.mak | 11 +-- 1 file changed, 5 insertions(+), 6 deletions(-) diff --git a/tests/fate/mpeg4.mak b/tests/fate/mpeg4.mak index 9a3cd4e2fb..d6cce97a70 100644 --- a/tests/fate/mpeg4.mak +++ b/tests/fate/mpeg4.mak @@ -3,23 +3,22

[FFmpeg-devel] [PATCH v2 12/20] fate/cover-art: fix multiple dependencies

2025-06-23 Thread Nicolas Gaullier
Signed-off-by: Nicolas Gaullier --- tests/fate/cover-art.mak | 16 1 file changed, 8 insertions(+), 8 deletions(-) diff --git a/tests/fate/cover-art.mak b/tests/fate/cover-art.mak index faa2182745..7ee9443864 100644 --- a/tests/fate/cover-art.mak +++ b/tests/fate/cover-art.mak

[FFmpeg-devel] [PATCH v2 13/20] fate/hlsenc: fix multiple dependencies

2025-06-23 Thread Nicolas Gaullier
Signed-off-by: Nicolas Gaullier --- tests/fate/hlsenc.mak | 18 +- 1 file changed, 9 insertions(+), 9 deletions(-) diff --git a/tests/fate/hlsenc.mak b/tests/fate/hlsenc.mak index 98341b2b81..72c75bec3a 100644 --- a/tests/fate/hlsenc.mak +++ b/tests/fate/hlsenc.mak @@ -5,7 +5,7

[FFmpeg-devel] [PATCH v2 07/20] fate/lavf-container: fix multiple dependencies

2025-06-23 Thread Nicolas Gaullier
Signed-off-by: Nicolas Gaullier --- tests/fate/lavf-container.mak | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/tests/fate/lavf-container.mak b/tests/fate/lavf-container.mak index 6a46f98b5d..395b97b492 100644 --- a/tests/fate/lavf-container.mak +++ b/tests/fate/lavf

[FFmpeg-devel] [PATCH v2 04/20] fate/all: switch-fix mov muxer dependency to mp4 muxer dependency

2025-06-23 Thread Nicolas Gaullier
Signed-off-by: Nicolas Gaullier --- tests/fate/enc_external.mak | 6 +++--- tests/fate/lavf-container.mak | 2 +- tests/fate/mov.mak| 14 +++--- tests/fate/subtitles.mak | 2 +- 4 files changed, 12 insertions(+), 12 deletions(-) diff --git a/tests/fate

[FFmpeg-devel] [PATCH v2 02/20] fate/all: add missing dependencies for extradata bsf

2025-06-23 Thread Nicolas Gaullier
--- tests/fate/concatdec.mak | 2 +- tests/fate/demux.mak | 16 tests/fate/flvenc.mak | 2 +- tests/fate/h264.mak | 8 +--- tests/fate/hevc.mak | 4 ++-- tests/fate/lavf-container.mak | 8 tests/fate/segment.mak|

[FFmpeg-devel] [PATCH v2 01/20] tests/Makefile: make easier to check for multiple dependencies

2025-06-23 Thread Nicolas Gaullier
--- tests/Makefile | 16 1 file changed, 8 insertions(+), 8 deletions(-) diff --git a/tests/Makefile b/tests/Makefile index 505d7f9c6d..7187b744cb 100644 --- a/tests/Makefile +++ b/tests/Makefile @@ -97,21 +97,21 @@ TRANSCODE = $(call ALLYES, $(firstword $(1))_ENCODER $(lastword

[FFmpeg-devel] [PATCH v2 00/20] fate: fix several dependencies

2025-06-23 Thread Nicolas Gaullier
For remembering https://ffmpeg.org/pipermail/ffmpeg-devel/2024-October/335422.html This v2 is a "full rebase", which means some problems have been fixed in the while (not many), some others are new. Nicolas Gaullier (20): tests/Makefile: make easier to check for multiple dependenc

Re: [FFmpeg-devel] [EXTERNE] Re: [PATCH 1/2] fftools/textformat: fix print 64 bit integers

2025-06-20 Thread Nicolas Gaullier
On 6/20/25 13:29, Marvin Scholz wrote: On 20 Jun 2025, at 13:07, Nicolas Gaullier wrote: Regression in ffprobe since textformat introduction in d7a3f68feae0b1c3718f9d2671c6d41c60a40680. Fixes #11638 Signed-off-by: Nicolas Gaullier --- fftools/textformat/avtextformat.c | 2 +- fftools

[FFmpeg-devel] [PATCH 2/2] fftools/textformat: renamings in print_unit_int for consistency (cosmetic)

2025-06-20 Thread Nicolas Gaullier
Signed-off-by: Nicolas Gaullier --- fftools/ffprobe.c | 2 +- fftools/graph/graphprint.c| 2 +- fftools/textformat/avtextformat.c | 4 ++-- fftools/textformat/avtextformat.h | 2 +- 4 files changed, 5 insertions(+), 5 deletions(-) diff --git a/fftools/ffprobe.c b/fftools

[FFmpeg-devel] [PATCH 1/2] fftools/textformat: fix print 64 bit integers

2025-06-20 Thread Nicolas Gaullier
Regression in ffprobe since textformat introduction in d7a3f68feae0b1c3718f9d2671c6d41c60a40680. Fixes #11638 Signed-off-by: Nicolas Gaullier --- fftools/textformat/avtextformat.c | 2 +- fftools/textformat/avtextformat.h | 2 +- 2 files changed, 2 insertions(+), 2 deletions(-) diff --git a

[FFmpeg-devel] [PATCH] avcodec/aac_parser: fix aac profile probing

2025-06-17 Thread Nicolas Gaullier
Reverts 64bb91fd3b5a00a8849531c7e8dd207f2a626096 except the key_frame setting for aac. See also 696ea1c2236842572df88d573e24a39be3f19c98. Don't force the profile as it may override the correct value set by the decoder (HE/HEv2). Fixes #11600 Signed-off-by: Nicolas Gaullier --- libav

Re: [FFmpeg-devel] [PATCH 6/7] avformat: add s337m support in mpegts, wav and mxf stereo tracks

2025-02-06 Thread Nicolas Gaullier
>De : ffmpeg-devel de la part de Tomas Härdin > >Envoyé : mercredi 5 février 2025 15:47 >Objet : Re: [FFmpeg-devel] [PATCH 6/7] avformat: add s337m support in mpegts, >wav and mxf stereo tracks > >Looks OK > >Sorry for the late review > >/Tomas Thanks to you, good to know; nevertheless, the pat

Re: [FFmpeg-devel] [PATCH] fate: Add a dependency on ffprobe for fate-flcl1905

2024-12-12 Thread Nicolas Gaullier
>De : ffmpeg-devel de la part de Ramiro Polla > >Envoyé : jeudi 12 décembre 2024 15:56 >On Thu, Dec 12, 2024 at 1:28 PM Martin Storsjö wrote: >> >> This fixes occasional failed tests, if doing "make fate" without a >> regular "make" or "make all" inbetween. > >I've had something similar locally

Re: [FFmpeg-devel] [PATCH 0/7] RFC: complete rework of s337m support

2024-12-12 Thread Nicolas Gaullier
>De : ffmpeg-devel de la part de Anton >Khirnov >Envoyé : jeudi 12 décembre 2024 09:36 > >I am not intimately familiar with these formats and their mutual >relationship, so these problems you refer to are not clear to me. >What I am familiar with is our APIs, and from that position I can say >t

Re: [FFmpeg-devel] [PATCH 5/7] avformat/mpegts: add s337m support

2024-12-12 Thread Nicolas Gaullier
>De : ffmpeg-devel de la part de Michael >Niedermayer >Envoyé : lundi 9 décembre 2024 04:55 > >/usr/bin/ld: libavformat/libavformat.so: undefined reference to `ff_reverse' > >thx Sorry, I forgot to check build against --enable-shared... Thanks to you. I can fix this in a new patch serie, but th

Re: [FFmpeg-devel] [PATCH 0/7] RFC: complete rework of s337m support

2024-12-06 Thread Nicolas Gaullier
>De : ffmpeg-devel de la part de Kieran >Kunhya via ffmpeg-devel >Envoyé : jeudi 5 décembre 2024 19:31 > >> I resample the decoded data. The s337m decoder requires the dolby_e decoder >> (same way the current ftr decoder requires the aac decoder, so should not be >> an issue). > >Why not just s

Re: [FFmpeg-devel] [PATCH 0/7] RFC: complete rework of s337m support

2024-12-06 Thread Nicolas Gaullier
>De : Anton Khirnov >Envoyé : jeudi 5 décembre 2024 15:06 >Hi, >I believe a similar approach was tried by Gyan earlier this year and >unanimously rejected by the TC [1-3]. > >[1] >https://patchwork.ffmpeg.org/project/ffmpeg/patch/20240127103854.9971-1-ffm...@gyani.pro/ >[2] Message-Id <9be400cf-9

Re: [FFmpeg-devel] [PATCH 0/7] RFC: complete rework of s337m support

2024-12-06 Thread Nicolas Gaullier
>De : ffmpeg-devel de la part de Lynne via >ffmpeg-devel >Envoyé : jeudi 5 décembre 2024 15:15 >I strongly object to using a resampler here. We *never* covert audio or >video in decoders from the native codec samplerate or pixel format. >This includes Opus. Opus carries the original samplerate o

Re: [FFmpeg-devel] [PATCH 0/7] RFC: complete rework of s337m support

2024-12-05 Thread Nicolas Gaullier
>De : Kieran Kunhya >Envoyé : mercredi 4 décembre 2024 23:06 > >> - a s337m decoder: it includes a resampler: output and input sample_rate >> are the same, sync is always correct. It would be possible to implement >> a full pcm fallback, but currently only a silence/pcm fallback is >> provided. A

Re: [FFmpeg-devel] [PATCH] avcodec/ac3dec: fix build when eac3 decoder is disabled

2024-11-04 Thread Nicolas Gaullier
>Envoyé : dimanche 27 octobre 2024 01:01 >Objet : [PATCH] avcodec/ac3dec: fix build when eac3 decoder is disabled > > libavcodec/ac3dec_fixed.c | 3 +++ > libavcodec/ac3dec_float.c | 2 ++ > 2 files changed, 5 insertions(+) Ping ? Maybe I should detail a little bit more about the "actual" issue in

[FFmpeg-devel] [PATCH 11/19] fate/ac3: fix multiple dependencies

2024-10-26 Thread Nicolas Gaullier
Signed-off-by: Nicolas Gaullier --- tests/fate/ac3.mak | 10 +- 1 file changed, 5 insertions(+), 5 deletions(-) diff --git a/tests/fate/ac3.mak b/tests/fate/ac3.mak index 1ecb5a3f54..714ae31990 100644 --- a/tests/fate/ac3.mak +++ b/tests/fate/ac3.mak @@ -65,17 +65,17 @@ fate-eac3-5: REF

[FFmpeg-devel] [PATCH 03/19] fate/demux: fix multiple dependencies

2024-10-26 Thread Nicolas Gaullier
Signed-off-by: Nicolas Gaullier --- tests/fate/demux.mak | 42 +- 1 file changed, 21 insertions(+), 21 deletions(-) diff --git a/tests/fate/demux.mak b/tests/fate/demux.mak index 24306caa11..fcf4e1664d 100644 --- a/tests/fate/demux.mak +++ b/tests/fate

[FFmpeg-devel] [PATCH 04/19] fate/audio: fix ffprobe dependency instead of ffmpeg

2024-10-26 Thread Nicolas Gaullier
Signed-off-by: Nicolas Gaullier --- tests/fate/audio.mak | 6 -- 1 file changed, 4 insertions(+), 2 deletions(-) diff --git a/tests/fate/audio.mak b/tests/fate/audio.mak index 4d6c479b6b..01db30f7b0 100644 --- a/tests/fate/audio.mak +++ b/tests/fate/audio.mak @@ -82,10 +82,12 @@ fate

[FFmpeg-devel] [PATCH 12/19] fate/audio: fix multiple dependencies

2024-10-26 Thread Nicolas Gaullier
Signed-off-by: Nicolas Gaullier --- tests/fate/audio.mak | 12 ++-- 1 file changed, 6 insertions(+), 6 deletions(-) diff --git a/tests/fate/audio.mak b/tests/fate/audio.mak index 01db30f7b0..9dfdfe0702 100644 --- a/tests/fate/audio.mak +++ b/tests/fate/audio.mak @@ -5,12 +5,12

[FFmpeg-devel] [PATCH 10/19] fate/aac: fix multiple dependencies

2024-10-26 Thread Nicolas Gaullier
Signed-off-by: Nicolas Gaullier --- tests/fate/aac.mak | 14 +++--- 1 file changed, 7 insertions(+), 7 deletions(-) diff --git a/tests/fate/aac.mak b/tests/fate/aac.mak index ff58392ad9..de7b912e20 100644 --- a/tests/fate/aac.mak +++ b/tests/fate/aac.mak @@ -251,21 +251,21 @@ fate-aac

[FFmpeg-devel] [PATCH 09/19] fate/vorbis: fix multiple dependencies

2024-10-26 Thread Nicolas Gaullier
Signed-off-by: Nicolas Gaullier --- tests/fate/vorbis.mak | 24 +--- 1 file changed, 13 insertions(+), 11 deletions(-) diff --git a/tests/fate/vorbis.mak b/tests/fate/vorbis.mak index 781430bcd3..73e837f5a6 100644 --- a/tests/fate/vorbis.mak +++ b/tests/fate/vorbis.mak

[FFmpeg-devel] [PATCH 05/19] fate/all: switch-fix mov muxer dependency to mp4 muxer dependency

2024-10-26 Thread Nicolas Gaullier
Signed-off-by: Nicolas Gaullier --- tests/fate/enc_external.mak | 6 +++--- tests/fate/lavf-container.mak | 12 ++-- tests/fate/mov.mak| 14 +++--- tests/fate/subtitles.mak | 2 +- 4 files changed, 17 insertions(+), 17 deletions(-) diff --git a/tests/fate

[FFmpeg-devel] [PATCH 08/19] fate/lavf-container: fix multiple dependencies

2024-10-26 Thread Nicolas Gaullier
Signed-off-by: Nicolas Gaullier --- tests/fate/lavf-container.mak | 28 ++-- 1 file changed, 14 insertions(+), 14 deletions(-) diff --git a/tests/fate/lavf-container.mak b/tests/fate/lavf-container.mak index db8c315d08..5d9e769458 100644 --- a/tests/fate/lavf

[FFmpeg-devel] [PATCH 06/19] fate/mov: fix multiple dependencies

2024-10-26 Thread Nicolas Gaullier
Signed-off-by: Nicolas Gaullier --- tests/fate/mov.mak | 38 ++ 1 file changed, 22 insertions(+), 16 deletions(-) diff --git a/tests/fate/mov.mak b/tests/fate/mov.mak index 68b1e4f30b..78da66a6f7 100644 --- a/tests/fate/mov.mak +++ b/tests/fate/mov.mak

[FFmpeg-devel] [PATCH 07/19] fate/gapless: fix multiple dependencies

2024-10-26 Thread Nicolas Gaullier
Signed-off-by: Nicolas Gaullier --- tests/fate/gapless.mak | 76 +- 1 file changed, 38 insertions(+), 38 deletions(-) diff --git a/tests/fate/gapless.mak b/tests/fate/gapless.mak index dfae964662..e6fba1436b 100644 --- a/tests/fate/gapless.mak +++ b/tests

[FFmpeg-devel] [PATCH 19/19] fate/all: add missing crc/framecrc/md5/framemd5/pipe dependencies

2024-10-26 Thread Nicolas Gaullier
When CMD is crc/framecrc, always use the macros CRC/FRAMECRC, even if it includes unnecessary requirements for rawvideo/pcm_s16le encoders (as actually noticed in a comment of the Makefile). Signed-off-by: Nicolas Gaullier --- tests/Makefile| 3 ++ tests/fate/aac.mak

[FFmpeg-devel] [PATCH 17/19] fate/all: fix multiple dependencies

2024-10-26 Thread Nicolas Gaullier
Signed-off-by: Nicolas Gaullier --- tests/fate/api.mak | 2 +- tests/fate/ffmpeg.mak | 6 +++--- tests/fate/filter-audio.mak | 8 tests/fate/filter-video.mak | 2 +- tests/fate/fits.mak | 4 ++-- tests/fate/gif.mak | 2 +- tests/fate/h264.mak | 6

[FFmpeg-devel] [PATCH 18/19] fate/all: add missing file protocol dependencies

2024-10-26 Thread Nicolas Gaullier
First, always require file protocol when FATE suite is used. Then, add missing dependencies while removing duplicates. Signed-off-by: Nicolas Gaullier --- tests/Makefile | 1 + tests/fate/audio.mak| 2 +- tests/fate/cbs.mak | 4 ++-- tests/fate/cover-art.mak

[FFmpeg-devel] [PATCH 15/19] fate/pcm: fix multiple dependencies

2024-10-26 Thread Nicolas Gaullier
Signed-off-by: Nicolas Gaullier --- tests/fate/pcm.mak | 12 ++-- 1 file changed, 6 insertions(+), 6 deletions(-) diff --git a/tests/fate/pcm.mak b/tests/fate/pcm.mak index b10233a9eb..75636118b2 100644 --- a/tests/fate/pcm.mak +++ b/tests/fate/pcm.mak @@ -1,4 +1,4 @@ -FATE_SAMPLES_PCM

[FFmpeg-devel] [PATCH 16/19] fate/hevc: fix multiple dependencies

2024-10-26 Thread Nicolas Gaullier
Signed-off-by: Nicolas Gaullier --- tests/fate/hevc.mak | 8 1 file changed, 4 insertions(+), 4 deletions(-) diff --git a/tests/fate/hevc.mak b/tests/fate/hevc.mak index 285348cc74..4c8af0e8ca 100644 --- a/tests/fate/hevc.mak +++ b/tests/fate/hevc.mak @@ -219,7 +219,7 @@ FATE_HEVC

[FFmpeg-devel] [PATCH 13/19] fate/cover-art: fix multiple dependencies

2024-10-26 Thread Nicolas Gaullier
Signed-off-by: Nicolas Gaullier --- tests/fate/cover-art.mak | 16 tests/fate/hlsenc.mak| 18 +- 2 files changed, 17 insertions(+), 17 deletions(-) diff --git a/tests/fate/cover-art.mak b/tests/fate/cover-art.mak index faa2182745..7ee9443864 100644 --- a

[FFmpeg-devel] [PATCH 14/19] fate/mpeg4: fix multiple dependencies

2024-10-26 Thread Nicolas Gaullier
Signed-off-by: Nicolas Gaullier --- tests/fate/mpeg4.mak | 11 +-- 1 file changed, 5 insertions(+), 6 deletions(-) diff --git a/tests/fate/mpeg4.mak b/tests/fate/mpeg4.mak index 9a3cd4e2fb..d6cce97a70 100644 --- a/tests/fate/mpeg4.mak +++ b/tests/fate/mpeg4.mak @@ -3,23 +3,22

[FFmpeg-devel] [PATCH 02/19] fate/all: add missing dependencies for extradata bsf

2024-10-26 Thread Nicolas Gaullier
--- tests/fate/concatdec.mak | 2 +- tests/fate/demux.mak | 16 tests/fate/flvenc.mak | 2 +- tests/fate/h264.mak | 8 +--- tests/fate/hevc.mak | 4 ++-- tests/fate/lavf-container.mak | 8 tests/fate/segment.mak|

[FFmpeg-devel] [PATCH 00/19] fate: fix several dependencies

2024-10-26 Thread Nicolas Gaullier
I don't know how I should split the patches. I decided to gather some of them together to limit the number of patches, but maybe I should split strictly one/mak file, let me know. Nicolas Gaullier (19): tests/Makefile: make easier to check for multiple dependencies fate/all: add missing

[FFmpeg-devel] [PATCH 01/19] tests/Makefile: make easier to check for multiple dependencies

2024-10-26 Thread Nicolas Gaullier
--- tests/Makefile | 16 1 file changed, 8 insertions(+), 8 deletions(-) diff --git a/tests/Makefile b/tests/Makefile index 9b70145015..37b445d864 100644 --- a/tests/Makefile +++ b/tests/Makefile @@ -97,21 +97,21 @@ TRANSCODE = $(call ALLYES, $(firstword $(1))_ENCODER $(lastword

[FFmpeg-devel] [PATCH] avcodec/ac3dec: fix build when eac3 decoder is disabled

2024-10-26 Thread Nicolas Gaullier
Signed-off-by: Nicolas Gaullier --- libavcodec/ac3dec_fixed.c | 3 +++ libavcodec/ac3dec_float.c | 2 ++ 2 files changed, 5 insertions(+) diff --git a/libavcodec/ac3dec_fixed.c b/libavcodec/ac3dec_fixed.c index c9e5cda69c..e0db9b2260 100644 --- a/libavcodec/ac3dec_fixed.c +++ b/libavcodec

Re: [FFmpeg-devel] [PATCH 4/4] fate/all: add missing crc/framecrc/md5/framemd5/pipe dependencies

2024-10-10 Thread Nicolas Gaullier
>>-FATE_SAMPLES_DEMUX-$(call ALLYES, S337M_DEMUXER DOLBY_E_PARSER >>FRAMECRC_MUXER) += fate-s337m-demux >>+FATE_SAMPLES_DEMUX-$(call FRAMECRC, S337M,, DOLBY_E_PARSER >>+FRAMECRC_MUXER) += fate-s337m-demux >> fate-s337m-demux: CMD = framecrc -i $(TARGET_SAMPLES)/dolby_e/16-11 -c >> copy -ss 2 -t

Re: [FFmpeg-devel] [PATCH 4/4] fate/all: add missing crc/framecrc/md5/framemd5/pipe dependencies

2024-10-10 Thread Nicolas Gaullier
>De : Nicolas Gaullier >Envoyé : mercredi 9 octobre 2024 22:30 > >diff --git a/tests/fate/demux.mak b/tests/fate/demux.mak >index c16b540d2f..fe5073397e 100644 >--- a/tests/fate/demux.mak >+++ b/tests/fate/demux.mak >@@ -1,82 +1,82 @@ >(..) >-FATE_SAMPLES_DEMU

[FFmpeg-devel] [PATCH 4/4] fate/all: add missing crc/framecrc/md5/framemd5/pipe dependencies

2024-10-09 Thread Nicolas Gaullier
Signed-off-by: Nicolas Gaullier --- tests/Makefile| 7 ++- tests/fate/aac.mak| 12 ++--- tests/fate/ac3.mak| 8 ++-- tests/fate/acodec.mak | 4 +- tests/fate/adpcm.mak | 38 +++ tests/fate/alac.mak | 2 +- tests

[FFmpeg-devel] [PATCH 3/4] fate/filter-audio: add missing flv nellymoser dependencies

2024-10-09 Thread Nicolas Gaullier
Signed-off-by: Nicolas Gaullier --- tests/fate/filter-audio.mak | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/tests/fate/filter-audio.mak b/tests/fate/filter-audio.mak index 643dffad67..903d293507 100644 --- a/tests/fate/filter-audio.mak +++ b/tests/fate/filter-audio.mak

[FFmpeg-devel] [PATCH 2/4] fate/demux: fix multiple dependencies

2024-10-09 Thread Nicolas Gaullier
Signed-off-by: Nicolas Gaullier --- tests/fate/demux.mak | 8 1 file changed, 4 insertions(+), 4 deletions(-) diff --git a/tests/fate/demux.mak b/tests/fate/demux.mak index f3f34d1126..c16b540d2f 100644 --- a/tests/fate/demux.mak +++ b/tests/fate/demux.mak @@ -79,10 +79,10 @@ fate-mlv

[FFmpeg-devel] [PATCH 1/4] fate/all: add missing file protocol dependencies

2024-10-09 Thread Nicolas Gaullier
Signed-off-by: Nicolas Gaullier --- tests/Makefile | 1 + tests/fate/audio.mak| 2 +- tests/fate/cbs.mak | 4 ++-- tests/fate/cover-art.mak| 8 tests/fate/demux.mak| 2 +- tests/fate/filter-audio.mak | 10 +- tests/fate/filter

[FFmpeg-devel] [PATCH 0/4] fate: add missing dependencies

2024-10-09 Thread Nicolas Gaullier
I checked: fate-list unchanged with basic setup --disable-protocol=file pass --disable-protocol=pipe pass --disable-muxer=crc pass --disable-muxer=framecrc pass --disable-muxer=md5 pass --disable-muxer=framemd5 pass So I shouldn't have made any major mistake. Nicolas Gaullier (4): fate/all

[FFmpeg-devel] [PATCH 0/1] avfilter/framesync: fix forward EOF pts

2024-10-04 Thread Nicolas Gaullier
Pinging again... post rebased to current master, issue still present and fixed with the patch. Nicolas Gaullier (1): avfilter/framesync: fix forward EOF pts libavfilter/framesync.c | 23 +++ 1 file changed, 11 insertions(+), 12 deletions(-) -- 2.30.2

[FFmpeg-devel] [PATCH 1/1] avfilter/framesync: fix forward EOF pts

2024-10-04 Thread Nicolas Gaullier
Note1: when the EOF pts is not accurate enough, the last frame can be dropped by vf_fps with default rounding. Note2: vf_scale use framesync since e82a3997cdd6c0894869b33ba42430ac3, so this is a very commonplace scenario. For example: ./ffprobe -f lavfi testsrc=d=1,scale,fps -of flat \ -count_f

Re: [FFmpeg-devel] [PATCH] libavformat/mxfdec.c: Recognize and Ignore MXF fill boxes

2024-09-13 Thread Nicolas Gaullier
>De : ffmpeg-devel De la part de martin >schitter >Envoyé : jeudi 12 septembre 2024 13:54 >On 12.09.24 13:14, Nicolas Gaullier wrote: >> The message "Recognize and Ignore" does not make it clear what issue or >> grave defect is solved here. >> I see i

  1   2   3   4   5   >