[FFmpeg-devel] [PATCH] avformat/utils: fix logic error in ff_mkdir_p

Fix ticket# 9605 Signed-off-by: Peter Ross --- libavformat/utils.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/libavformat/utils.c b/libavformat/utils.c index e643821fc9..1164cebe68 100644 --- a/libavformat/utils.c +++ b/libavformat/utils.c @@ -1135,7 +1135,7 @@ int ff_mk

[FFmpeg-devel] [PATCH] avformat/img2dec: increase probe score for fourcc variants

Signed-off-by: Peter Ross --- libavformat/img2dec.c | 7 +++ 1 file changed, 3 insertions(+), 4 deletions(-) diff --git a/libavformat/img2dec.c b/libavformat/img2dec.c index 2583ca2465..8608252d83 100644 --- a/libavformat/img2dec.c +++ b/libavformat/img2dec.c @@ -1112,7 +1112,6 @@ static int

Re: [FFmpeg-devel] [PATCH] Fix setsockopt IP_MULTICAST_TTL on OpenBSD

On Wed, Jan 12, 2022 at 12:13:13AM -0500, Brad Smith wrote: > Fix setsockopt() usage on OpenBSD with IP_MULTICAST_TTL. The field > type should be an unsigned char on anything but Linux. > > > diff --git a/libavformat/udp.c b/libavformat/udp.c > index 180d96a988..29aa865fff 100644 > --- a/libavfor

Re: [FFmpeg-devel] [PATCH V2 1/3] libavcodec/qsvenc: Add max slice size support to hevc_qsv

On Tue, 2022-01-25 at 15:12 +0800, Wenbin Chen wrote: > Add max_slice_size option to hevc_qsv as well. > > Signed-off-by: Wenbin Chen > --- > doc/encoders.texi | 3 +++ > libavcodec/qsvenc.c | 9 - > 2 files changed, 7 insertions(+), 5 deletions(-) > > diff --git a/doc/encoders.texi b

[FFmpeg-devel] [PATCH V2 3/3] libavcodec/qsvenc: Add intra refresh to hevc_qsv and add new intra refresh parameter

Add intra refresh support to hevc_qsv as well. Add an new intra refresh type: "horizontal", and an new param ref_cycle_dist. This param specify the distance between the beginnings of the intra-refresh cycles in frames. Signed-off-by: Wenbin Chen --- doc/encoders.texi| 26

[FFmpeg-devel] [PATCH V2 1/3] libavcodec/qsvenc: Add max slice size support to hevc_qsv

Add max_slice_size option to hevc_qsv as well. Signed-off-by: Wenbin Chen --- doc/encoders.texi | 3 +++ libavcodec/qsvenc.c | 9 - 2 files changed, 7 insertions(+), 5 deletions(-) diff --git a/doc/encoders.texi b/doc/encoders.texi index e3adbf4325..8966610263 100644 --- a/doc/encoder

[FFmpeg-devel] [PATCH V2 2/3] libavcodec/qsvenc: Add b_strategy option to hevc_qsv

Add b_strategy option to hevc_qsv. By enabling this option, encoder can use b frames as reference. Signed-off-by: Wenbin Chen --- doc/encoders.texi | 3 +++ libavcodec/qsvenc.c | 6 -- 2 files changed, 7 insertions(+), 2 deletions(-) diff --git a/doc/encoders.texi b/doc/encoders.texi inde

Re: [FFmpeg-devel] [PATCH] Fix setsockopt IP_MULTICAST_TTL on OpenBSD

On Mon, Jan 24, 2022 at 01:40:47PM +0100, Michael Niedermayer wrote: > On Sun, Jan 23, 2022 at 02:55:38PM -0500, Brad Smith wrote: > > On 1/23/2022 6:57 AM, Michael Niedermayer wrote: > > > > > On Wed, Jan 12, 2022 at 12:13:13AM -0500, Brad Smith wrote: > > > > Fix setsockopt() usage on OpenBSD wi

Re: [FFmpeg-devel] [PATCH 3/3] libavcodec/qsvenc: Add intra refresh to hevc_qsv and add new intra refresh parameter

> On Mon, 2022-01-24 at 10:59 +0800, Wenbin Chen wrote: > > Add intra refresh support to hevc_qsv as well. > > Add an new intra refresh type: "horizontal", and an new param > > ref_cycle_dist. This param specify the distance between the > > beginnings of the intra-refresh cycles in frames. > > > >

Re: [FFmpeg-devel] [PATCH 1/3] libavcodec/qsvenc: Add max slice size support to hevc_qsv

On Mon, 2022-01-24 at 10:59 +0800, Wenbin Chen wrote: > Add max_slice_size option to hevc_qsv as well. > > Signed-off-by: Wenbin Chen > --- > doc/encoders.texi | 3 +++ > libavcodec/qsvenc.c | 9 - > 2 files changed, 7 insertions(+), 5 deletions(-) > > diff --git a/doc/encoders.texi b

Re: [FFmpeg-devel] [PATCH 2/3] libavcodec/qsvenc: Add b_strategy option to hevc_qsv

On Mon, 2022-01-24 at 10:59 +0800, Wenbin Chen wrote: > Add b_strategy option to hevc_qsv. By enabling this option, encoder can > use b frames as reference. > > Signed-off-by: Wenbin Chen > --- > doc/encoders.texi | 3 +++ > libavcodec/qsvenc.c | 6 -- > 2 files changed, 7 insertions(+), 2

Re: [FFmpeg-devel] [PATCH 3/3] libavcodec/qsvenc: Add intra refresh to hevc_qsv and add new intra refresh parameter

On Mon, 2022-01-24 at 10:59 +0800, Wenbin Chen wrote: > Add intra refresh support to hevc_qsv as well. > Add an new intra refresh type: "horizontal", and an new param > ref_cycle_dist. This param specify the distance between the > beginnings of the intra-refresh cycles in frames. > > Signed-off-by

Re: [FFmpeg-devel] [PATCH 001/293 v8] avutil/channel_layout: Add a new channel layout API

On 1/24/2022 6:08 PM, Marton Balint wrote: On Mon, 24 Jan 2022, James Almer wrote: From: Anton Khirnov The new API is more extensible and allows for custom layouts. More accurate information is exported, eg for decoders that do not set a channel layout, lavc will not make one up for them

Re: [FFmpeg-devel] [PATCH 001/293 v8] avutil/channel_layout: Add a new channel layout API

On Mon, 24 Jan 2022, James Almer wrote: From: Anton Khirnov The new API is more extensible and allows for custom layouts. More accurate information is exported, eg for decoders that do not set a channel layout, lavc will not make one up for them. Deprecate the old API working with just uin

[FFmpeg-devel] [PATCH 001/293 v8] avutil/channel_layout: Add a new channel layout API

From: Anton Khirnov The new API is more extensible and allows for custom layouts. More accurate information is exported, eg for decoders that do not set a channel layout, lavc will not make one up for them. Deprecate the old API working with just uint64_t bitmasks. Expanded and completed by Vit

[FFmpeg-devel] [PATCH v2 18/18] avcodec/h264_parse: Move ff_h264_get_profile() to h264_ps.h

It is a more fitting place for it. Signed-off-by: Andreas Rheinhardt --- libavcodec/h264_parse.h| 5 - libavcodec/h264_ps.h | 5 + libavcodec/mediacodecdec.c | 1 + 3 files changed, 6 insertions(+), 5 deletions(-) diff --git a/libavcodec/h264_parse.h b/libavcodec/h264_parse.h

[FFmpeg-devel] [PATCH v2 17/18] avcodec/h264_*: Remove unnecessary internal.h inclusions

Also remove some other unnecessary headers while at it. Signed-off-by: Andreas Rheinhardt --- libavcodec/h264_cabac.c | 1 - libavcodec/h264_cavlc.c | 1 - libavcodec/h264_direct.c | 1 - libavcodec/h264_loopfilter.c | 3 --- libavcodec/h264_mvpred.h | 2 -- 5 files changed, 8

[FFmpeg-devel] [PATCH v2 16/18] avcodec/h264dec: Remove unnecessary headers

E.g. the inclusion of parser.h comes from a time when the parser used a H264Context. Signed-off-by: Andreas Rheinhardt --- libavcodec/h264dec.c | 1 + libavcodec/h264dec.h | 3 --- 2 files changed, 1 insertion(+), 3 deletions(-) diff --git a/libavcodec/h264dec.c b/libavcodec/h264dec.c index ed9

[FFmpeg-devel] [PATCH v2 15/18] avcodec/h264*: Remove unnecessary h264_mvpred.h inclusions

This is only needed by h264_cabac.c and h264_cavlc.c. Also fix up the other headers while at it. Signed-off-by: Andreas Rheinhardt --- libavcodec/h264_picture.c | 9 - libavcodec/h264_slice.c | 3 +-- libavcodec/h264dec.c | 9 - 3 files changed, 1 insertion(+), 20 deletion

[FFmpeg-devel] [PATCH v2 14/18] lavc/svq3: stop including h264dec.h

From: Anton Khirnov The only thing that is actually used directly from there is the PART_NOT_AVAILABLE constant, which can be trivially copied to svq3 decoder itself. Otherwise it only depends on other indirectly included headers. --- libavcodec/svq3.c | 7 ++- 1 file changed, 6 insertions(

[FFmpeg-devel] [PATCH v2 13/18] lavc/x86/h264_qpel: stop unnecessarily including h264dec

From: Anton Khirnov --- libavcodec/x86/h264_qpel.c | 1 - 1 file changed, 1 deletion(-) diff --git a/libavcodec/x86/h264_qpel.c b/libavcodec/x86/h264_qpel.c index 320d98933a..dda50ded89 100644 --- a/libavcodec/x86/h264_qpel.c +++ b/libavcodec/x86/h264_qpel.c @@ -24,7 +24,6 @@ #include "libavut

[FFmpeg-devel] [PATCH v2 12/18] lavc/vdpau: stop unnecessarily including h264dec

From: Anton Khirnov --- libavcodec/vdpau.c | 1 - 1 file changed, 1 deletion(-) diff --git a/libavcodec/vdpau.c b/libavcodec/vdpau.c index 7c29746adb..f96ac15e2a 100644 --- a/libavcodec/vdpau.c +++ b/libavcodec/vdpau.c @@ -26,7 +26,6 @@ #include "avcodec.h" #include "decode.h" #include "inte

[FFmpeg-devel] [PATCH v2 11/18] lavc/h264: move MB_TYPE defs from h264dec.h to h264_parse

From: Anton Khirnov Allows to stop including h264dec.h in h264data.c. --- libavcodec/h264_parse.h | 3 +++ libavcodec/h264data.c | 2 +- libavcodec/h264dec.h| 2 -- 3 files changed, 4 insertions(+), 3 deletions(-) diff --git a/libavcodec/h264_parse.h b/libavcodec/h264_parse.h index 52b224

[FFmpeg-devel] [PATCH v2 10/18] lavc/h264_parse: stop including h264dec.h

From: Anton Khirnov It is unnecessary and only files that are parts of the decoder (as opposed to standalone code called by the decoder) are allowed to include h264dec.h --- libavcodec/h264_parse.c | 4 +++- 1 file changed, 3 insertions(+), 1 deletion(-) diff --git a/libavcodec/h264_parse.c b/l

[FFmpeg-devel] [PATCH v2 06/18] lavc/h264: replace MAX_DELAYED_PIC_COUNT by H264_MAX_DPB_FRAMES

From: Anton Khirnov --- libavcodec/h264_ps.c| 4 ++-- libavcodec/h264_slice.c | 12 ++-- libavcodec/h264dec.h| 6 ++ 3 files changed, 10 insertions(+), 12 deletions(-) diff --git a/libavcodec/h264_ps.c b/libavcodec/h264_ps.c index e21c2b56ac..f68d5bf81c 100644 --- a/libavc

[FFmpeg-devel] [PATCH v2 09/18] lavc/h264data.h: stop including h264dec.h

From: Anton Khirnov This header does not need anything from there. --- libavcodec/h264data.h | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/libavcodec/h264data.h b/libavcodec/h264data.h index 2968b08b7e..988352aa9a 100644 --- a/libavcodec/h264data.h +++ b/libavcodec/h264data

[FFmpeg-devel] [PATCH v2 08/18] lavc/h264: move some shared code from h264dec to h264_parse

From: Anton Khirnov --- libavcodec/h264_parse.h | 32 libavcodec/h264dec.h | 26 -- tests/checkasm/h264dsp.c | 1 + 3 files changed, 33 insertions(+), 26 deletions(-) diff --git a/libavcodec/h264_parse.h b/libavcodec/h264_parse.h in

[FFmpeg-devel] [PATCH v2 07/18] lavc/h264_parser: add missing headers

From: Anton Khirnov --- libavcodec/h264_parser.c | 3 +++ 1 file changed, 3 insertions(+) diff --git a/libavcodec/h264_parser.c b/libavcodec/h264_parser.c index aee7cdb934..4002bcad77 100644 --- a/libavcodec/h264_parser.c +++ b/libavcodec/h264_parser.c @@ -40,8 +40,11 @@ #include "get_bits.h"

[FFmpeg-devel] [PATCH v2 04/18] lavc/h264dec.h: Move MMCOOpcode to h264_parse.h

From: Anton Khirnov Both parser and decoder use these, so h264_parse is the proper place for them. --- libavcodec/h264_parse.h | 13 + libavcodec/h264dec.h| 13 - 2 files changed, 13 insertions(+), 13 deletions(-) diff --git a/libavcodec/h264_parse.h b/libavcodec/h26

[FFmpeg-devel] [PATCH v2 05/18] lavc/h264: replace MAX_DELAYED_PIC_COUNT with FF_ARRAY_ELEMS where appropriate

From: Anton Khirnov --- libavcodec/h264_refs.c | 2 +- libavcodec/h264_slice.c | 2 +- libavcodec/h264dec.c| 4 ++-- 3 files changed, 4 insertions(+), 4 deletions(-) diff --git a/libavcodec/h264_refs.c b/libavcodec/h264_refs.c index 93526b11e2..94b473257f 100644 --- a/libavcodec/h264_refs.

[FFmpeg-devel] [PATCH v2 03/18] lavc/h264: replace MAX_MMCO_COUNT with H264_MAX_MMCO_COUNT

From: Anton Khirnov They apparently serve the same purpose; the latter is one larger and has a comment explaining how the value is derived, so seems more trustworthy. --- libavcodec/h264_parser.c | 2 +- libavcodec/h264_refs.c | 2 +- libavcodec/h264dec.h | 6 ++ 3 files changed, 4 ins

[FFmpeg-devel] [PATCH v2 02/18] avcodec/h264dec: Move pack8to16 to its only user

Namely to h264_cabac.c. Signed-off-by: Andreas Rheinhardt --- libavcodec/h264_cabac.c | 9 + libavcodec/h264dec.h| 9 - 2 files changed, 9 insertions(+), 9 deletions(-) diff --git a/libavcodec/h264_cabac.c b/libavcodec/h264_cabac.c index 040fa0a257..884d392022 100644 --- a/l

[FFmpeg-devel] [PATCH v2 01/18] avcodec/h264dec: Move find_start_code() to its only user

Signed-off-by: Andreas Rheinhardt --- libavcodec/h264_parser.c | 9 + libavcodec/h264dec.h | 11 --- 2 files changed, 9 insertions(+), 11 deletions(-) diff --git a/libavcodec/h264_parser.c b/libavcodec/h264_parser.c index cfbf6f4486..881cab7536 100644 --- a/libavcodec/h264_p

[FFmpeg-devel] [PATCH 18/18] avcodec/h264_parse: Move ff_h264_get_profile() to h264_ps.h

It is a more fitting place for it. Signed-off-by: Andreas Rheinhardt --- libavcodec/h264_parse.h| 5 - libavcodec/h264_ps.h | 5 + libavcodec/mediacodecdec.c | 1 + 3 files changed, 6 insertions(+), 5 deletions(-) diff --git a/libavcodec/h264_parse.h b/libavcodec/h264_parse.h

[FFmpeg-devel] [PATCH 17/18] avcodec/h264_*: Remove unnecessary internal.h inclusions

Also remove some other unnecessary headers while at it. Signed-off-by: Andreas Rheinhardt --- libavcodec/h264_cabac.c | 1 - libavcodec/h264_cavlc.c | 1 - libavcodec/h264_direct.c | 1 - libavcodec/h264_loopfilter.c | 3 --- libavcodec/h264_mvpred.h | 2 -- 5 files changed, 8

[FFmpeg-devel] [PATCH 16/18] avcodec/h264_parse: Move find_start_code() to its only user

Signed-off-by: Andreas Rheinhardt --- libavcodec/h264_parse.h | 13 + libavcodec/h264_parser.c | 9 + 2 files changed, 10 insertions(+), 12 deletions(-) diff --git a/libavcodec/h264_parse.h b/libavcodec/h264_parse.h index 9b6391edda..3901b46ac2 100644 --- a/libavcodec/h264_

[FFmpeg-devel] [PATCH 15/18] avcodec/h264dec: Remove unnecessary headers

E.g. the inclusion of parser.h comes from a time when the parser used a H264Context. Signed-off-by: Andreas Rheinhardt --- libavcodec/h264dec.c | 1 + libavcodec/h264dec.h | 4 2 files changed, 1 insertion(+), 4 deletions(-) diff --git a/libavcodec/h264dec.c b/libavcodec/h264dec.c index ed

[FFmpeg-devel] [PATCH 14/18] avcodec/h264*: Remove unnecessary h264_mvpred.h inclusions

This is only needed by h264_cabac.c and h264_cavlc.c. Also fix up the other headers while at it. Signed-off-by: Andreas Rheinhardt --- libavcodec/h264_picture.c | 9 - libavcodec/h264_slice.c | 3 +-- libavcodec/h264dec.c | 9 - 3 files changed, 1 insertion(+), 20 deletion

Re: [FFmpeg-devel] [PATCH 13/13] lavc/svq3: stop including h264dec.h

Anton Khirnov: > The only thing that is actually used directly from there is the > PART_NOT_AVAILABLE constant, which can be trivially copied to svq3 > decoder itself. > > Otherwise it only depends on other indirectly included headers. > --- > libavcodec/svq3.c | 7 ++- > 1 file changed, 6 in

Re: [FFmpeg-devel] [PATCH 02/13] lavc/h264_parser: stop accessing H264Context

On 1/24/2022 2:00 PM, Anton Khirnov wrote: Parsers should not mess with decoder private data. It is also completely unnecessary here. --- libavcodec/h264_parser.c | 8 +++- 1 file changed, 3 insertions(+), 5 deletions(-) diff --git a/libavcodec/h264_parser.c b/libavcodec/h264_parser.c

[FFmpeg-devel] [PATCH 13/13] lavc/svq3: stop including h264dec.h

The only thing that is actually used directly from there is the PART_NOT_AVAILABLE constant, which can be trivially copied to svq3 decoder itself. Otherwise it only depends on other indirectly included headers. --- libavcodec/svq3.c | 7 ++- 1 file changed, 6 insertions(+), 1 deletion(-) dif

[FFmpeg-devel] [PATCH 07/13] lavc/h264: move some shared code from h264dec to h264_parse

--- libavcodec/h264_parse.h | 37 + libavcodec/h264dec.h | 35 --- tests/checkasm/h264dsp.c | 1 + 3 files changed, 38 insertions(+), 35 deletions(-) diff --git a/libavcodec/h264_parse.h b/libavcodec/h264_parse.h index 9f32

[FFmpeg-devel] [PATCH 11/13] lavc/vdpau: stop unnecessarily including h264dec

--- libavcodec/vdpau.c | 1 - 1 file changed, 1 deletion(-) diff --git a/libavcodec/vdpau.c b/libavcodec/vdpau.c index 7c29746adb..f96ac15e2a 100644 --- a/libavcodec/vdpau.c +++ b/libavcodec/vdpau.c @@ -26,7 +26,6 @@ #include "avcodec.h" #include "decode.h" #include "internal.h" -#include "h26

[FFmpeg-devel] [PATCH 12/13] lavc/x86/h264_qpel: stop unnecessarily including h264dec

--- libavcodec/x86/h264_qpel.c | 1 - 1 file changed, 1 deletion(-) diff --git a/libavcodec/x86/h264_qpel.c b/libavcodec/x86/h264_qpel.c index 320d98933a..dda50ded89 100644 --- a/libavcodec/x86/h264_qpel.c +++ b/libavcodec/x86/h264_qpel.c @@ -24,7 +24,6 @@ #include "libavutil/mem_internal.h" #i

[FFmpeg-devel] [PATCH 10/13] lavc/h264: move MB_TYPE defs from h264dec.h to h264_parse

Allows to stop including h264dec.h in h264data.c. --- libavcodec/h264_parse.h | 3 +++ libavcodec/h264data.c | 2 +- libavcodec/h264dec.h| 2 -- 3 files changed, 4 insertions(+), 3 deletions(-) diff --git a/libavcodec/h264_parse.h b/libavcodec/h264_parse.h index 95a80c425f..9b6391edda 10064

[FFmpeg-devel] [PATCH 08/13] lavc/h264data.h: stop including h264dec.h

This header does not need anything from there. --- libavcodec/h264data.h | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/libavcodec/h264data.h b/libavcodec/h264data.h index 2968b08b7e..988352aa9a 100644 --- a/libavcodec/h264data.h +++ b/libavcodec/h264data.h @@ -21,7 +21,7 @@

[FFmpeg-devel] [PATCH 06/13] lavc/h264_parser: add missing headers

--- libavcodec/h264_parser.c | 3 +++ 1 file changed, 3 insertions(+) diff --git a/libavcodec/h264_parser.c b/libavcodec/h264_parser.c index 816b04845e..b221459c1b 100644 --- a/libavcodec/h264_parser.c +++ b/libavcodec/h264_parser.c @@ -40,8 +40,11 @@ #include "get_bits.h" #include "golomb.h"

[FFmpeg-devel] [PATCH 09/13] lavc/h264_parse: stop including h264dec.h

It is unnecessary and only files that are parts of the decoder (as opposed to standalone code called by the decoder) are allowed to include h264dec.h --- libavcodec/h264_parse.c | 4 +++- 1 file changed, 3 insertions(+), 1 deletion(-) diff --git a/libavcodec/h264_parse.c b/libavcodec/h264_parse.c

[FFmpeg-devel] [PATCH 03/13] lavc/h264dec.h: move find_start_code and MMCOOpcode to h264_parse.h

Both parser and decoder use these, so h264_parse is the proper place for them. --- libavcodec/h264_parse.h | 28 libavcodec/h264dec.h| 23 --- 2 files changed, 28 insertions(+), 23 deletions(-) diff --git a/libavcodec/h264_parse.h b/libavcodec/

[FFmpeg-devel] [PATCH 02/13] lavc/h264_parser: stop accessing H264Context

Parsers should not mess with decoder private data. It is also completely unnecessary here. --- libavcodec/h264_parser.c | 8 +++- 1 file changed, 3 insertions(+), 5 deletions(-) diff --git a/libavcodec/h264_parser.c b/libavcodec/h264_parser.c index e3a11702c5..816b04845e 100644 --- a/libavcod

[FFmpeg-devel] [PATCH 05/13] lavc/h264: replace MAX_DELAYED_PIC_COUNT by H264_MAX_DPB_FRAMES

--- libavcodec/h264_ps.c| 4 ++-- libavcodec/h264_slice.c | 12 ++-- libavcodec/h264dec.h| 6 ++ 3 files changed, 10 insertions(+), 12 deletions(-) diff --git a/libavcodec/h264_ps.c b/libavcodec/h264_ps.c index e21c2b56ac..f68d5bf81c 100644 --- a/libavcodec/h264_ps.c +++ b/l

[FFmpeg-devel] [PATCH 04/13] lavc/h264: replace MAX_DELAYED_PIC_COUNT with FF_ARRAY_ELEMS where appropriate

--- libavcodec/h264_refs.c | 2 +- libavcodec/h264_slice.c | 2 +- libavcodec/h264dec.c| 4 ++-- 3 files changed, 4 insertions(+), 4 deletions(-) diff --git a/libavcodec/h264_refs.c b/libavcodec/h264_refs.c index 93526b11e2..94b473257f 100644 --- a/libavcodec/h264_refs.c +++ b/libavcodec/h26

[FFmpeg-devel] [PATCH 01/13] lavc/h264: replace MAX_MMCO_COUNT with H264_MAX_MMCO_COUNT

They apparently serve the same purpose; the latter is one larger and has a comment explaining how the value is derived, so seems more trustworthy. --- libavcodec/h264_parser.c | 2 +- libavcodec/h264_refs.c | 2 +- libavcodec/h264dec.h | 6 ++ 3 files changed, 4 insertions(+), 6 deletion

[FFmpeg-devel] [PATCH] avcodec/h264_parser: don't alter decoder private data

Signed-off-by: James Almer --- libavcodec/h264_parser.c | 7 ++- 1 file changed, 2 insertions(+), 5 deletions(-) diff --git a/libavcodec/h264_parser.c b/libavcodec/h264_parser.c index bb84cea821..cfbf6f4486 100644 --- a/libavcodec/h264_parser.c +++ b/libavcodec/h264_parser.c @@ -247,7 +247,6

Re: [FFmpeg-devel] [PATCH 5/5] Add AudioToolbox audio input device.

Le lun. 24 janv. 2022 à 10:19, Marvin Scholz a écrit : > > > > On 24 Jan 2022, at 16:42, Romain Beauxis wrote: > > > Hi Marvin, > > > > Le mer. 19 janv. 2022 à 10:59, Marvin Scholz a > > écrit : > >> On 19 Jan 2022, at 15:42, Romain Beauxis wrote: > >> > >> Hi, thanks for the patch. I've not done

Re: [FFmpeg-devel] [PATCH 5/5] Add AudioToolbox audio input device.

On 24 Jan 2022, at 16:42, Romain Beauxis wrote: Hi Marvin, Le mer. 19 janv. 2022 à 10:59, Marvin Scholz a écrit : On 19 Jan 2022, at 15:42, Romain Beauxis wrote: Hi, thanks for the patch. I've not done a full code review yet, just a few initial remarks below: This patch adds support f

[FFmpeg-devel] Patch review request

Hi, I have submitted a patch few months ago which makes it possible to load images referenced in image elements through a file URL scheme. Here is the patch: https://patchwork.ffmpeg.org/project/ffmpeg/patch/20210803094534.1000-1-daniel.pe...@tum.de/ The current alternative to display image

Re: [FFmpeg-devel] [PATCH 5/5] Add AudioToolbox audio input device.

Hi Marvin, Le mer. 19 janv. 2022 à 10:59, Marvin Scholz a écrit : > On 19 Jan 2022, at 15:42, Romain Beauxis wrote: > > Hi, thanks for the patch. I've not done a full code review yet, just a > few > initial remarks below: > > > This patch adds support for a new, audio-specific input device using

[FFmpeg-devel] [PATCH] ffmpeg: add packet duration to muxer logging

From: Jaakko Perttilä Especially useful when debugging subtitle output. Signed-off-by: Jan Ekström --- fftools/ffmpeg.c | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/fftools/ffmpeg.c b/fftools/ffmpeg.c index c89d94520f..af65be157e 100644 --- a/fftools/ffmpeg.c +++ b/fft

[FFmpeg-devel] [PATCH v2 31/31] avutil/fifo: Deprecate old FIFO API

From: Anton Khirnov Users should switch to the superior AVFifo API. Unfortunately AVFifoBuffer fields cannot be marked as deprecated because it would trigger a warning wherever fifo.h is #included, due to inlined av_fifo_peek2(). --- doc/APIchanges | 8 libavutil/fifo.c| 4 +

[FFmpeg-devel] [PATCH v2 30/31] ffmpeg: switch to new FIFO API

From: Anton Khirnov --- fftools/ffmpeg.c| 69 - fftools/ffmpeg.h| 6 ++-- fftools/ffmpeg_filter.c | 14 - fftools/ffmpeg_opt.c| 4 +-- 4 files changed, 37 insertions(+), 56 deletions(-) diff --git a/fftools/ffmpeg.c b/fftools

[FFmpeg-devel] [PATCH v2 29/31] ffplay: switch to new FIFO API

From: Anton Khirnov --- fftools/ffplay.c | 22 +- 1 file changed, 9 insertions(+), 13 deletions(-) diff --git a/fftools/ffplay.c b/fftools/ffplay.c index e7b20be76b..ac48d8765d 100644 --- a/fftools/ffplay.c +++ b/fftools/ffplay.c @@ -115,7 +115,7 @@ typedef struct MyAVPacket

[FFmpeg-devel] [PATCH v2 28/31] lavfi/vf_deshake_opencl: switch to new FIFO API

From: Anton Khirnov --- libavfilter/vf_deshake_opencl.c | 92 +++-- 1 file changed, 29 insertions(+), 63 deletions(-) diff --git a/libavfilter/vf_deshake_opencl.c b/libavfilter/vf_deshake_opencl.c index 9c761ba5ad..c2b5bef897 100644 --- a/libavfilter/vf_deshake_openc

[FFmpeg-devel] [PATCH v2 27/31] lavfi/qsvvpp: switch to new FIFO API

From: Anton Khirnov --- libavfilter/qsvvpp.c | 46 ++-- libavfilter/qsvvpp.h | 2 +- 2 files changed, 20 insertions(+), 28 deletions(-) diff --git a/libavfilter/qsvvpp.c b/libavfilter/qsvvpp.c index d1218355c7..35769dfd60 100644 --- a/libavfilter/qsvvpp.

[FFmpeg-devel] [PATCH v2 26/31] lavu/threadmessage: switch to new FIFO API

From: Anton Khirnov --- libavutil/threadmessage.c | 38 +++--- 1 file changed, 19 insertions(+), 19 deletions(-) diff --git a/libavutil/threadmessage.c b/libavutil/threadmessage.c index 764b7fb813..6f25da76d7 100644 --- a/libavutil/threadmessage.c +++ b/libavutil

[FFmpeg-devel] [PATCH v2 25/31] lavu/audio_fifo: switch to new FIFO API

From: Anton Khirnov --- libavutil/audio_fifo.c | 44 -- 1 file changed, 17 insertions(+), 27 deletions(-) diff --git a/libavutil/audio_fifo.c b/libavutil/audio_fifo.c index 243efc39e4..b1355e55a0 100644 --- a/libavutil/audio_fifo.c +++ b/libavutil/audio_f

[FFmpeg-devel] [PATCH v2 24/31] lavu/audio_fifo: drop an unnecessary include

From: Anton Khirnov Nothing in audio_fifo.h uses anything from fifo.h --- libavutil/audio_fifo.h | 1 - 1 file changed, 1 deletion(-) diff --git a/libavutil/audio_fifo.h b/libavutil/audio_fifo.h index d8a9194a8d..9d570b04c0 100644 --- a/libavutil/audio_fifo.h +++ b/libavutil/audio_fifo.h @@ -28

[FFmpeg-devel] [PATCH v2 23/31] lavd/jack: switch to the new FIFO API

From: Anton Khirnov --- libavdevice/jack.c | 30 +++--- 1 file changed, 15 insertions(+), 15 deletions(-) diff --git a/libavdevice/jack.c b/libavdevice/jack.c index 0d5465e407..b9a167bed3 100644 --- a/libavdevice/jack.c +++ b/libavdevice/jack.c @@ -50,8 +50,8 @@ typedef

[FFmpeg-devel] [PATCH v2 22/31] lavf/async: switch to new FIFO API

From: Anton Khirnov --- libavformat/async.c | 68 ++--- 1 file changed, 33 insertions(+), 35 deletions(-) diff --git a/libavformat/async.c b/libavformat/async.c index 5a81507ef1..547417aa1e 100644 --- a/libavformat/async.c +++ b/libavformat/async.c @@ -47

[FFmpeg-devel] [PATCH v2 21/31] lavf/udp: switch to new FIFO API

From: Anton Khirnov --- libavformat/udp.c | 34 +- 1 file changed, 17 insertions(+), 17 deletions(-) diff --git a/libavformat/udp.c b/libavformat/udp.c index 83c042d079..1737aee1c2 100644 --- a/libavformat/udp.c +++ b/libavformat/udp.c @@ -98,7 +98,7 @@ typedef s

[FFmpeg-devel] [PATCH v2 20/31] lavf/swfenc: switch to new FIFO API

From: Anton Khirnov --- libavformat/swfenc.c | 22 ++ 1 file changed, 14 insertions(+), 8 deletions(-) diff --git a/libavformat/swfenc.c b/libavformat/swfenc.c index 1fd2ad81a3..9eb22ee9b3 100644 --- a/libavformat/swfenc.c +++ b/libavformat/swfenc.c @@ -38,7 +38,7 @@ typedef

[FFmpeg-devel] [PATCH v2 19/31] lavf/mpegenc: switch to new FIFO API

From: Anton Khirnov --- libavformat/mpegenc.c | 40 +++- 1 file changed, 23 insertions(+), 17 deletions(-) diff --git a/libavformat/mpegenc.c b/libavformat/mpegenc.c index b1d8bf9c38..64248695bd 100644 --- a/libavformat/mpegenc.c +++ b/libavformat/mpegenc.c @

[FFmpeg-devel] [PATCH v2 18/31] lavf/dvenc: switch to new FIFO API

From: Anton Khirnov --- libavformat/dvenc.c | 22 -- 1 file changed, 12 insertions(+), 10 deletions(-) diff --git a/libavformat/dvenc.c b/libavformat/dvenc.c index 03b63cff89..e1188109c6 100644 --- a/libavformat/dvenc.c +++ b/libavformat/dvenc.c @@ -49,7 +49,7 @@ struct DVMu

[FFmpeg-devel] [PATCH v2 17/31] lavf/dvenc: return an error on audio/video desync

From: Anton Khirnov --- libavformat/dvenc.c | 8 ++-- 1 file changed, 6 insertions(+), 2 deletions(-) diff --git a/libavformat/dvenc.c b/libavformat/dvenc.c index b76539b59f..03b63cff89 100644 --- a/libavformat/dvenc.c +++ b/libavformat/dvenc.c @@ -255,8 +255,10 @@ static int dv_assemble_fr

[FFmpeg-devel] [PATCH v2 16/31] avcodec/qsvenc: Reindent after the previous commit

Signed-off-by: Andreas Rheinhardt --- libavcodec/qsvenc.c | 20 ++-- 1 file changed, 10 insertions(+), 10 deletions(-) diff --git a/libavcodec/qsvenc.c b/libavcodec/qsvenc.c index a12f7bce42..a1093ae768 100644 --- a/libavcodec/qsvenc.c +++ b/libavcodec/qsvenc.c @@ -1832,17 +1832,

[FFmpeg-devel] [PATCH v2 15/31] lavc/qsvenc: switch to new FIFO API

From: Anton Khirnov --- libavcodec/qsvenc.c | 126 +++- libavcodec/qsvenc.h | 2 +- 2 files changed, 55 insertions(+), 73 deletions(-) diff --git a/libavcodec/qsvenc.c b/libavcodec/qsvenc.c index acb82f321c..a12f7bce42 100644 --- a/libavcodec/qsvenc.c +

[FFmpeg-devel] [PATCH v2 14/31] lavc/qsvdec: switch to the new FIFO API

From: Anton Khirnov --- libavcodec/qsvdec.c | 88 ++--- 1 file changed, 35 insertions(+), 53 deletions(-) diff --git a/libavcodec/qsvdec.c b/libavcodec/qsvdec.c index d9e0fef1f1..56cd5c86c9 100644 --- a/libavcodec/qsvdec.c +++ b/libavcodec/qsvdec.c @@ -56

[FFmpeg-devel] [PATCH v2 13/31] lavc/nvenc: switch to the new FIFO API

From: Anton Khirnov --- libavcodec/nvenc.c | 49 ++ libavcodec/nvenc.h | 8 2 files changed, 27 insertions(+), 30 deletions(-) diff --git a/libavcodec/nvenc.c b/libavcodec/nvenc.c index 850c46022b..effd6381da 100644 --- a/libavcodec/nvenc.c

[FFmpeg-devel] [PATCH v2 12/31] lavc/libvpxenc: remove unneeded context variable

From: Anton Khirnov discard_hdr10_plus is 0 IFF hdr10_plus_fifo is non-NULL, so we can test for the latter and avoid an extra variable. --- libavcodec/libvpxenc.c | 7 ++- 1 file changed, 2 insertions(+), 5 deletions(-) diff --git a/libavcodec/libvpxenc.c b/libavcodec/libvpxenc.c index ab5d

[FFmpeg-devel] [PATCH v2 11/31] lavc/libvpxenc: switch to the new FIFO API

From: Anton Khirnov --- libavcodec/libvpxenc.c | 35 ++- 1 file changed, 10 insertions(+), 25 deletions(-) diff --git a/libavcodec/libvpxenc.c b/libavcodec/libvpxenc.c index 10e5a22fa9..ab5d31e4c4 100644 --- a/libavcodec/libvpxenc.c +++ b/libavcodec/libvpxenc.c @

[FFmpeg-devel] [PATCH v2 10/31] lavc/libvorbisenc: switch to new FIFO API

From: Anton Khirnov --- libavcodec/libvorbisenc.c | 20 +--- 1 file changed, 9 insertions(+), 11 deletions(-) diff --git a/libavcodec/libvorbisenc.c b/libavcodec/libvorbisenc.c index fa0d5f4b42..b657f0157a 100644 --- a/libavcodec/libvorbisenc.c +++ b/libavcodec/libvorbisenc.c @@

[FFmpeg-devel] [PATCH v2 09/31] lavc/cuviddec: convert to the new FIFO API

From: Anton Khirnov --- libavcodec/cuviddec.c | 22 ++ 1 file changed, 10 insertions(+), 12 deletions(-) diff --git a/libavcodec/cuviddec.c b/libavcodec/cuviddec.c index b1a3d674ab..1b525cd804 100644 --- a/libavcodec/cuviddec.c +++ b/libavcodec/cuviddec.c @@ -78,7 +78,7 @@ t

[FFmpeg-devel] [PATCH v2 08/31] lavc/cuviddec: do not reallocate the fifo unnecessarily

From: Anton Khirnov --- libavcodec/cuviddec.c | 8 +--- 1 file changed, 1 insertion(+), 7 deletions(-) diff --git a/libavcodec/cuviddec.c b/libavcodec/cuviddec.c index f03bbd8c4b..b1a3d674ab 100644 --- a/libavcodec/cuviddec.c +++ b/libavcodec/cuviddec.c @@ -1030,13 +1030,7 @@ static void cu

[FFmpeg-devel] [PATCH v2 07/31] lavc/amfenc: switch to new FIFO API

From: Anton Khirnov --- libavcodec/amfenc.c | 43 ++- libavcodec/amfenc.h | 2 +- 2 files changed, 15 insertions(+), 30 deletions(-) diff --git a/libavcodec/amfenc.c b/libavcodec/amfenc.c index fb23ed738c..0e5117c5a3 100644 --- a/libavcodec/amfenc.c +++

[FFmpeg-devel] [PATCH v2 06/31] lavc/avcodec: switch to new FIFO API

From: Anton Khirnov --- libavcodec/avcodec.c | 17 ++--- libavcodec/decode.c | 24 +--- libavcodec/internal.h | 2 +- 3 files changed, 16 insertions(+), 27 deletions(-) diff --git a/libavcodec/avcodec.c b/libavcodec/avcodec.c index c00a9b2af8..4df834c708 1006

[FFmpeg-devel] [PATCH v2 05/31] lavu/tests/fifo: switch to the new API

From: Anton Khirnov --- libavutil/tests/fifo.c | 47 +++--- 1 file changed, 21 insertions(+), 26 deletions(-) diff --git a/libavutil/tests/fifo.c b/libavutil/tests/fifo.c index a17d913233..579602ccf3 100644 --- a/libavutil/tests/fifo.c +++ b/libavutil/tests/f

[FFmpeg-devel] [PATCH v2 04/31] lavu/fifo: add a flag for automatically growing the FIFO as needed

From: Anton Khirnov This will not increase the FIFO beyond 1MB, unless the caller explicitly specifies otherwise. --- doc/APIchanges | 2 +- libavutil/fifo.c | 39 +-- libavutil/fifo.h | 15 ++- 3 files changed, 52 insertions(+), 4 deletions(-)

[FFmpeg-devel] [PATCH v2 03/31] lavu/fifo: Add new AVFifo API based upon the notion of element size

From: Anton Khirnov Many AVFifoBuffer users operate on fixed-size elements (e.g. pointers), but the current FIFO API deals exclusively in bytes, requiring extra complexity in all these callers. Add a new AVFifo API creating a FIFO with an element size that may be larger than a byte. All operatio

[FFmpeg-devel] [PATCH v2 02/31] lavu/fifo: disallow overly large fifo sizes

From: Anton Khirnov The API currently allows creating FIFOs up to - UINT_MAX: av_fifo_alloc(), av_fifo_realloc(), av_fifo_grow() - SIZE_MAX: av_fifo_alloc_array() However the usable limit is determined by - rndx/wndx being uint32_t - av_fifo_[size,space] returning int so no FIFO should be larger

[FFmpeg-devel] [PATCH v2 01/31] avutil/fifo: Use av_fifo_generic_peek_at() for av_fifo_generic_peek()

Avoids code duplication. It furthermore properly checks for buf_size to be > 0 before doing anything. Signed-off-by: Andreas Rheinhardt --- libavutil/fifo.c | 21 + 1 file changed, 1 insertion(+), 20 deletions(-) diff --git a/libavutil/fifo.c b/libavutil/fifo.c index d741bdd

[FFmpeg-devel] [PATCH v2 00/31] New FIFO API

This is a modified version of Anton's earlier patchset [1]. The main changes are that the new FIFO API is a clean break with the old API (the new API uses a new structure (called AVFifo) instead of sharing AVFifoBuffer with the old API) and that it is now documented that one may attempt to read/wri

Re: [FFmpeg-devel] [GASPP PATCH] Filter out -D parameters from the call to GCC/Clang to assemble

On Tue, 18 Jan 2022, Martin Storsjö wrote: Clang warns about unused -D parameters when operating on .s files (or if invoked with "-x assembler") while GCC doesn't. --- As noobdy else than me essentially maintains gas-preprocessor, I'll go ahead and push this after a couple days if nobody speaks

Re: [FFmpeg-devel] [PATCH] Fix setsockopt IP_MULTICAST_TTL on OpenBSD

On Sun, Jan 23, 2022 at 02:55:38PM -0500, Brad Smith wrote: > On 1/23/2022 6:57 AM, Michael Niedermayer wrote: > > > On Wed, Jan 12, 2022 at 12:13:13AM -0500, Brad Smith wrote: > > > Fix setsockopt() usage on OpenBSD with IP_MULTICAST_TTL. The field > > > type should be an unsigned char on anythin

Re: [FFmpeg-devel] [PATCH v4] tests: add test for ffmpeg's fix_sub_duration feature

On Mon, Jan 24, 2022 at 12:43 PM Jan Ekström wrote: > > On Mon, Jan 10, 2022 at 3:28 PM Jan Ekström wrote: > > > > From: Jan Ekström > > > > This long-existing feature calculates subtitle durations by keeping > > it around until the following subtitle is decoded, and then utilizes > > the follow

Re: [FFmpeg-devel] [PATCH] avfilter: Added siti filter

Am 18.01.22 um 14:58 schrieb Thilo Borgmann: Am 15.01.22 um 09:27 schrieb Paul B Mahol: On Sat, Jan 15, 2022 at 9:25 AM Paul B Mahol wrote: On Fri, Jan 14, 2022 at 5:22 PM Thilo Borgmann wrote: Hi, Am 19.01.21 um 05:49 schrieb Lynne: Jan 19, 2021, 01:07 by borba...@fb.com: Calculate

Re: [FFmpeg-devel] [PATCH v4] tests: add test for ffmpeg's fix_sub_duration feature

On Mon, Jan 10, 2022 at 3:28 PM Jan Ekström wrote: > > From: Jan Ekström > > This long-existing feature calculates subtitle durations by keeping > it around until the following subtitle is decoded, and then utilizes > the following subtitle's pts as the end point of the previous one. > As there

[FFmpeg-devel] [PATCH v2 3/3] lavc/qsvdec: export AVFilmGrainParams side data

From: Haihao Xiang When AV_CODEC_EXPORT_DATA_FILM_GRAIN is present, AV1 decoder should disable film grain application and export the corresponding side data Signed-off-by: Haihao Xiang --- libavcodec/qsv_internal.h | 3 ++ libavcodec/qsvdec.c | 91 +++

[FFmpeg-devel] [PATCH v2 2/3] lavc/qsvdec: track the runtime session version

From: Haihao Xiang We may check the runtime version for the given features Signed-off-by: Haihao Xiang --- libavcodec/qsvdec.c | 13 + 1 file changed, 13 insertions(+) diff --git a/libavcodec/qsvdec.c b/libavcodec/qsvdec.c index 783d252002..8b83d5695f 100644 --- a/libavcodec/qsvde

[FFmpeg-devel] [PATCH v2 1/3] lavc/qsv: allow to add more parameter buffers to QSV frame

From: Haihao Xiang Signed-off-by: Haihao Xiang --- v2: rebased the patchset against the latest FFmpeg and added code to make sure the corresponding extra parameter buffer is added for AV1 only. libavcodec/qsv.c | 27 +++ libavcodec/qsv_internal.h | 8 +++-