Re: [FFmpeg-devel] [PATCH] libavformat/usmdec: add support for HCA stream decryption

Hello, Here's a sample file for testing: https://streams.videolan.org/ffmpeg/incoming/encrypted-hca_lowkey-3201512.usm Use -hca_lowkey 3201512 for decryption. One way to test would be the following command: ffmpeg -hca_lowkey 3201512 -i encrypted-hca_lowkey-3201512.usm -filter_complex \ "[0:a]ame

Re: [FFmpeg-devel] [PATCH] fftools/ffmpeg_filter: at EOF, unref frame used for FPS conversion logic

> On Jun 24, 2025, at 11:09, Steven Zhou wrote: > > fftools/ffmpeg_filter: at EOF, unref frame used for FPS conversion logic > > Ever since commit 64f3feb added `avcodec_free_context(&enc->enc_ctx);` to > `enc_free()` the encoder context is closed before filtergraph output context. > The filt

[FFmpeg-devel] [PATCH] fftools/ffmpeg_filter: at EOF, unref frame used for FPS conversion logic

fftools/ffmpeg_filter: at EOF, unref frame used for FPS conversion logic Ever since commit 64f3feb added `avcodec_free_context(&enc->enc_ctx);` to `enc_free()` the encoder context is closed before filtergraph output context. The filtergraph output context holds a reference to last AVFrame to pass

Re: [FFmpeg-devel] [PATCH] Revert "Add FUNDING.json"

Hi Derek You said "The only mails you'll see from me in the future are code or reviews." On Mon, Jun 23, 2025 at 09:20:56PM +0100, Derek Buitenhuis wrote: > This was pushed with outstanding reservations, there where 2 comments in one mail but no objection The first was about SPI and taxes

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

--- Begin Message --- Apologies for the late reply. Yeah, the other solution to solve this problem is to sprinkle .gotpcrel all over the x86 assembly whenever it tries to access a variable that's defined in the C file. Using `attribute(small)`, as in this patch, is behind a macro that implies tha

[FFmpeg-devel] [PATCH v8 3/3] fftools/resources: Update .gitignore

From: softworkz Signed-off-by: softworkz --- fftools/resources/.gitignore | 5 ++--- 1 file changed, 2 insertions(+), 3 deletions(-) diff --git a/fftools/resources/.gitignore b/fftools/resources/.gitignore index bda2c59a1c..cc742817b5 100644 --- a/fftools/resources/.gitignore +++ b/fftools/res

[FFmpeg-devel] [PATCH v8 2/3] ffbuild/commonmak: Consolidate pattern rules for compression

From: softworkz This commit simplifies and consolidates all the rules around ptx and resource file compression. Signed-off-by: softworkz --- ffbuild/common.mak | 43 +-- 1 file changed, 17 insertions(+), 26 deletions(-) diff --git a/ffbuild/common.mak b

[FFmpeg-devel] [PATCH v8 1/3] fftools/resources: Fix double-build by disabling .d file generation

From: softworkz Signed-off-by: softworkz --- ffbuild/common.mak | 6 ++ fftools/Makefile | 1 + fftools/resources/Makefile | 12 +--- 3 files changed, 12 insertions(+), 7 deletions(-) diff --git a/ffbuild/common.mak b/ffbuild/common.mak index ddf48923ea..81e8a46

[FFmpeg-devel] [PATCH v8 0/3] ffbuild/commonmak: Fix rebuild check with implicit rule chains

V2 == * Fix MSVC build (use the universal command pattern) V3 == * Skip dependency generation by clearing CC_DEPS instead (as suggested by Ramiro - thanks!) V4 == * Always keep .ptx files (as suggested by Timo - thanks) Tested all scenarios: * .ptx.c and .ptx.gz still get de

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

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/d

[FFmpeg-devel] [PATCH v3] lavc: add an av1_vulkan encoder

This commit adds a Vulkan AV1 encoder, using the native acceleration API. --- configure |1 + libavcodec/Makefile|2 + libavcodec/allcodecs.c |1 + libavcodec/vulkan_encode_av1.c | 1409 libavcodec/vulkan_video.c

[FFmpeg-devel] [PATCH] tools: Remove libav-merge-next-commit

Libav is no longer extant. Signed-off-by: Derek Buitenhuis --- tools/libav-merge-next-commit | 22 -- 1 file changed, 22 deletions(-) delete mode 100755 tools/libav-merge-next-commit diff --git a/tools/libav-merge-next-commit b/tools/libav-merge-next-commit deleted file mod

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

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

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

Re: [FFmpeg-devel] [PATCH] avformat/demux: Fix segfault due to avcodec_open2 failure (v2)

ping ... as this is a segfault fix for `ffplay/ffprobe -i 1_poc.mp4` I think it deserves some review/approval priority. On Mon, Jun 9, 2025 at 9:45 PM Pavel Koshevoy wrote: > Fixes 'ffprobe 1_poc.mp4' segfault introduced with > commit 0021484d05f9b0f032fa319399de6e24eea0c04f > > codec_close shou

[FFmpeg-devel] [PATCH] Revert "Add FUNDING.json"

This was pushed with outstanding reservations, with no positive affirmation or review from any other devs, and without a push noticed to say it was going to be pushed. Further, the wallet is controlled by a single individual, and not FFmpeg. This reverts commit a8c21a7158fa6d41c332da7ccd203ea4057

[FFmpeg-devel] [PATCH] doc: Remove libav-merge.txt

He's dead, Jim. Signed-off-by: Derek Buitenhuis --- doc/libav-merge.txt | 115 1 file changed, 115 deletions(-) delete mode 100644 doc/libav-merge.txt diff --git a/doc/libav-merge.txt b/doc/libav-merge.txt deleted file mode 100644 index bcd0aacba5..

[FFmpeg-devel] [PATCH] hwcontext_vulkan: add option to disable video queues when creating a device

This helps reduce queue usage even further. --- libavutil/hwcontext_vulkan.c | 22 -- 1 file changed, 16 insertions(+), 6 deletions(-) diff --git a/libavutil/hwcontext_vulkan.c b/libavutil/hwcontext_vulkan.c index 88145ef2d0..f31020c416 100644 --- a/libavutil/hwcontext_vulkan.

[FFmpeg-devel] [PATCH] avfilter/f_ebur128: properly propagate true peak

From: Niklas Haas After 3b26b782ee, `ebur128->true_peak` was only set to the maximum of the current "true peak per frame" values, when it should report the true peak for the entire stream. Fixes: 3b26b782eeded9b9ab7fac013cd1a83a30d68206 --- libavfilter/f_ebur128.c | 4 +++- 1 file changed, 3 in

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

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 @@ FATE_SA

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

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 @@ /c

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

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-la

[FFmpeg-devel] [PATCH] avformat/iamf_parse: prevent overreads in update_extradata

Fixes: libavcodec/put_bits.h:232:32: runtime error: shift exponent -19 is negative Fixes: Assertion n>=0 && n<=32 failed at ./libavcodec/get_bits.h:406 Fixes: 398527871/clusterfuzz-testcase-minimized-ffmpeg_dem_IAMF_fuzzer-6602025714647040 Found-by: continuous fuzzing process https://github.com

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

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 @@ -1,

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

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

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 @@ -1,4

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

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

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

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

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-$(cal

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

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 @@ MPEG4_RE

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

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

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

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-c

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

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/enc_external.

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

--- 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

--- 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

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 dependencies fate/all

[FFmpeg-devel] [PATCH v7 3/3] fftools/resources: Update .gitignore

From: softworkz Signed-off-by: softworkz --- fftools/resources/.gitignore | 5 ++--- 1 file changed, 2 insertions(+), 3 deletions(-) diff --git a/fftools/resources/.gitignore b/fftools/resources/.gitignore index bda2c59a1c..cc742817b5 100644 --- a/fftools/resources/.gitignore +++ b/fftools/res

[FFmpeg-devel] [PATCH v7 1/3] fftools/resources: Fix double-build by disabling .d file generation

From: softworkz Signed-off-by: softworkz --- ffbuild/common.mak | 6 ++ fftools/Makefile | 1 + fftools/resources/Makefile | 12 +--- 3 files changed, 12 insertions(+), 7 deletions(-) diff --git a/ffbuild/common.mak b/ffbuild/common.mak index ddf48923ea..81e8a46

[FFmpeg-devel] [PATCH v7 2/3] ffbuild/commonmak: Consolidate pattern rules for compression

From: softworkz This commit simplifies and consolidates all the rules around ptx and resource file compression. Signed-off-by: softworkz --- ffbuild/common.mak | 43 +-- 1 file changed, 17 insertions(+), 26 deletions(-) diff --git a/ffbuild/common.mak b

[FFmpeg-devel] [PATCH v7 0/3] ffbuild/commonmak: Fix rebuild check with implicit rule chains

V2 == * Fix MSVC build (use the universal command pattern) V3 == * Skip dependency generation by clearing CC_DEPS instead (as suggested by Ramiro - thanks!) V4 == * Always keep .ptx files (as suggested by Timo - thanks) Tested all scenarios: * .ptx.c and .ptx.gz still get de

Re: [FFmpeg-devel] [PATCH] avcodec/g728dec: Remove AVCodec.sample_fmts

On Mon, Jun 23, 2025 at 12:43:54PM +0200, Andreas Rheinhardt wrote: > Patch attached. > > - Andreas > From e2b8892a583a09ca4aa607af8813b4947c2e6fc1 Mon Sep 17 00:00:00 2001 > From: Andreas Rheinhardt > Date: Mon, 23 Jun 2025 12:41:36 +0200 > Subject: [PATCH] avcodec/g728dec: Remove AVCodec.sampl

Re: [FFmpeg-devel] [PATCH 1/7] avutil: add an API to handle 3D Reference Displays Information

On 16.06.2025 20:26, Andreas Rheinhardt wrote: Timo Rothenpieler: On 16.06.2025 14:55, James Almer wrote: On 6/16/2025 9:38 AM, Timo Rothenpieler wrote: On 13/06/2025 16:07, Timo Rothenpieler wrote: On 10/06/2025 00:09, Andreas Rheinhardt wrote: James Almer: On 6/9/2025 5:59 PM, Timo Rothen

Re: [FFmpeg-devel] [PATCH 1/4] avformat/iamf_parse: Check extradata size

On 6/23/2025 12:14 PM, James Almer wrote: On 6/23/2025 9:57 AM, Andreas Rheinhardt wrote: James Almer: On 6/23/2025 9:44 AM, Michael Niedermayer wrote: On Fri, Jun 20, 2025 at 12:28:13AM +0200, Andreas Rheinhardt wrote: Michael Niedermayer: Fixes: Assertion n>=0 && n<=32 failed at ./libavcod

Re: [FFmpeg-devel] [PATCH 1/4] avformat/iamf_parse: Check extradata size

On 6/23/2025 9:57 AM, Andreas Rheinhardt wrote: James Almer: On 6/23/2025 9:44 AM, Michael Niedermayer wrote: On Fri, Jun 20, 2025 at 12:28:13AM +0200, Andreas Rheinhardt wrote: Michael Niedermayer: Fixes: Assertion n>=0 && n<=32 failed at ./libavcodec/get_bits.h:406 Fixes: 398527871/clusterf

Re: [FFmpeg-devel] [PATCH] Add FUNDING.json

Hi all On Thu, Jun 19, 2025 at 09:12:23PM +0200, Michael Niedermayer wrote: > We have been contacted by Monet du Plessis of the etherium foundation and > Natascha Buck (Grants at the Better Internet Foundation) > > "The Ethereum Foundation is thrilled to inform you that your contribution to > op

Re: [FFmpeg-devel] [PATCH 1/5] avformat/mov: Check that sample_count is allocated in mov_parse_heif_items()

On 6/21/2025 6:15 PM, Michael Niedermayer wrote: Fixes: NULL pointer dereference Fixes: 416811958/clusterfuzz-testcase-minimized-ffmpeg_dem_MOV_fuzzer-5425269114732544 Found-by: continuous fuzzing process https://github.com/google/oss-fuzz/tree/master/projects/ffmpeg Signed-off-by: Michael Nie

Re: [FFmpeg-devel] [PATCH 4/5] avformat/iff: Check nb_channels == 0 in CHNL

On Sun, Jun 22, 2025 at 02:40:23PM +1000, Peter Ross wrote: > On Sat, Jun 21, 2025 at 11:15:20PM +0200, Michael Niedermayer wrote: > > Fixes: division by 0 > > Fixes: > > 418396712/clusterfuzz-testcase-minimized-ffmpeg_dem_IFF_fuzzer-6104388018176000 > > Fixes: > > 418478219/clusterfuzz-testcase-

Re: [FFmpeg-devel] [PATCH 4/8] tools/target_dec_fuzzer: adjust threshold for VP8

On Fri, Jun 20, 2025 at 02:32:51AM +0200, Michael Niedermayer wrote: > Fixes: Timeout > Fixes: > 416589179/clusterfuzz-testcase-minimized-ffmpeg_AV_CODEC_ID_VP8_fuzzer-5432788428062720 > > Found-by: continuous fuzzing process > https://github.com/google/oss-fuzz/tree/master/projects/ffmpeg > Sig

Re: [FFmpeg-devel] [PATCH 1/4] avformat/iamf_parse: Check extradata size

On Mon, Jun 23, 2025 at 04:19:52PM +0200, Michael Niedermayer wrote: > On Mon, Jun 23, 2025 at 09:47:55AM -0300, James Almer wrote: > > On 6/23/2025 9:44 AM, Michael Niedermayer wrote: > > > On Fri, Jun 20, 2025 at 12:28:13AM +0200, Andreas Rheinhardt wrote: > > > > Michael Niedermayer: > > > > > F

Re: [FFmpeg-devel] [PATCH 1/4] avformat/iamf_parse: Check extradata size

On Mon, Jun 23, 2025 at 09:47:55AM -0300, James Almer wrote: > On 6/23/2025 9:44 AM, Michael Niedermayer wrote: > > On Fri, Jun 20, 2025 at 12:28:13AM +0200, Andreas Rheinhardt wrote: > > > Michael Niedermayer: > > > > Fixes: Assertion n>=0 && n<=32 failed at ./libavcodec/get_bits.h:406 > > > > Fix

Re: [FFmpeg-devel] [PATCH v3 1/3] avformat/whip: replace AV_OPT_FLAG_DECODING_PARAM to ENCODING

> On Jun 21, 2025, at 12:58, Zhao Zhili > wrote: > > > >> On Jun 21, 2025, at 10:15, Jack Lau >> wrote: >> >> >> >>> On Jun 13, 2025, at 16:56, Jack Lau via ffmpeg-devel >>> wrote: >>> >>> >>> From: Jack Lau >>> Subject: [PATCH v3 1/3] avformat/whip: replace AV_OPT_FLAG_DECODING_PA

[FFmpeg-devel] [PATCH 48/48] avcodec/rv34: Fix spelling mistake

From: Andreas Rheinhardt Signed-off-by: Andreas Rheinhardt --- libavcodec/rv34.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/libavcodec/rv34.c b/libavcodec/rv34.c index bd8800ee5d..f78b91e7df 100644 --- a/libavcodec/rv34.c +++ b/libavcodec/rv34.c @@ -1800,7 +1800,7 @@ in

[FFmpeg-devel] [PATCH 47/48] avcodec/rv34: Don't report progress unnecessarily

From: Andreas Rheinhardt ff_mpv_frame_end() already does it. Signed-off-by: Andreas Rheinhardt --- libavcodec/rv34.c | 4 1 file changed, 4 deletions(-) diff --git a/libavcodec/rv34.c b/libavcodec/rv34.c index 4feab95eb5..bd8800ee5d 100644 --- a/libavcodec/rv34.c +++ b/libavcodec/rv34.c

[FFmpeg-devel] [PATCH 46/48] avcodec/mpegvideo: Move loop_filter to {H263Dec, MPVEnc, VC1}Context

From: Andreas Rheinhardt Signed-off-by: Andreas Rheinhardt --- libavcodec/dxva2_vc1.c | 4 ++-- libavcodec/h261enc.c | 4 ++-- libavcodec/h263dec.c | 4 ++-- libavcodec/h263dec.h | 1 + libavcodec/intelh263dec.c | 2 +- libavcodec/ituh263dec.c| 8 libav

[FFmpeg-devel] [PATCH 45/48] avcodec/mpegvideo: Move partitioned_frame to {H263Dec, MPVEnc}Context

From: Andreas Rheinhardt Signed-off-by: Andreas Rheinhardt --- libavcodec/h263dec.c | 16 libavcodec/h263dec.h | 1 + libavcodec/mpeg4videodec.c | 10 +- libavcodec/mpeg4videoenc.c | 2 +- libavcodec/mpegvideo.h | 1 - libavcodec/mpegvideo_enc.c | 10

[FFmpeg-devel] [PATCH 44/48] avcodec/mpeg_er: Allow to skip setting partitioned_frame, p[pb]_time

From: Andreas Rheinhardt Instead of setting these unconditionally (they are always zero for H.261, MPEG-1/2, RV30/40, VC-1), add a variant of ff_mpeg_er_frame_start() that sets them and remove setting them from ff_mpeg_er_frame_start(). Also pass these values via parameters instead of reading the

[FFmpeg-devel] [PATCH 43/48] avcodec/mpegvideo: Move fields to {H263Dec, MPVEnc}Context when possible

From: Andreas Rheinhardt Signed-off-by: Andreas Rheinhardt --- libavcodec/flvdec.c| 6 +-- libavcodec/flvenc.c| 2 +- libavcodec/h261enc.c | 2 +- libavcodec/h263dec.c | 14 +++ libavcodec/h263dec.h | 14 +++ libavcodec/intelh263dec.c | 4 +- libav

[FFmpeg-devel] [PATCH 42/48] avcodec/flvdec: Binarize h263_flv

From: Andreas Rheinhardt It used to be a tri-state encoding both whether to use FLV picture headers and whether to use the FLV way of encoding escape values, but the former is now unnecessary due to the switch to a function pointer for reading the header. So binarize h263_flv. Signed-off-by: And

[FFmpeg-devel] [PATCH 41/48] avcodec/ituh263enc: Inline value of h263_flv

From: Andreas Rheinhardt It is always two when we encode FLV1. Signed-off-by: Andreas Rheinhardt --- libavcodec/flvenc.c| 2 +- libavcodec/ituh263enc.c| 12 libavcodec/mpegvideo_enc.c | 1 - 3 files changed, 5 insertions(+), 10 deletions(-) diff --git a/libavcodec/f

[FFmpeg-devel] [PATCH 40/48] avcodec/h263dec: Use function ptr for decode_picture_header

From: Andreas Rheinhardt Signed-off-by: Andreas Rheinhardt --- libavcodec/h263dec.c | 27 ++- libavcodec/h263dec.h | 8 +++- libavcodec/mpeg4videodec.c | 3 ++- libavcodec/mpeg4videodec.h | 1 - libavcodec/msmpeg4dec.c| 8 +--- libavcodec/msmp

[FFmpeg-devel] [PATCH 39/48] avcodec/msmpeg4dec: Move ff_msmpeg4_decode_init() down

From: Andreas Rheinhardt Will avoid a forward declaration lateron. Signed-off-by: Andreas Rheinhardt --- libavcodec/msmpeg4dec.c | 79 + 1 file changed, 40 insertions(+), 39 deletions(-) diff --git a/libavcodec/msmpeg4dec.c b/libavcodec/msmpeg4dec.c ind

[FFmpeg-devel] [PATCH 38/48] avcodec/mpegvideo: Move SLICE_* defs to h263dec.h, h261dec.c

From: Andreas Rheinhardt Signed-off-by: Andreas Rheinhardt --- libavcodec/h261dec.c | 4 libavcodec/h263dec.h | 4 libavcodec/mpegvideo.h | 5 - 3 files changed, 8 insertions(+), 5 deletions(-) diff --git a/libavcodec/h261dec.c b/libavcodec/h261dec.c index e47bc00a71..32d4190

[FFmpeg-devel] [PATCH 37/48] avcodec/mpegvideo: Move mb_skip_run to {RV34Dec, MPVEnc}Context

From: Andreas Rheinhardt Signed-off-by: Andreas Rheinhardt --- libavcodec/h261enc.c | 12 ++-- libavcodec/mpeg12enc.c | 8 libavcodec/mpegvideo.h | 1 - libavcodec/mpegvideo_enc.c | 11 ++- libavcodec/mpegvideoenc.h | 2 ++ libavcodec/rv34.c |

[FFmpeg-devel] [PATCH 36/48] avcodec/mpeg12dec: Put mb_skip_run on the stack

From: Andreas Rheinhardt Signed-off-by: Andreas Rheinhardt --- libavcodec/mpeg12dec.c | 23 +++ 1 file changed, 11 insertions(+), 12 deletions(-) diff --git a/libavcodec/mpeg12dec.c b/libavcodec/mpeg12dec.c index db6f9d66a5..998f6aa2e5 100644 --- a/libavcodec/mpeg12dec.c ++

[FFmpeg-devel] [PATCH 35/48] avcodec/mpegvideo: Move mb_num_left to {H263, RV34}DecContext

From: Andreas Rheinhardt Signed-off-by: Andreas Rheinhardt --- libavcodec/h263dec.h | 2 ++ libavcodec/ituh263dec.c| 2 +- libavcodec/mpeg4videodec.c | 4 ++-- libavcodec/mpegvideo.h | 1 - libavcodec/rv10.c | 2 +- libavcodec/rv34.c | 16 l

[FFmpeg-devel] [PATCH 34/48] avcodec/mpegvideo: Move fields only used by H.263 decoders to H263DecCtx

From: Andreas Rheinhardt Signed-off-by: Andreas Rheinhardt --- libavcodec/flvdec.c| 4 ++-- libavcodec/h263dec.c | 28 +- libavcodec/h263dec.h | 20 +++ libavcodec/intelh263dec.c | 8 libavcodec/ituh263dec.c| 40 +++

[FFmpeg-devel] [PATCH 33/48] avcodec/mpeg12dec: Don't use MPVContext.block

From: Andreas Rheinhardt Instead add the necessary blocks directly into Mpeg12SliceContext. This allows to completely remove MPVContext.block. Signed-off-by: Andreas Rheinhardt --- libavcodec/mpeg12dec.c | 40 -- libavcodec/mpeg4videodec.c | 1 - libavc

[FFmpeg-devel] [PATCH 32/48] avcodec/mpeg12dec: Move MpegEncContext.gb to Mpeg12SliceContext

From: Andreas Rheinhardt It was its last user. Signed-off-by: Andreas Rheinhardt --- libavcodec/mpeg12dec.c | 207 + libavcodec/mpegvideo.h | 4 - 2 files changed, 104 insertions(+), 107 deletions(-) diff --git a/libavcodec/mpeg12dec.c b/libavcodec/mp

[FFmpeg-devel] [PATCH 31/48] avcodec/mpegvideo: Add missing headers

From: Andreas Rheinhardt These files currently rely on implicit inclusions of avassert.h and/or mathops.h via get_bits.h. Signed-off-by: Andreas Rheinhardt --- libavcodec/error_resilience.c | 2 ++ libavcodec/me_cmp.c| 1 + libavcodec/mips/mpegvideo_msa.c|

[FFmpeg-devel] [PATCH 30/48] avcodec/mpeg12dec: Add Mpeg12SliceContext

From: Andreas Rheinhardt This is in preparation for removing the GetBitContext from MPVContext. Signed-off-by: Andreas Rheinhardt --- libavcodec/mpeg12dec.c | 863 + libavcodec/mpegvideo.c | 2 +- libavcodec/mpegvideo.h | 1 + 3 files changed, 436 in

[FFmpeg-devel] [PATCH 29/48] avcodec/h263dec: Stop using MpegEncContext.gb

From: Andreas Rheinhardt Add a GetBitContext to H263DecContext instead. This is in preparation for removing MpegEncContext.gb. Also move last_resync_gb to H263DecContext. Signed-off-by: Andreas Rheinhardt --- libavcodec/flvdec.c| 24 +-- libavcodec/h263dec.c | 52 +++--- libavc

[FFmpeg-devel] [PATCH 28/48] avcodec/h263dec: Don't use MpegEncContext.block

From: Andreas Rheinhardt Instead add the necessary blocks directly to H263DecContext (only six are needed, not 12 as ff_mpv_common_init() currently allocates). Signed-off-by: Andreas Rheinhardt --- libavcodec/h263dec.c | 4 ++-- libavcodec/h263dec.h | 3 +++ libavcodec/ituh263dec

[FFmpeg-devel] [PATCH 27/48] avcodec/h263dec: Remove redundant block parameter from decode_mb

From: Andreas Rheinhardt With the exception of mpeg4_decode_studio_mb(), all decode_mb functions implicitly presumed that the block provided as argument coincides with MpegEncContext.block (they zeroed the latter and then used the former to decode the block); mpeg4_decode_studio_mb() meanwhile di

[FFmpeg-devel] [PATCH 23/48] avcodec/mpeg4videodec: Avoid unnecessary indirections

From: Andreas Rheinhardt This basically reverts d4967c04e040b3b2f937cad88599af825147ec94. Said commit was based on the false premise that it would be an aliasing violation to upcast a pointer to structure to a pointer to a bigger structure containing the original structure as its first element, b

[FFmpeg-devel] [PATCH 25/48] avcodec/mpegvideo: Move dct_precision to Mpeg4DecContext

From: Andreas Rheinhardt Signed-off-by: Andreas Rheinhardt --- libavcodec/mpeg4videodec.c | 25 ++--- libavcodec/mpeg4videodec.h | 1 + libavcodec/mpegvideo.h | 1 - 3 files changed, 15 insertions(+), 12 deletions(-) diff --git a/libavcodec/mpeg4videodec.c b/libavcode

[FFmpeg-devel] [PATCH 24/48] avcodec/{h263, mpeg4video}dec: Pass MPVContext*, not Mpeg4DecContext*

From: Andreas Rheinhardt The code in h263dec.c is not supposed to know that the MPEG-4 decoder uses an Mpeg4DecContext as private context at all (said context is only exposed in a header so that hardware accelerations can access it and for the parser to use). Passing an MPVContext* directly also

[FFmpeg-devel] [PATCH 22/48] avcodec/mpegvideo: Move unrestricted_mv to MotionEstContext

From: Andreas Rheinhardt Signed-off-by: Andreas Rheinhardt --- libavcodec/motion_est.c| 4 ++-- libavcodec/motion_est.h| 1 + libavcodec/mpegvideo.h | 1 - libavcodec/mpegvideo_enc.c | 18 +- libavcodec/rv20enc.c | 2 +- libavcodec/snowenc.c | 2 +-

[FFmpeg-devel] [PATCH 21/48] avcodec/mpegvideo: Move flipflop_rounding to {MSMPEG4Dec, MPVEnc}Context

From: Andreas Rheinhardt Forgotten in 9964212545554d9ae1afaabd23a7f9a361997d01. Signed-off-by: Andreas Rheinhardt --- libavcodec/mpegvideo.h | 1 - libavcodec/mpegvideo_enc.c | 14 +++--- libavcodec/mpegvideoenc.h | 1 + libavcodec/msmpeg4dec.c| 8 libavcodec/msmpe

[FFmpeg-devel] [PATCH 20/48] avcodec/mpeg12dec: Deduplicate variables

From: Andreas Rheinhardt This situation was created in merge commit 71d008ebe4f96974433eecfd3575bc82eb4b06a8. Signed-off-by: Andreas Rheinhardt --- libavcodec/mpeg12dec.c | 3 +-- 1 file changed, 1 insertion(+), 2 deletions(-) diff --git a/libavcodec/mpeg12dec.c b/libavcodec/mpeg12dec.c index

[FFmpeg-devel] [PATCH 19/48] avcodec/vc1: Don't use MpegEncContext.block

From: Andreas Rheinhardt Instead add the necessary blocks directly to VC1Context (only six are needed, not 12 as ff_mpv_common_init() currently allocates). Signed-off-by: Andreas Rheinhardt --- libavcodec/vc1.h | 4 libavcodec/vc1_block.c | 28 ++-- libavcod

[FFmpeg-devel] [PATCH 18/48] avcodec/vc1: Stop using MpegEncContext.gb

From: Andreas Rheinhardt Add a GetBitContext to VC1Context instead. This is in preparation for removing MpegEncContext.gb. Signed-off-by: Andreas Rheinhardt --- libavcodec/dxva2_vc1.c | 2 +- libavcodec/mss2.c | 4 +- libavcodec/vaapi_vc1.c | 2 +- libavcodec/vc1.c | 14 +++

[FFmpeg-devel] [PATCH 17/48] avcodec/intrax8: Don't pretend to need more than one int16_t[64]

From: Andreas Rheinhardt Intrax8 needs only a single block. Signed-off-by: Andreas Rheinhardt --- libavcodec/intrax8.c | 16 libavcodec/intrax8.h | 4 ++-- libavcodec/vc1dec.c | 2 +- libavcodec/wmv2dec.c | 2 +- 4 files changed, 12 insertions(+), 12 deletions(-) diff --g

[FFmpeg-devel] [PATCH 16/48] avcodec/rv34: Don't use MpegEncContext.block

From: Andreas Rheinhardt These decoders only need a single 4x4 block; put it in RV34DecContext. This is in preparation for removing MpegEncContext.block. Signed-off-by: Andreas Rheinhardt --- libavcodec/rv34.c | 7 +++ libavcodec/rv34.h | 1 + 2 files changed, 4 insertions(+), 4 deletions(

[FFmpeg-devel] [PATCH 15/48] avcodec/rv34: Don't use MpegEncContext.gb

From: Andreas Rheinhardt This is in preparation for removing MpegEncContext.gb. Signed-off-by: Andreas Rheinhardt --- libavcodec/rv30.c | 2 +- libavcodec/rv34.c | 28 ++-- libavcodec/rv34.h | 2 ++ libavcodec/rv40.c | 2 +- 4 files changed, 18 insertions(+), 16 dele

[FFmpeg-devel] [PATCH 14/48] avcodec/mpeg12dec: Remove unused function parameter

From: Andreas Rheinhardt Signed-off-by: Andreas Rheinhardt --- libavcodec/mpeg12dec.c | 5 +++-- 1 file changed, 3 insertions(+), 2 deletions(-) diff --git a/libavcodec/mpeg12dec.c b/libavcodec/mpeg12dec.c index ec8b213206..a056dd87cc 100644 --- a/libavcodec/mpeg12dec.c +++ b/libavcodec/mpeg12

[FFmpeg-devel] [PATCH 13/48] avcodec/mpeg12dec: Put GetBitContext on the stack where advantageous

From: Andreas Rheinhardt This is in preparation for no longer using MpegEncContext.gb. Signed-off-by: Andreas Rheinhardt --- libavcodec/mpeg12dec.c | 192 ++--- 1 file changed, 102 insertions(+), 90 deletions(-) diff --git a/libavcodec/mpeg12dec.c b/libavco

[FFmpeg-devel] [PATCH 12/48] avcodec/h261dec: Don't use MpegEncContext.block

From: Andreas Rheinhardt Instead put it into H261DecContext. This is in preparation for removing MpegEncContext.block. Signed-off-by: Andreas Rheinhardt --- libavcodec/h261dec.c | 11 +++ 1 file changed, 7 insertions(+), 4 deletions(-) diff --git a/libavcodec/h261dec.c b/libavcodec/h2

[FFmpeg-devel] [PATCH 11/48] avcodec/h261dec: Stop using MpegEncContext.gb

From: Andreas Rheinhardt This is in preparation for removing said field from MpegEncContext. Signed-off-by: Andreas Rheinhardt --- libavcodec/h261dec.c | 80 +++- 1 file changed, 41 insertions(+), 39 deletions(-) diff --git a/libavcodec/h261dec.c b/liba

[FFmpeg-devel] [PATCH 10/48] avcodec/mpegvideodec: Remove size expectation from ff_mpv_reconstruct_mb

From: Andreas Rheinhardt In the common case of decoding 420 content, only six blocks are used and it makes perfect sense for the caller to only have that many. Signed-off-by: Andreas Rheinhardt --- libavcodec/mpegvideo_dec.c | 2 +- libavcodec/mpegvideodec.h | 2 +- 2 files changed, 2 inserti

[FFmpeg-devel] [PATCH 09/48] avcodec/mpeg_er: Don't use MpegEncContext.block

From: Andreas Rheinhardt It is unused (because unquantizing/the idct has been disabled) apart from FF_DEBUG_DCT_COEFF debug code which makes no sense when this function is called via error resilience. So pass a NULL as block when calling ff_mpv_reconstruct_mb() from mpeg_er_decode_mb() and disabl

[FFmpeg-devel] [PATCH 08/48] avcodec/mpegvideo_dec: Reindent after the previous commit

From: Andreas Rheinhardt Also use dedicated variables for the accesses. Signed-off-by: Andreas Rheinhardt --- libavcodec/mpegvideo_dec.c | 21 - 1 file changed, 12 insertions(+), 9 deletions(-) diff --git a/libavcodec/mpegvideo_dec.c b/libavcodec/mpegvideo_dec.c index c687

[FFmpeg-devel] [PATCH 07/48] avcodec/mpegvideo_dec: Factor debugging dct coefficients out

From: Andreas Rheinhardt This allows to hint to the compiler that the branch is likely not taken by marking the factored out function as av_cold. It is also in preparation for further commits. Signed-off-by: Andreas Rheinhardt --- libavcodec/mpegvideo_dec.c | 26 +++--- 1 f

[FFmpeg-devel] [PATCH 06/48] avcodec/mpegvideo: Add MPVContext typedef

From: Andreas Rheinhardt MpegEncContext is also used for decoding, so it is a misnomer. This commit therefore adds MPVContext as typedef for it in order to gradually switch the code to the new name (in line with MPVEncContext). Signed-off-by: Andreas Rheinhardt --- libavcodec/mpegvideo.h | 1 +

[FFmpeg-devel] [PATCH 05/48] avcodec/mpegvideoenc: Allocate blocks as part of MPVEncContext

From: Andreas Rheinhardt This avoids mpegvideo.c having to deal with the fact that the encoders use two sets of blocks and is in preparation for not allocating blocks at all. Signed-off-by: Andreas Rheinhardt --- libavcodec/h263enc.h | 2 +- libavcodec/mpeg4videodec.c | 1 - libavcodec

[FFmpeg-devel] [PATCH 04/48] avcodec/ituh263dec: Only initialize ff_h263_rl_inter when needed

From: Andreas Rheinhardt The H.263 decoder itself does not need max_run/max_level of this RLTable, only the MPEG-4 decoder does. Signed-off-by: Andreas Rheinhardt --- libavcodec/ituh263dec.c| 1 - libavcodec/mpeg4videodec.c | 1 + 2 files changed, 1 insertion(+), 1 deletion(-) diff --git

[FFmpeg-devel] [PATCH 03/48] avcodec/ituh263enc: Simplify creating LUT

From: Andreas Rheinhardt Only very few combinations (2x102) of 16384 correspond to valid codes; so just initialize all codes via memset and then set the few valid codes explicitly instead of initializing everything in the same way. Signed-off-by: Andreas Rheinhardt --- libavcodec/h263data.h

[FFmpeg-devel] [PATCH 02/48] avcodec/rl: Avoid branch in index lookup

From: Andreas Rheinhardt This uses the same logic as the MPEG-1/2 and SpeedHQ encoders. Signed-off-by: Andreas Rheinhardt --- libavcodec/rl.h | 6 +- 1 file changed, 1 insertion(+), 5 deletions(-) diff --git a/libavcodec/rl.h b/libavcodec/rl.h index c45d8659d1..2588de2440 100644 --- a/lib

[FFmpeg-devel] [PATCH 01/48] avcodec/ituh263dec: Use correct logcontext

From: Andreas Rheinhardt Signed-off-by: Andreas Rheinhardt --- libavcodec/ituh263dec.c | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/libavcodec/ituh263dec.c b/libavcodec/ituh263dec.c index f1f96010b2..95cd8f0b13 100644 --- a/libavcodec/ituh263dec.c +++ b/libavcodec/ituh2

  1   2   >