Re: [FFmpeg-devel] [PATCH 3/4] lavc/vvc_parser: Fixed the has_b_frames setting

2024-04-20 Thread myp...@gmail.com
On Sat, Apr 20, 2024 at 11:50 AM Nuo Mi wrote: > > On Fri, Apr 19, 2024 at 9:55 PM Jun Zhao wrote: > > > From: Jun Zhao > > > > has_b_frames used in decoder for size of the frame reordering > > buffer, setting this field from dpb_max_num_reorder_pics. > > > > Signed-off-by: Jun Zhao > > --- >

Re: [FFmpeg-devel] [PATCH 4/4] lavc/vvc/refs: Use dpb_max_num_reorder_pics to control output

2024-04-20 Thread myp...@gmail.com
Send it privately via email On Sat, Apr 20, 2024 at 11:55 AM Nuo Mi wrote: > > Hi Barry and Eliny, > LGTM. > Thank you for the patch. > Is it possible to provide the clip so we can add it to our CI? > > ci like https://github.com/ffvvc/FFmpeg/actions > > > On Fri, Apr 19, 2024 at 9:48 PM Jun

[FFmpeg-devel] [PATCH 2/2] avformat/demux: extract extradata from packets when context update is requested

2024-04-20 Thread James Almer
If the demuxer doesn't set extradata in the stream's codecpar, a need_context_update request will delete the previously extracted extradata in the stream's internal AVCodecContext. As we can't ensure the old extradata is valid for the stream in its post context update request state, try to get

[FFmpeg-devel] [PATCH 1/2] avcodec/codec_par: always clear extradata_size in avcodec_parameters_to_context()

2024-04-20 Thread James Almer
Missed in d383ae43c266b160348db04f2fd17ccf30286784. Signed-off-by: James Almer --- libavcodec/codec_par.c | 1 + 1 file changed, 1 insertion(+) diff --git a/libavcodec/codec_par.c b/libavcodec/codec_par.c index 212cb97d77..790ea01d10 100644 --- a/libavcodec/codec_par.c +++

[FFmpeg-devel] Fix for [ffmpeg] spdif: Unusual frame timing 40 samples/frame is not implemented.

2024-04-20 Thread Adolfo Rodrigues
Hello. I'm new at this so please bare with me. I have created a sample file that produces an audio drop at the same time it produces the following message: [ffmpeg] spdif: Unusual frame timing: 10248 => 12042, 40 samples/frame is not implemented. Update your FFmpeg version to the newest one from

Re: [FFmpeg-devel] [RFC] 5 year plan & Inovation

2024-04-20 Thread Michael Niedermayer
On Fri, Apr 19, 2024 at 08:00:28PM +0200, Diederick C. Niehorster wrote: > On Fri, Apr 19, 2024, 19:35 Zhao Zhili wrote: > > > > > > -Original Message- > > > From: ffmpeg-devel On Behalf Of > > Niklas Haas > > > Sent: 2024年4月19日 22:50 > > > To: FFmpeg development discussions and patches

Re: [FFmpeg-devel] [PATCH v2 1/3] doc: Explain what "context" means

2024-04-20 Thread Andrew Sayers
On Sat, Apr 20, 2024 at 06:48:32PM +0200, Stefano Sabatini wrote: > On date Saturday 2024-04-20 13:19:41 +0100, Andrew Sayers wrote: > > Based largely on the explanation by Stefano Sabatini: > > https://ffmpeg.org/pipermail/ffmpeg-devel/2024-April/325854.html > > --- > > doc/jargon.md | 169

Re: [FFmpeg-devel] [RFC] 5 year plan & Inovation

2024-04-20 Thread Michael Niedermayer
On Thu, Apr 18, 2024 at 11:01:42PM +0100, Andrew Sayers wrote: > On 18/04/2024 20:50, Michael Niedermayer wrote: > > > [...] > > > Without getting too far off-topic, I would also be interested in knowing > > > how > > > docs are actually generated in practice. I've tried generating > > >

Re: [FFmpeg-devel] [PATCH 3/5] avcodec/av1dec: Use ProgressFrames

2024-04-20 Thread James Almer
On 4/19/2024 1:07 PM, Andreas Rheinhardt wrote: AV1 can put a frame into multiple reference slots; up until now, this involved creating a new reference to the underlying AVFrame; therefore av1_frame_ref() could fail. This commit changes this by using the ProgressFrame API to share the underlying

[FFmpeg-devel] [PATCH] lavc/videodsp: Remove emulated_edge_mc duplicate @param

2024-04-20 Thread Frank Plowman
emulated_edge_mc has no parameter called dst_stride. The description is a duplicate of the description for dst_linesize. Signed-off-by: Frank Plowman --- libavcodec/videodsp.h | 2 -- 1 file changed, 2 deletions(-) diff --git a/libavcodec/videodsp.h b/libavcodec/videodsp.h index

Re: [FFmpeg-devel] [PATCH 2/3] lavf/mkvtimestamp_v2: review implementation to match mkvextract behavior

2024-04-20 Thread Stefano Sabatini
On date Saturday 2024-04-20 18:47:58 +0200, Andreas Rheinhardt wrote: > Stefano Sabatini: > > On date Saturday 2024-04-20 15:18:39 +0200, Andreas Rheinhardt wrote: > >> Stefano Sabatini: > >>> Harmonize internal implementation with the mkvextract behavior: > >>> - print PTS in place of DTS values

Re: [FFmpeg-devel] [PATCH v2 1/3] doc: Explain what "context" means

2024-04-20 Thread Stefano Sabatini
On date Saturday 2024-04-20 13:19:41 +0100, Andrew Sayers wrote: > Based largely on the explanation by Stefano Sabatini: > https://ffmpeg.org/pipermail/ffmpeg-devel/2024-April/325854.html > --- > doc/jargon.md | 169 ++ > 1 file changed, 169

Re: [FFmpeg-devel] [PATCH 2/3] lavf/mkvtimestamp_v2: review implementation to match mkvextract behavior

2024-04-20 Thread Andreas Rheinhardt
Stefano Sabatini: > On date Saturday 2024-04-20 15:18:39 +0200, Andreas Rheinhardt wrote: >> Stefano Sabatini: >>> Harmonize internal implementation with the mkvextract behavior: >>> - print PTS in place of DTS values >>> - ignore NOPTS values >>> - sort PTS values >>> --- >>>

Re: [FFmpeg-devel] [PATCH 1/3] doc: Explain what "context" means

2024-04-20 Thread Stefano Sabatini
On date Saturday 2024-04-20 13:18:29 +0100, Andrew Sayers wrote: > On 20/04/2024 08:25, Stefano Sabatini wrote: > > On date Thursday 2024-04-18 16:06:12 +0100, Andrew Sayers wrote: > > > Based largely on the explanation by Stefano Sabatini: > > >

Re: [FFmpeg-devel] [PATCH 1/3] lavc/vp8dsp: R-V V loop_filter_simple

2024-04-20 Thread flow gg
github link: https://github.com/hleft/FFmpeg/tree/vp8vp9 flow gg 于2024年4月20日周六 23:55写道: > > ___ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org https://ffmpeg.org/mailman/listinfo/ffmpeg-devel To unsubscribe, visit link above, or email

Re: [FFmpeg-devel] [PATCH 2/3] lavf/mkvtimestamp_v2: review implementation to match mkvextract behavior

2024-04-20 Thread Stefano Sabatini
On date Saturday 2024-04-20 15:18:39 +0200, Andreas Rheinhardt wrote: > Stefano Sabatini: > > Harmonize internal implementation with the mkvextract behavior: > > - print PTS in place of DTS values > > - ignore NOPTS values > > - sort PTS values > > --- > > libavformat/mkvtimestamp_v2.c | 69

[FFmpeg-devel] [PATCH 3/3] lavc/vp8dsp: R-V V loop_filter

2024-04-20 Thread flow gg
From cff79c9500b94f4c0abdd9cd68c91cc736366c78 Mon Sep 17 00:00:00 2001 From: sunyuechi Date: Sat, 20 Apr 2024 23:26:58 +0800 Subject: [PATCH 3/3] lavc/vp8dsp: R-V V loop_filter C908: vp8_loop_filter8uv_v_c: 745.5 vp8_loop_filter8uv_v_rvv_i32: 467.2 vp8_loop_filter16y_h_c: 674.2

[FFmpeg-devel] [PATCH 2/3] lavc/vp8dsp: R-V V loop_filter_inner

2024-04-20 Thread flow gg
From c033ab8d30135dc02b09b1747c0761baefdcbb4a Mon Sep 17 00:00:00 2001 From: sunyuechi Date: Sat, 20 Apr 2024 23:13:07 +0800 Subject: [PATCH 2/3] lavc/vp8dsp: R-V V loop_filter_inner C908: vp8_loop_filter8uv_inner_v_c: 738.2 vp8_loop_filter8uv_inner_v_rvv_i32: 455.2

[FFmpeg-devel] [PATCH 1/3] lavc/vp8dsp: R-V V loop_filter_simple

2024-04-20 Thread flow gg
From 2f516e0236bd84d78ce6fd7e55c4b1a3c9d99baa Mon Sep 17 00:00:00 2001 From: sunyuechi Date: Sat, 20 Apr 2024 23:32:10 +0800 Subject: [PATCH 1/3] lavc/vp8dsp: R-V V loop_filter_simple C908: vp8_loop_filter_simple_h_c: 416.0 vp8_loop_filter_simple_h_rvv_i32: 187.5 vp8_loop_filter_simple_v_c:

Re: [FFmpeg-devel] [PATCH] Remove outdated comment about AVFrame and AVOptions

2024-04-20 Thread James Almer
On 4/20/2024 10:29 AM, Andrew Sayers wrote: --- I think this is out-of-date now avcodec_get_frame_class is gone. Yes, there's no AVClass as first element in the struct. Happy to add docs instead if I've missed something? libavutil/frame.h | 3 --- 1 file changed, 3 deletions(-) diff

Re: [FFmpeg-devel] [PATCH] avcodec/pthread_frame: Fix leak of coded side data

2024-04-20 Thread James Almer
On 4/18/2024 6:30 PM, Andreas Rheinhardt wrote: Happens in the mov-elst-ends-betn-b-and-i FATE test with frame-threading. Signed-off-by: Andreas Rheinhardt --- libavcodec/pthread_frame.c | 2 ++ 1 file changed, 2 insertions(+) diff --git a/libavcodec/pthread_frame.c

Re: [FFmpeg-devel] [PATCH] avcodec/pthread_frame: Fix leak of coded side data

2024-04-20 Thread Andreas Rheinhardt
Andreas Rheinhardt: > Happens in the mov-elst-ends-betn-b-and-i FATE test with > frame-threading. > > Signed-off-by: Andreas Rheinhardt > --- > libavcodec/pthread_frame.c | 2 ++ > 1 file changed, 2 insertions(+) > > diff --git a/libavcodec/pthread_frame.c b/libavcodec/pthread_frame.c > index

Re: [FFmpeg-devel] [PATCH v2 1/3] avcodec/x86/vvc/vvcdsp_init: add put prototypes

2024-04-20 Thread Nuo Mi
On Thu, Apr 18, 2024 at 2:30 AM Wu Jianhua wrote: > > 发件人: Nuo Mi > > 发送时间: 2024年4月17日 6:14 > > 收件人: FFmpeg development discussions and patches > > 抄送: Wu Jianhua > > 主题: Re: [FFmpeg-devel] [PATCH v2 1/3] avcodec/x86/vvc/vvcdsp_init: add > put prototypes > > > > Hi Jianhua, > > thank you for

[FFmpeg-devel] [PATCH] Remove outdated comment about AVFrame and AVOptions

2024-04-20 Thread Andrew Sayers
--- I think this is out-of-date now avcodec_get_frame_class is gone. Happy to add docs instead if I've missed something? libavutil/frame.h | 3 --- 1 file changed, 3 deletions(-) diff --git a/libavutil/frame.h b/libavutil/frame.h index 60bb966f8b..7b8d9aee08 100644 --- a/libavutil/frame.h +++

[FFmpeg-devel] [PATCH v4 3/3] avformat/avformat: Document return codes for av_format_(de)init

2024-04-20 Thread Andrew Sayers
--- libavformat/avformat.h | 4 1 file changed, 4 insertions(+) diff --git a/libavformat/avformat.h b/libavformat/avformat.h index 8afdcd9fd0..f624fb1e2e 100644 --- a/libavformat/avformat.h +++ b/libavformat/avformat.h @@ -1944,12 +1944,16 @@ const char *avformat_license(void); * This

[FFmpeg-devel] [PATCH v4 2/3] avformat/network: Return 0/AVERROR from ff_network_init()

2024-04-20 Thread Andrew Sayers
--- libavformat/avio.c| 4 ++-- libavformat/network.c | 7 +++ libavformat/rtsp.c| 12 ++-- libavformat/rtspdec.c | 4 ++-- libavformat/sapdec.c | 4 ++-- libavformat/sapenc.c | 4 ++-- 6 files changed, 17 insertions(+), 18 deletions(-) diff --git a/libavformat/avio.c

[FFmpeg-devel] [PATCH v4 1/3] avformat/network: add ff_neterrno2() for cases where we already have an errno

2024-04-20 Thread Andrew Sayers
For example, WSAStartup()'s documentation says: "A call to the WSAGetLastError function is not needed and should not be used" --- libavformat/network.c | 5 - libavformat/network.h | 2 ++ 2 files changed, 6 insertions(+), 1 deletion(-) diff --git a/libavformat/network.c

Re: [FFmpeg-devel] [PATCH 2/3] lavf/mkvtimestamp_v2: review implementation to match mkvextract behavior

2024-04-20 Thread Andreas Rheinhardt
Stefano Sabatini: > Harmonize internal implementation with the mkvextract behavior: > - print PTS in place of DTS values > - ignore NOPTS values > - sort PTS values > --- > libavformat/mkvtimestamp_v2.c | 69 +-- > 1 file changed, 65 insertions(+), 4 deletions(-) >

[FFmpeg-devel] [PATCH v2 3/3] all: Link to "context" from all contexts with documentation

2024-04-20 Thread Andrew Sayers
Some headings needed to be rewritten to accomodate the text, (hopefully) without changing the meaning. --- libavcodec/aacdec.h | 2 +- libavcodec/aacenc.h | 2 +- libavcodec/ac3enc.h | 2 +- libavcodec/amfenc.h | 2 +- libavcodec/atrac.h

[FFmpeg-devel] [PATCH v2 2/3] lavu: Clarify relationship between AVClass, AVOption and context

2024-04-20 Thread Andrew Sayers
--- libavutil/log.h | 11 --- libavutil/opt.h | 7 --- 2 files changed, 12 insertions(+), 6 deletions(-) diff --git a/libavutil/log.h b/libavutil/log.h index ab7ceabe22..b5c739dab1 100644 --- a/libavutil/log.h +++ b/libavutil/log.h @@ -59,9 +59,14 @@ typedef enum { struct

[FFmpeg-devel] [PATCH v2 1/3] doc: Explain what "context" means

2024-04-20 Thread Andrew Sayers
Based largely on the explanation by Stefano Sabatini: https://ffmpeg.org/pipermail/ffmpeg-devel/2024-April/325854.html --- doc/jargon.md | 169 ++ 1 file changed, 169 insertions(+) create mode 100644 doc/jargon.md diff --git a/doc/jargon.md

Re: [FFmpeg-devel] [PATCH 1/3] doc: Explain what "context" means

2024-04-20 Thread Andrew Sayers
On 20/04/2024 08:25, Stefano Sabatini wrote: On date Thursday 2024-04-18 16:06:12 +0100, Andrew Sayers wrote: Based largely on the explanation by Stefano Sabatini: https://ffmpeg.org/pipermail/ffmpeg-devel/2024-April/325854.html --- doc/jargon.md | 96

Re: [FFmpeg-devel] [PATCH 3/3] doc/muxers: add mkvtimestamp_v2

2024-04-20 Thread Stefano Sabatini
On date Saturday 2024-04-20 13:48:35 +0200, Stefano Sabatini wrote: > --- > doc/muxers.texi | 11 +-- > 1 file changed, 5 insertions(+), 6 deletions(-) Sorry, discard this in favor of the new patch. ___ ffmpeg-devel mailing list

[FFmpeg-devel] [PATCH] doc/muxers: add mkvtimestamp_v2

2024-04-20 Thread Stefano Sabatini
--- doc/muxers.texi | 8 1 file changed, 8 insertions(+) diff --git a/doc/muxers.texi b/doc/muxers.texi index 6340c8e54d..e1d6a0e557 100644 --- a/doc/muxers.texi +++ b/doc/muxers.texi @@ -2933,6 +2933,14 @@ MicroDVD subtitle format muxer. This muxer accepts a single @samp{microdvd}

[FFmpeg-devel] [PATCH 1/3] lavf/mkvtimestamp_v2: use name in place of description in long name

2024-04-20 Thread Stefano Sabatini
--- libavformat/mkvtimestamp_v2.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/libavformat/mkvtimestamp_v2.c b/libavformat/mkvtimestamp_v2.c index dde431ab7d..1eb2daf10a 100644 --- a/libavformat/mkvtimestamp_v2.c +++ b/libavformat/mkvtimestamp_v2.c @@ -43,7 +43,7 @@ static

[FFmpeg-devel] [PATCH 3/3] doc/muxers: add mkvtimestamp_v2

2024-04-20 Thread Stefano Sabatini
--- doc/muxers.texi | 11 +-- 1 file changed, 5 insertions(+), 6 deletions(-) diff --git a/doc/muxers.texi b/doc/muxers.texi index 6340c8e54d..4cd53a4449 100644 --- a/doc/muxers.texi +++ b/doc/muxers.texi @@ -2933,14 +2933,13 @@ MicroDVD subtitle format muxer. This muxer accepts a

[FFmpeg-devel] [PATCH 2/3] lavf/mkvtimestamp_v2: review implementation to match mkvextract behavior

2024-04-20 Thread Stefano Sabatini
Harmonize internal implementation with the mkvextract behavior: - print PTS in place of DTS values - ignore NOPTS values - sort PTS values --- libavformat/mkvtimestamp_v2.c | 69 +-- 1 file changed, 65 insertions(+), 4 deletions(-) diff --git

[FFmpeg-devel] lavf/mkvtimestamp_v2: review mkvtimestamp_v2 implementation, add documentation

2024-04-20 Thread Stefano Sabatini
[PATCH 1/3] lavf/mkvtimestamp_v2: use name in place of description in [PATCH 2/3] lavf/mkvtimestamp_v2: review implementation to match [PATCH 3/3] doc/muxers: add mkvtimestamp_v2 ___ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org

Re: [FFmpeg-devel] [PATCH v3 1/3] avformat/network: add ff_neterrno2() for cases where we already have an errno

2024-04-20 Thread Stefano Sabatini
On date Friday 2024-04-19 20:07:59 +0100, Andrew Sayers wrote: > For example, WSAStartup()'s documentation says: > > "A call to the WSAGetLastError function is not needed and should not be > used" > --- > libavformat/network.c | 7 ++- > 1 file changed, 6 insertions(+), 1 deletion(-)

Re: [FFmpeg-devel] [PATCH] tools: add target_enc_fuzzer.c

2024-04-20 Thread Stefano Sabatini
On date Saturday 2024-04-20 03:10:37 +0200, Michael Niedermayer wrote: > Sponsored-by: Sovereign Tech Fund > Signed-off-by: Michael Niedermayer > --- > Makefile | 3 + > tools/Makefile| 3 + > tools/target_enc_fuzzer.c | 213 ++

Re: [FFmpeg-devel] [PATCH 1/3] doc: Explain what "context" means

2024-04-20 Thread Stefano Sabatini
On date Thursday 2024-04-18 16:06:12 +0100, Andrew Sayers wrote: > Based largely on the explanation by Stefano Sabatini: > https://ffmpeg.org/pipermail/ffmpeg-devel/2024-April/325854.html > --- > doc/jargon.md | 96 +++ > 1 file changed, 96