Re: [FFmpeg-devel] Update for my hls and dash patches?

2023-01-04 Thread Steven Liu
Basel Sayeh 于2023年1月5日周四 09:19写道: > > > > > > > > Hello, > > > > > > > > > Any update or comment on these patch sets? > > > > > > > > > Enable HTTP persistent connections for hls_delete_file & > > > dashenc_delete_file: > > > https://patchwork.ffmpeg.org/project/ffmpeg/list/?series=8033 > > >

Re: [FFmpeg-devel] [PATCH v2] vaapi: support VAProfileH264High10

2023-01-04 Thread Xiang, Haihao
On Di, 2022-12-27 at 11:12 +0800, Jianfeng Zheng wrote: > see https://github.com/intel/libva/pull/664 > > Signed-off-by: jianfeng.zheng > --- > configure | 13 + > libavcodec/h264_slice.c| 6 ++ > libavcodec/vaapi_decode.c | 10 ++ >

[FFmpeg-devel] [PATCH] lavf/rtpenc_jpeg: Retrieve sampling factor from SOF0

2023-01-04 Thread Yeoh, Hoong Tee
In ff_rtp_send_jpeg, the type is defined based on PIX_FMT and color-range parsed in. There is limitation on current design where need to include support newly introduced PIX_FMT such as AV_PIX_FMT_QSV and there might be more and more in future. Hence, retrive the sampling factor from SOF0 in

Re: [FFmpeg-devel] Update for my hls and dash patches?

2023-01-04 Thread Basel Sayeh
> > > > Hello, > > > > > > Any update or comment on these patch sets? > > > > > > Enable HTTP persistent connections for hls_delete_file & > > dashenc_delete_file: > > https://patchwork.ffmpeg.org/project/ffmpeg/list/?series=8033 >

Re: [FFmpeg-devel] [PATCH V2 1/2] lavfi/dnn: Modify error message for incorrect backend_type

2023-01-04 Thread Guo, Yejun
> -Original Message- > From: ffmpeg-devel On Behalf Of > Ting Fu > Sent: Monday, January 2, 2023 11:50 PM > To: ffmpeg-devel@ffmpeg.org > Subject: [FFmpeg-devel] [PATCH V2 1/2] lavfi/dnn: Modify error message for > incorrect backend_type > > Signed-off-by: Ting Fu > --- >

Re: [FFmpeg-devel] [PATCH 2/8] fftools/ffmpeg: stop using AVCodecContext.sample_rate in decode_audio()

2023-01-04 Thread Michael Niedermayer
On Wed, Jan 04, 2023 at 05:42:40PM +0100, Anton Khirnov wrote: > Use the decoded frame's sample_rate instead, which is the authoritative > value. > > Drop a now-obsolete check validating AVCodecContext.sample_rate. > --- > fftools/ffmpeg.c | 15 ++- > 1 file changed, 6 insertions(+),

[FFmpeg-devel] [PATCH 2/2] avformat/movenc: Add loop parameter to animated AVIF

2023-01-04 Thread Vignesh Venkatasubramanian
The HEIF specification permits specifying the looping behavior of animated sequences by using the EditList (elst) box. The track duration will be set to the total duration of all the loops (or infinite) and the duration of a single loop will be set in the edit list box. The default behavior is to

[FFmpeg-devel] [PATCH 1/2] avformat/movenc: Add movie_timescale option to AVIF

2023-01-04 Thread Vignesh Venkatasubramanian
Allow specifying the movie_timescale options to AVIF ouptut. This also makes sure that when movie_timescale is not specified, the default value of 1000 is used instead of 0. Animated AVIF files which don't specify the movie_timescale will have the correct duration written in the track and movie

Re: [FFmpeg-devel] [PATCH 8/8] fftools/ffmpeg: always generate CFR output when -r is used

2023-01-04 Thread Anton Khirnov
Quoting Paul B Mahol (2023-01-04 18:37:31) > On Wed, Jan 4, 2023 at 5:46 PM Anton Khirnov wrote: > > > Current code may, depending on the muxer, decide to use VSYNC_VFR tagged > > with the specified framerate, without actually performing framerate > > conversion. This is clearly wrong and

Re: [FFmpeg-devel] [PATCH] swresample fixes

2023-01-04 Thread Andreas Rheinhardt
Paul B Mahol: > On Wed, Jan 4, 2023 at 6:26 PM Andreas Rheinhardt < > andreas.rheinha...@outlook.com> wrote: > >> Paul B Mahol: >>> diff --git a/libavfilter/af_aresample.c b/libavfilter/af_aresample.c >>> index 7923377c8c..2744388f75 100644 >>> --- a/libavfilter/af_aresample.c >>> +++

Re: [FFmpeg-devel] [PATCH 8/8] fftools/ffmpeg: always generate CFR output when -r is used

2023-01-04 Thread Paul B Mahol
On Wed, Jan 4, 2023 at 5:46 PM Anton Khirnov wrote: > Current code may, depending on the muxer, decide to use VSYNC_VFR tagged > with the specified framerate, without actually performing framerate > conversion. This is clearly wrong and against the documentation, which > states unambiguously

Re: [FFmpeg-devel] [PATCH] swresample fixes

2023-01-04 Thread Paul B Mahol
On Wed, Jan 4, 2023 at 6:26 PM Andreas Rheinhardt < andreas.rheinha...@outlook.com> wrote: > Paul B Mahol: > > diff --git a/libavfilter/af_aresample.c b/libavfilter/af_aresample.c > > index 7923377c8c..2744388f75 100644 > > --- a/libavfilter/af_aresample.c > > +++ b/libavfilter/af_aresample.c > >

Re: [FFmpeg-devel] [PATCH 23/30] lavc/ffv1enc: handle frame durations and AV_CODEC_CAP_ENCODER_REORDERED_OPAQUE

2023-01-04 Thread Andreas Rheinhardt
Anton Khirnov: > --- > libavcodec/ffv1enc.c | 16 +++- > 1 file changed, 15 insertions(+), 1 deletion(-) > > diff --git a/libavcodec/ffv1enc.c b/libavcodec/ffv1enc.c > index 0237ac48eb..553abb558f 100644 > --- a/libavcodec/ffv1enc.c > +++ b/libavcodec/ffv1enc.c > @@ -1233,7 +1233,20

Re: [FFmpeg-devel] [PATCH] swresample fixes

2023-01-04 Thread Andreas Rheinhardt
Paul B Mahol: > diff --git a/libavfilter/af_aresample.c b/libavfilter/af_aresample.c > index 7923377c8c..2744388f75 100644 > --- a/libavfilter/af_aresample.c > +++ b/libavfilter/af_aresample.c > @@ -209,8 +209,14 @@ FF_ENABLE_DEPRECATION_WARNINGS > } else { > outsamplesref->pts =

Re: [FFmpeg-devel] [PATCH] swresample fixes

2023-01-04 Thread Paul B Mahol
On Wed, Jan 4, 2023 at 6:18 PM Andreas Rheinhardt < andreas.rheinha...@outlook.com> wrote: > Paul B Mahol: > > diff --git a/libavfilter/af_aresample.c b/libavfilter/af_aresample.c > > index 971c861d0e..7923377c8c 100644 > > --- a/libavfilter/af_aresample.c > > +++ b/libavfilter/af_aresample.c > >

Re: [FFmpeg-devel] [PATCH] swresample fixes

2023-01-04 Thread Andreas Rheinhardt
Paul B Mahol: > diff --git a/libavfilter/af_aresample.c b/libavfilter/af_aresample.c > index 971c861d0e..7923377c8c 100644 > --- a/libavfilter/af_aresample.c > +++ b/libavfilter/af_aresample.c > @@ -209,18 +209,16 @@ FF_ENABLE_DEPRECATION_WARNINGS > } else { > outsamplesref->pts =

Re: [FFmpeg-devel] [PATCH] swresample fixes

2023-01-04 Thread James Almer
On 1/4/2023 1:59 PM, Paul B Mahol wrote: From 0415ed37bee0c2b640920edad87ec927dda95fb5 Mon Sep 17 00:00:00 2001 From: Paul B Mahol Date: Wed, 4 Jan 2023 17:53:01 +0100 Subject: [PATCH 2/3] swresample/swresample_frame: fix regression in detecting changes Do not overwrite return variable

Re: [FFmpeg-devel] [PATCH] swresample fixes

2023-01-04 Thread Andreas Rheinhardt
Paul B Mahol: > diff --git a/libswresample/swresample_frame.c > b/libswresample/swresample_frame.c > index 53ac487136..319ce045a1 100644 > --- a/libswresample/swresample_frame.c > +++ b/libswresample/swresample_frame.c > @@ -84,7 +84,7 @@ static int config_changed(SwrContext *s, >

[FFmpeg-devel] [PATCH] swresample fixes

2023-01-04 Thread Paul B Mahol
Patches attached. From 3959bcb707f52339bac41acc9aec856cad3aced1 Mon Sep 17 00:00:00 2001 From: Paul B Mahol Date: Wed, 4 Jan 2023 17:55:10 +0100 Subject: [PATCH 3/3] avfilter/af_aresample: if frame parameters change update swr context Signed-off-by: Paul B Mahol --- libavfilter/af_aresample.c

[FFmpeg-devel] [PATCH 4/8] fftools/ffmpeg: rename a variable to be more descriptive

2023-01-04 Thread Anton Khirnov
--- fftools/ffmpeg.c | 26 ++ 1 file changed, 14 insertions(+), 12 deletions(-) diff --git a/fftools/ffmpeg.c b/fftools/ffmpeg.c index 00e4be68ea..66a24a1a9c 100644 --- a/fftools/ffmpeg.c +++ b/fftools/ffmpeg.c @@ -1088,7 +1088,7 @@ static void do_video_out(OutputFile

[FFmpeg-devel] [PATCH 8/8] fftools/ffmpeg: always generate CFR output when -r is used

2023-01-04 Thread Anton Khirnov
Current code may, depending on the muxer, decide to use VSYNC_VFR tagged with the specified framerate, without actually performing framerate conversion. This is clearly wrong and against the documentation, which states unambiguously that -r should produce CFR output for video encoding. FATE test

[FFmpeg-devel] [PATCH 6/8] fftools/ffmpeg: reindent after previous commit

2023-01-04 Thread Anton Khirnov
--- fftools/ffmpeg.c | 116 +++ 1 file changed, 58 insertions(+), 58 deletions(-) diff --git a/fftools/ffmpeg.c b/fftools/ffmpeg.c index 04de57e41e..f722ae7632 100644 --- a/fftools/ffmpeg.c +++ b/fftools/ffmpeg.c @@ -1040,67 +1040,67 @@ static void

[FFmpeg-devel] [PATCH 7/8] doc/ffmpeg: improve -r documentation

2023-01-04 Thread Anton Khirnov
Explain different behavior for encoding and streamcopy. --- doc/ffmpeg.texi | 13 - 1 file changed, 12 insertions(+), 1 deletion(-) diff --git a/doc/ffmpeg.texi b/doc/ffmpeg.texi index 65634b82cf..67b3294256 100644 --- a/doc/ffmpeg.texi +++ b/doc/ffmpeg.texi @@ -877,9 +877,20 @@ This

[FFmpeg-devel] [PATCH 2/8] fftools/ffmpeg: stop using AVCodecContext.sample_rate in decode_audio()

2023-01-04 Thread Anton Khirnov
Use the decoded frame's sample_rate instead, which is the authoritative value. Drop a now-obsolete check validating AVCodecContext.sample_rate. --- fftools/ffmpeg.c | 15 ++- 1 file changed, 6 insertions(+), 9 deletions(-) diff --git a/fftools/ffmpeg.c b/fftools/ffmpeg.c index

[FFmpeg-devel] [PATCH 3/8] fftools/ffmpeg: fix stream id in an error message.

2023-01-04 Thread Anton Khirnov
Broken in 7ef7a22251b8 --- fftools/ffmpeg.c | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/fftools/ffmpeg.c b/fftools/ffmpeg.c index 2dbfeca020..00e4be68ea 100644 --- a/fftools/ffmpeg.c +++ b/fftools/ffmpeg.c @@ -3797,7 +3797,8 @@ static int transcode(void)

[FFmpeg-devel] [PATCH 5/8] fftools/ffmpeg: move video frame dup/drop logic into its own function

2023-01-04 Thread Anton Khirnov
--- fftools/ffmpeg.c | 137 ++- 1 file changed, 75 insertions(+), 62 deletions(-) diff --git a/fftools/ffmpeg.c b/fftools/ffmpeg.c index 66a24a1a9c..04de57e41e 100644 --- a/fftools/ffmpeg.c +++ b/fftools/ffmpeg.c @@ -1030,6 +1030,79 @@ static void

[FFmpeg-devel] [PATCH 1/8] doc/ffmpeg.texi: drop a non-existent option

2023-01-04 Thread Anton Khirnov
-ilme has not existed for 17 years, since 637b5326f3441b53e2f1004085c4d570ba2d7758 --- doc/ffmpeg.texi | 6 -- 1 file changed, 6 deletions(-) diff --git a/doc/ffmpeg.texi b/doc/ffmpeg.texi index 0367930a3b..65634b82cf 100644 --- a/doc/ffmpeg.texi +++ b/doc/ffmpeg.texi @@ -1011,12 +1011,6 @@

Re: [FFmpeg-devel] [PATCH 22/30] lavc/adxenc: support AV_CODEC_CAP_ENCODER_REORDERED_OPAQUE

2023-01-04 Thread Andreas Rheinhardt
Anton Khirnov: > --- > libavcodec/adxenc.c | 15 ++- > 1 file changed, 14 insertions(+), 1 deletion(-) > > diff --git a/libavcodec/adxenc.c b/libavcodec/adxenc.c > index e1bf344b01..9e0ceb61eb 100644 > --- a/libavcodec/adxenc.c > +++ b/libavcodec/adxenc.c > @@ -185,6 +185,18 @@

Re: [FFmpeg-devel] [PATCH 30/30] lavc/libaomenc: pass through frame durations to encoded packets

2023-01-04 Thread Anton Khirnov
Quoting James Almer (2022-11-27 21:19:33) > On 11/27/2022 2:03 PM, Anton Khirnov wrote: > > --- > > libavcodec/libaomenc.c | 7 +-- > > 1 file changed, 5 insertions(+), 2 deletions(-) > > > > diff --git a/libavcodec/libaomenc.c b/libavcodec/libaomenc.c > > index bd576fdd3a..c117ca4ee6

Re: [FFmpeg-devel] [Internet]Re: [PATCH v2 5/7] avcodec/mediacodecenc: remove the strategy to create DTS

2023-01-04 Thread Zhao Zhili
> -Original Message- > From: ffmpeg-devel On Behalf Of Anton > Khirnov > Sent: 2023年1月4日 23:16 > To: 'FFmpeg development discussions and patches' > Subject: Re: [FFmpeg-devel] [Internet]Re: [PATCH v2 5/7] > avcodec/mediacodecenc: remove the strategy to create DTS > > Quoting Zhao

Re: [FFmpeg-devel] [Internet]Re: [PATCH v2 5/7] avcodec/mediacodecenc: remove the strategy to create DTS

2023-01-04 Thread Anton Khirnov
Quoting Zhao Zhili (2023-01-04 15:46:52) > > > From: ffmpeg-devel On Behalf Of Tomas > > Härdin > > Sent: 2023年1月4日 22:00 > > To: FFmpeg development discussions and patches > > Subject: Re: [FFmpeg-devel] [Internet]Re: [PATCH v2 5/7] > > avcodec/mediacodecenc: remove the strategy to create

Re: [FFmpeg-devel] [Internet]Re: [PATCH v2 5/7] avcodec/mediacodecenc: remove the strategy to create DTS

2023-01-04 Thread Zhao Zhili
> From: ffmpeg-devel On Behalf Of Tomas Härdin > Sent: 2023年1月4日 22:00 > To: FFmpeg development discussions and patches > Subject: Re: [FFmpeg-devel] [Internet]Re: [PATCH v2 5/7] > avcodec/mediacodecenc: remove the strategy to create DTS > > ons 2023-01-04 klockan 19:31 +0800 skrev

Re: [FFmpeg-devel] [Internet]Re: [PATCH v2 5/7] avcodec/mediacodecenc: remove the strategy to create DTS

2023-01-04 Thread Tomas Härdin
ons 2023-01-04 klockan 19:31 +0800 skrev zhilizhao(赵志立): > > > On Jan 4, 2023, at 18:16, Anton Khirnov wrote: > > > > Does this mean the encoder will produce packets with > > dts=AV_NOPTS_VALUE? > > MediaCodec should not encode B frames by default, so dts = pts by > default. > B frames can be

[FFmpeg-devel] [PATCH] avformat/oggparsevorbis: Don't append to title metadata

2023-01-04 Thread Zsolt Vadász
This patch fixes #9642. Signed-off-by: Zsolt Vadasz --- libavformat/oggparsevorbis.c | 10 +++--- 1 file changed, 7 insertions(+), 3 deletions(-) diff --git a/libavformat/oggparsevorbis.c b/libavformat/oggparsevorbis.c index 061840c2ed..02be4e07b8 100644 --- a/libavformat/oggparsevorbis.c

Re: [FFmpeg-devel] [Internet]Re: [PATCH v2 5/7] avcodec/mediacodecenc: remove the strategy to create DTS

2023-01-04 Thread zhilizhao(赵志立)
> On Jan 4, 2023, at 18:16, Anton Khirnov wrote: > > Does this mean the encoder will produce packets with dts=AV_NOPTS_VALUE? MediaCodec should not encode B frames by default, so dts = pts by default. B frames can be enabled explicitly, in that case dts is AV_NOPTS_VALUE. Android system’s MP4

Re: [FFmpeg-devel] [PATCH] avformat/rtpproto: add support for RTP/UDP socket reuse

2023-01-04 Thread Camille Oudot
> Because it is an useful option. Only not for this use case. > Because nobody noticed it was badly named when it was applied. > > > Why would it not apply to "rtp" also? > > Because nobody made the UDP socket context a child object of the RTP > context. So does it still makes sense to have a

Re: [FFmpeg-devel] [PATCH] avcodec/mpeg12dec: flush a53 data

2023-01-04 Thread Anton Khirnov
Quoting Aman Karmani (2023-01-02 20:18:36) > From: Aman Karmani > > Signed-off-by: Aman Karmani > --- > avcodec/mpeg12dec: flush a53 data > > Published-As: > https://github.com/ffstaging/FFmpeg/releases/tag/pr-ffstaging-45%2Ftmm1%2Fmpeg2-a53-flush-v1 > Fetch-It-Via: git fetch

Re: [FFmpeg-devel] [PATCH v2 5/7] avcodec/mediacodecenc: remove the strategy to create DTS

2023-01-04 Thread Anton Khirnov
Does this mean the encoder will produce packets with dts=AV_NOPTS_VALUE? -- Anton Khirnov ___ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org https://ffmpeg.org/mailman/listinfo/ffmpeg-devel To unsubscribe, visit link above, or email