[FFmpeg-devel] [PATCH V2] vcodec/vp8data: Move ff_vp8_dct_cat_prob to vp8data.c

2023-05-08 Thread Dai, Jianhui J
Avoid compiling error if included by multiple sources. Signed-off-by: Jianhui Dai --- libavcodec/Makefile | 4 ++-- libavcodec/vp8data.c | 42 ++ libavcodec/vp8data.h | 20 +--- 3 files changed, 45 insertions(+), 21 deletions(-) create

Re: [FFmpeg-devel] [PATCH v8 3/6] avformat/flvenc: support mux av1 in enhanced flv

2023-05-08 Thread Gyan Doshi
On 2023-05-09 08:11 am, Neal Gompa wrote: On Mon, May 8, 2023 at 9:55 PM Steven Liu wrote: Neal Gompa 于2023年5月9日周二 07:08写道: On Thu, Apr 13, 2023 at 5:45 AM Steven Liu wrote: Signed-off-by: Steven Liu --- libavformat/flvenc.c | 25 - 1 file changed, 20

Re: [FFmpeg-devel] [PATCH v1 1/6] avcodec/la: add LSX optimization for h264 idct.

2023-05-08 Thread yinshiyou-hf
> -原始邮件- > 发件人: "Hao Chen" > 发送时间:2023-05-04 16:49:47 (星期四) > 收件人: ffmpeg-devel@ffmpeg.org > 抄送: "Shiyou Yin" > 主题: [FFmpeg-devel] [PATCH v1 1/6] avcodec/la: add LSX optimization for h264 > idct. > > diff --git a/libavcodec/loongarch/h264dsp_lasx.h >

Re: [FFmpeg-devel] [PATCH v8 3/6] avformat/flvenc: support mux av1 in enhanced flv

2023-05-08 Thread Neal Gompa
On Mon, May 8, 2023 at 9:55 PM Steven Liu wrote: > > Neal Gompa 于2023年5月9日周二 07:08写道: > > > > On Thu, Apr 13, 2023 at 5:45 AM Steven Liu wrote: > > > > > > Signed-off-by: Steven Liu > > > --- > > > libavformat/flvenc.c | 25 - > > > 1 file changed, 20 insertions(+), 5

Re: [FFmpeg-devel] Add support for animated WebP

2023-05-08 Thread James Zern
On Fri, May 5, 2023 at 10:47 AM Thilo Borgmann wrote: > > Am 27.04.23 um 20:08 schrieb Thilo Borgmann: > > Hi, > > > > support for animated WebP had been proposed twice in the past [1][2]. > > This did stall when Lynne called for some review on the demuxer in [1]. > > > > So these are the rebased

Re: [FFmpeg-devel] [PATCH] vcodec/vp8data: Make ff_vp8_dct_cat_prob static

2023-05-08 Thread Dai, Jianhui J
Thank for the suggestion. In a WIP patch, I try to reuse ` vp8_token_update_probs` by include vp8data.h; But face the compiling error with ` ff_vp8_dct_cat_prob`. Let me move ` ff_vp8_dct_cat_prob` to vp8data.c at first. Jianhui Dai -Original Message- From: ffmpeg-devel On Behalf Of

Re: [FFmpeg-devel] [PATCH] vcodec/vp8data: Make ff_vp8_dct_cat_prob static

2023-05-08 Thread James Almer
On 5/8/2023 10:49 PM, Dai, Jianhui J wrote: Avoid compiling error if included by multiple sources. Signed-off-by: Jianhui Dai --- libavcodec/vp8data.h | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/libavcodec/vp8data.h b/libavcodec/vp8data.h index 1fcce134eb..ac4af5f733

Re: [FFmpeg-devel] [PATCH v8 3/6] avformat/flvenc: support mux av1 in enhanced flv

2023-05-08 Thread Steven Liu
Neal Gompa 于2023年5月9日周二 07:08写道: > > On Thu, Apr 13, 2023 at 5:45 AM Steven Liu wrote: > > > > Signed-off-by: Steven Liu > > --- > > libavformat/flvenc.c | 25 - > > 1 file changed, 20 insertions(+), 5 deletions(-) > > > > diff --git a/libavformat/flvenc.c

[FFmpeg-devel] [PATCH] vcodec/vp8data: Make ff_vp8_dct_cat_prob static

2023-05-08 Thread Dai, Jianhui J
Avoid compiling error if included by multiple sources. Signed-off-by: Jianhui Dai --- libavcodec/vp8data.h | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/libavcodec/vp8data.h b/libavcodec/vp8data.h index 1fcce134eb..ac4af5f733 100644 --- a/libavcodec/vp8data.h +++

Re: [FFmpeg-devel] [PATCH 11/13] lavc/libvpxenc: send frame durations to the encoder

2023-05-08 Thread James Zern
On Sun, May 7, 2023 at 6:34 AM Anton Khirnov wrote: > > Adapt similar code from libaomenc - stop using ticks_per_frame except as > a last resort. > --- > libavcodec/libvpxenc.c | 15 +-- > 1 file changed, 13 insertions(+), 2 deletions(-) > > diff --git a/libavcodec/libvpxenc.c

Re: [FFmpeg-devel] [PATCH] avformat/hls: fail on probing non hls/m3u8 file extensions

2023-05-08 Thread Michael Niedermayer
On Sun, May 07, 2023 at 10:42:56PM +0200, Anton Khirnov wrote: > Quoting Michael Niedermayer (2023-05-03 14:30:38) > > Its unexpected that a .avi or other "standard" file turns into a playlist. > > The goal of this patch is to avoid this unexpected behavior and possible > > privacy or security

Re: [FFmpeg-devel] [PATCH v8 3/6] avformat/flvenc: support mux av1 in enhanced flv

2023-05-08 Thread Neal Gompa
On Thu, Apr 13, 2023 at 5:45 AM Steven Liu wrote: > > Signed-off-by: Steven Liu > --- > libavformat/flvenc.c | 25 - > 1 file changed, 20 insertions(+), 5 deletions(-) > > diff --git a/libavformat/flvenc.c b/libavformat/flvenc.c > index 57a26245ff..7b43ecaefa 100644 >

Re: [FFmpeg-devel] [PATCH] avcodec/mxfdec: Recognize AAC per SMPTE ST 381-4

2023-05-08 Thread Marton Balint
On Thu, 27 Apr 2023, Ammon Riley wrote: This patch simply recognizes the AAC audio tracks during decode -- it does not add functionality to encode AAC in MXF. A sample file (st381-4-sample.mxf) has been uploaded to https://streams.videolan.org/upload/, and is also available at

Re: [FFmpeg-devel] [PATCH 1/3] avformat/dashdec: fail on probing non mpd file extension

2023-05-08 Thread Michael Niedermayer
On Mon, May 08, 2023 at 04:05:40PM +0200, Tobias Rapp wrote: > On 08/05/2023 14:00, James Almer wrote: > > > On 5/6/2023 10:25 AM, Michael Niedermayer wrote: > > > Its unexpected that a .avi or other "standard" file turns into a > > > playlist. > > > The goal of this patch is to avoid this

Re: [FFmpeg-devel] [PATCH 2/3] avformat/imfdec: fail on probing non xml file extension

2023-05-08 Thread Pierre-Anthony Lemieux
On Mon, May 8, 2023 at 3:01 PM Michael Niedermayer wrote: > > On Mon, May 08, 2023 at 11:40:49AM -0700, Pierre-Anthony Lemieux wrote: > > On Mon, May 8, 2023 at 11:23 AM Michael Niedermayer > > wrote: > > > > > > On Sun, May 07, 2023 at 10:09:58PM -0700, Pierre-Anthony Lemieux wrote: > > > > On

Re: [FFmpeg-devel] [PATCH 2/3] avformat/imfdec: fail on probing non xml file extension

2023-05-08 Thread Michael Niedermayer
On Mon, May 08, 2023 at 11:40:49AM -0700, Pierre-Anthony Lemieux wrote: > On Mon, May 8, 2023 at 11:23 AM Michael Niedermayer > wrote: > > > > On Sun, May 07, 2023 at 10:09:58PM -0700, Pierre-Anthony Lemieux wrote: > > > On Sun, May 7, 2023 at 12:18 PM Michael Niedermayer > > > wrote: > > > > >

Re: [FFmpeg-devel] [PATCH] avfilter: add zoneplate video source filter

2023-05-08 Thread Stefano Sabatini
On date Monday 2023-05-08 01:30:34 +0200, Paul B Mahol wrote: > New version, faster and with slice threading and docs. > From 8b9ab6e3401d69f115b5d331fec73fd8c01ea1bd Mon Sep 17 00:00:00 2001 > From: Paul B Mahol > Date: Sat, 6 May 2023 22:52:47 +0200 > Subject: [PATCH] avfilter: add zoneplate

Re: [FFmpeg-devel] [PATCH] avfilter/vf_mpdecimate: Add option to keep the first N similar frames before dropping

2023-05-08 Thread Thilo Borgmann
Am 08.05.23 um 20:55 schrieb Paul B Mahol: On Mon, May 8, 2023 at 8:06 PM Thilo Borgmann wrote: Am 08.05.23 um 15:58 schrieb Paul B Mahol: On Mon, May 8, 2023 at 3:43 PM Thilo Borgmann wrote: Am 08.05.23 um 15:32 schrieb Thilo Borgmann: This allows for decimating large similar portions

Re: [FFmpeg-devel] [PATCH] tools/normalize: port to python3, extend syntax

2023-05-08 Thread Stefano Sabatini
On date Wednesday 2023-04-05 01:12:10 +0200, Stefano Sabatini wrote: > Also add documentation and logging. > --- > tools/normalize.py | 106 +++-- > 1 file changed, 73 insertions(+), 33 deletions(-) Ping, will push in a few days.

Re: [FFmpeg-devel] Embedded documentation?

2023-05-08 Thread Nicolas George
Stefano Sabatini (12023-05-08): > Can you share more details about the plan? In particular, is the doc > going to be embedded in the code itself (e.g. in the C > implementation)? Or should we have some dedicated headers containing > the docs? > > We should also avoid to duplicate the same

Re: [FFmpeg-devel] [PATCH] avfilter/vf_mpdecimate: Add option to keep the first N similar frames before dropping

2023-05-08 Thread Paul B Mahol
On Mon, May 8, 2023 at 8:06 PM Thilo Borgmann wrote: > Am 08.05.23 um 15:58 schrieb Paul B Mahol: > > On Mon, May 8, 2023 at 3:43 PM Thilo Borgmann > > wrote: > > > >> Am 08.05.23 um 15:32 schrieb Thilo Borgmann: > >>> This allows for decimating large similar portions of a video while > >>

Re: [FFmpeg-devel] [PATCH 2/3] avformat/imfdec: fail on probing non xml file extension

2023-05-08 Thread Pierre-Anthony Lemieux
On Mon, May 8, 2023 at 11:23 AM Michael Niedermayer wrote: > > On Sun, May 07, 2023 at 10:09:58PM -0700, Pierre-Anthony Lemieux wrote: > > On Sun, May 7, 2023 at 12:18 PM Michael Niedermayer > > wrote: > > > > > > On Sat, May 06, 2023 at 11:01:20AM -0700, Pierre-Anthony Lemieux wrote: > > > > On

Re: [FFmpeg-devel] [PATCH 2/3] avformat/imfdec: fail on probing non xml file extension

2023-05-08 Thread Michael Niedermayer
On Sun, May 07, 2023 at 10:09:58PM -0700, Pierre-Anthony Lemieux wrote: > On Sun, May 7, 2023 at 12:18 PM Michael Niedermayer > wrote: > > > > On Sat, May 06, 2023 at 11:01:20AM -0700, Pierre-Anthony Lemieux wrote: > > > On Sat, May 6, 2023 at 6:25 AM Michael Niedermayer > > > wrote: > > > > > >

Re: [FFmpeg-devel] [PATCH] avfilter/vf_mpdecimate: Add option to keep the first N similar frames before dropping

2023-05-08 Thread Thilo Borgmann
Am 08.05.23 um 15:58 schrieb Paul B Mahol: On Mon, May 8, 2023 at 3:43 PM Thilo Borgmann wrote: Am 08.05.23 um 15:32 schrieb Thilo Borgmann: This allows for decimating large similar portions of a video while preserving small ones. --- libavfilter/vf_mpdecimate.c | 19 +--

Re: [FFmpeg-devel] [PATCH v3 1/2] avformat/avcodec: Add DTS-UHD demuxer and parser, movenc support.

2023-05-08 Thread Roy Funderburk
Hi, If there is anything I could do to improve this submission, please tell me. Thanks, -Roy On 4/16/23 9:12 PM, Roy Funderburk wrote: > Parsing of DTS-UHD input files per ETSI TS 102 114 is added > as parser for codec id AV_CODEC_ID_DTSUHD. > > Signed-off-by: Roy Funderburk

Re: [FFmpeg-devel] [PATCH 1/3] avformat/dashdec: fail on probing non mpd file extension

2023-05-08 Thread Pierre-Anthony Lemieux
On Mon, May 8, 2023 at 10:11 AM Michael Niedermayer wrote: > > On Mon, May 08, 2023 at 04:05:40PM +0200, Tobias Rapp wrote: > > On 08/05/2023 14:00, James Almer wrote: > > > > > On 5/6/2023 10:25 AM, Michael Niedermayer wrote: > > > > Its unexpected that a .avi or other "standard" file turns into

Re: [FFmpeg-devel] [PATCH 1/3] avformat/dashdec: fail on probing non mpd file extension

2023-05-08 Thread Michael Niedermayer
On Mon, May 08, 2023 at 04:05:40PM +0200, Tobias Rapp wrote: > On 08/05/2023 14:00, James Almer wrote: > > > On 5/6/2023 10:25 AM, Michael Niedermayer wrote: > > > Its unexpected that a .avi or other "standard" file turns into a > > > playlist. > > > The goal of this patch is to avoid this

Re: [FFmpeg-devel] Embedded documentation?

2023-05-08 Thread Nicolas George
Zhanbang He (12023-05-08): > Can it supports chatGPT? Of course not. Only ChatGnuTP, for obvious reasons of freedom. Regards, -- Nicolas George signature.asc Description: PGP signature ___ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org

Re: [FFmpeg-devel] Embedded documentation?

2023-05-08 Thread Zhanbang He
Can it supports chatGPT? On Monday, May 8, 2023, Nicolas George wrote: > Stefano Sabatini (12023-05-08): > > I cannot parse this, where is the threshold value defined? > > The threshold is the enum constant that was being described. > > > Maybe an example would clarify this, since there is

Re: [FFmpeg-devel] Embedded documentation?

2023-05-08 Thread Nicolas George
Stefano Sabatini (12023-05-08): > I cannot parse this, where is the threshold value defined? The threshold is the enum constant that was being described. > Maybe an example would clarify this, since there is ambiguity about > what default and explanations are. Let us think how this is meant to

Re: [FFmpeg-devel] Embedded documentation?

2023-05-08 Thread Nicolas George
Stefano Sabatini (12023-05-08): > I think the problem with HTML is that then you need to parse it if you > want to display it, so I'd tend to rather go with markdown: > 1. it provides readable raw output > 2. there are plenty of libraries which can render it to various > formats (including HTML)

Re: [FFmpeg-devel] [PATCH 1/3] avformat/dashdec: fail on probing non mpd file extension

2023-05-08 Thread Pierre-Anthony Lemieux
On Mon, May 8, 2023 at 7:05 AM Tobias Rapp wrote: > > On 08/05/2023 14:00, James Almer wrote: > > > On 5/6/2023 10:25 AM, Michael Niedermayer wrote: > >> Its unexpected that a .avi or other "standard" file turns into a > >> playlist. > >> The goal of this patch is to avoid this unexpected

Re: [FFmpeg-devel] [PATCH 04/13] lavf: use AV_CODEC_PROP_FIELDS where appropriate

2023-05-08 Thread Michael Niedermayer
On Sun, May 07, 2023 at 03:32:46PM +0200, Anton Khirnov wrote: > H.264 and mpeg12 parsers need to be adjusted at the same time to stop > using the value of AVCodecContext.ticks_per_frame, because it is not set > correctly unless the codec has been opened. Previously this would result > in both the

Re: [FFmpeg-devel] [PATCH 04/13] lavf: use AV_CODEC_PROP_FIELDS where appropriate

2023-05-08 Thread Michael Niedermayer
On Sun, May 07, 2023 at 03:32:46PM +0200, Anton Khirnov wrote: > H.264 and mpeg12 parsers need to be adjusted at the same time to stop > using the value of AVCodecContext.ticks_per_frame, because it is not set > correctly unless the codec has been opened. Previously this would result > in both the

Re: [FFmpeg-devel] [PATCH] fftools/ffprobe: print exported stream AVOptions

2023-05-08 Thread James Almer
On 5/7/2023 7:51 PM, Stefano Sabatini wrote: On date Sunday 2023-05-07 19:30:03 -0300, James Almer wrote: On 5/7/2023 6:20 PM, Stefano Sabatini wrote: On date Friday 2023-05-05 20:18:56 -0300, James Almer wrote: Similar to the decoder AVOptions above. Please briefly specify the use case in

Re: [FFmpeg-devel] [PATCH 1/3] avformat/dashdec: fail on probing non mpd file extension

2023-05-08 Thread Tobias Rapp
On 08/05/2023 14:00, James Almer wrote: On 5/6/2023 10:25 AM, Michael Niedermayer wrote: Its unexpected that a .avi or other "standard" file turns into a playlist. The goal of this patch is to avoid this unexpected behavior and possible privacy or security differences. This is similar to the

Re: [FFmpeg-devel] [PATCH] avfilter/vf_mpdecimate: Add option to keep the first N similar frames before dropping

2023-05-08 Thread Paul B Mahol
On Mon, May 8, 2023 at 3:43 PM Thilo Borgmann wrote: > Am 08.05.23 um 15:32 schrieb Thilo Borgmann: > > This allows for decimating large similar portions of a video while > preserving small ones. > > --- > > libavfilter/vf_mpdecimate.c | 19 +-- > > 1 file changed, 17

Re: [FFmpeg-devel] [PATCH] avfilter/vf_mpdecimate: Add option to keep the first N similar frames before dropping

2023-05-08 Thread Thilo Borgmann
Am 08.05.23 um 15:32 schrieb Thilo Borgmann: This allows for decimating large similar portions of a video while preserving small ones. --- libavfilter/vf_mpdecimate.c | 19 +-- 1 file changed, 17 insertions(+), 2 deletions(-) docs and micro bump in v2. -Thilo

[FFmpeg-devel] [PATCH v2] avfilter/vf_mpdecimate: Add option to keep the first N similar frames before dropping

2023-05-08 Thread Thilo Borgmann
This allows for decimating large similar portions of a video while preserving small ones. --- doc/filters.texi| 7 +++ libavfilter/version.h | 2 +- libavfilter/vf_mpdecimate.c | 19 +-- 3 files changed, 25 insertions(+), 3 deletions(-) diff --git

[FFmpeg-devel] [PATCH] avfilter/vf_mpdecimate: Add option to keep the first N similar frames before dropping

2023-05-08 Thread Thilo Borgmann
This allows for decimating large similar portions of a video while preserving small ones. --- libavfilter/vf_mpdecimate.c | 19 +-- 1 file changed, 17 insertions(+), 2 deletions(-) diff --git a/libavfilter/vf_mpdecimate.c b/libavfilter/vf_mpdecimate.c index

Re: [FFmpeg-devel] [PATCH 1/3] avformat/dashdec: fail on probing non mpd file extension

2023-05-08 Thread James Almer
On 5/6/2023 10:25 AM, Michael Niedermayer wrote: Its unexpected that a .avi or other "standard" file turns into a playlist. The goal of this patch is to avoid this unexpected behavior and possible privacy or security differences. This is similar to the same change to hls Signed-off-by: Michael

[FFmpeg-devel] [PATCH 14/22] fftools/ffmpeg: simplify tracking -readrate start time

2023-05-08 Thread Anton Khirnov
There is no point in having a per-stream wallclock start time, since they are all computed at the same instant. Keep a per-file start time instead, initialized when the demuxer thread starts. --- fftools/ffmpeg.c | 8 fftools/ffmpeg.h | 2 -- fftools/ffmpeg_demux.c | 6 +-

[FFmpeg-devel] [PATCH 11/22] fftools/ffmpeg_demux: move InputStream.[next_]dts to private data

2023-05-08 Thread Anton Khirnov
They are no longer used outside of ffmpeg_demux. --- fftools/ffmpeg.h | 4 --- fftools/ffmpeg_demux.c | 59 +- 2 files changed, 35 insertions(+), 28 deletions(-) diff --git a/fftools/ffmpeg.h b/fftools/ffmpeg.h index e4a697af54..b8c0e7db84 100644

[FFmpeg-devel] [PATCH 13/22] fftools/ffmpeg_demux: move InputStream.streamcopy_needed to private data

2023-05-08 Thread Anton Khirnov
It is no longer used outside of ffmpeg_demux. --- fftools/ffmpeg.h | 1 - fftools/ffmpeg_demux.c | 8 ++-- 2 files changed, 6 insertions(+), 3 deletions(-) diff --git a/fftools/ffmpeg.h b/fftools/ffmpeg.h index 559381531e..7eb6301c74 100644 --- a/fftools/ffmpeg.h +++ b/fftools/ffmpeg.h

[FFmpeg-devel] [PATCH 07/22] fftools/ffmpeg_demux: reindent after previous commit

2023-05-08 Thread Anton Khirnov
--- fftools/ffmpeg_demux.c | 28 ++-- 1 file changed, 14 insertions(+), 14 deletions(-) diff --git a/fftools/ffmpeg_demux.c b/fftools/ffmpeg_demux.c index 9082de6a80..d49bb8317c 100644 --- a/fftools/ffmpeg_demux.c +++ b/fftools/ffmpeg_demux.c @@ -481,20 +481,20 @@ fail:

[FFmpeg-devel] [PATCH 05/22] fftools/ffmpeg: attach InputStream.dts to demuxed packets when needed

2023-05-08 Thread Anton Khirnov
This way computing it and using it for streamcopy does not need to happen in sync. Will be useful in following commits, where updating InputStream.dts will be moved to the demuxing thread. --- fftools/ffmpeg.c | 22 -- fftools/ffmpeg.h | 7 +++

[FFmpeg-devel] [PATCH 12/22] fftools/ffmpeg_demux: move InputStream.wrap_correction_done to private data

2023-05-08 Thread Anton Khirnov
It is no longer used outside of ffmpeg_demux. --- fftools/ffmpeg.h | 2 -- fftools/ffmpeg_demux.c | 9 + 2 files changed, 5 insertions(+), 6 deletions(-) diff --git a/fftools/ffmpeg.h b/fftools/ffmpeg.h index b8c0e7db84..559381531e 100644 --- a/fftools/ffmpeg.h +++

[FFmpeg-devel] [PATCH 09/22] fftools/ffmpeg_demux: move InputStream.{nb_packets, data_size} to private data

2023-05-08 Thread Anton Khirnov
They are no longer used outside of ffmpeg_demux. --- fftools/ffmpeg.h | 4 fftools/ffmpeg_demux.c | 19 +-- 2 files changed, 13 insertions(+), 10 deletions(-) diff --git a/fftools/ffmpeg.h b/fftools/ffmpeg.h index 1900f3deb6..8e96f27d5d 100644 --- a/fftools/ffmpeg.h

[FFmpeg-devel] [PATCH 02/22] fftools/ffmpeg: reindent after previous commit

2023-05-08 Thread Anton Khirnov
--- fftools/ffmpeg.c | 58 1 file changed, 29 insertions(+), 29 deletions(-) diff --git a/fftools/ffmpeg.c b/fftools/ffmpeg.c index 2bd3ce3f13..817b643c48 100644 --- a/fftools/ffmpeg.c +++ b/fftools/ffmpeg.c @@ -1870,36 +1870,36 @@ static void

[FFmpeg-devel] [PATCH 20/22] fftools/ffmpeg_demux: move InputFile.ts_offset_discont, last_ts to private data

2023-05-08 Thread Anton Khirnov
They are no longer used outside of ffmpeg_demux. --- fftools/ffmpeg.h | 5 - fftools/ffmpeg_demux.c | 31 +++ 2 files changed, 19 insertions(+), 17 deletions(-) diff --git a/fftools/ffmpeg.h b/fftools/ffmpeg.h index 5863ca1faf..f88792d7eb 100644 ---

[FFmpeg-devel] [PATCH 22/22] fftools/ffmpeg: rename transcode_init()

2023-05-08 Thread Anton Khirnov
It does no initialization anymore, except for setting transcode_init_done - the bulk of the function is printing the input/output maps. It also cannot fail anymore, so remove the useless return value. --- fftools/ffmpeg.c | 18 -- 1 file changed, 4 insertions(+), 14 deletions(-)

[FFmpeg-devel] [PATCH 21/22] fftools/ffmpeg_demux: stop logging to demuxer context

2023-05-08 Thread Anton Khirnov
Only the demuxer itself should do that. --- fftools/ffmpeg_demux.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/fftools/ffmpeg_demux.c b/fftools/ffmpeg_demux.c index 461f819e9e..0d265cbfea 100644 --- a/fftools/ffmpeg_demux.c +++ b/fftools/ffmpeg_demux.c @@ -641,14

[FFmpeg-devel] [PATCH 18/22] fftools/ffmpeg: log corrupt-frame errors to the appropriate context

2023-05-08 Thread Anton Khirnov
--- fftools/ffmpeg.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/fftools/ffmpeg.c b/fftools/ffmpeg.c index 981714d027..092c5e179a 100644 --- a/fftools/ffmpeg.c +++ b/fftools/ffmpeg.c @@ -779,8 +779,8 @@ static void check_decode_result(InputStream *ist, int

[FFmpeg-devel] [PATCH 19/22] fftools/ffmpeg: stop accessing input format from decoding code

2023-05-08 Thread Anton Khirnov
Export the corresponding flag in InputFile instead. This will allow making the demuxer AVFormatContext private in future commits, similarly to what was previously done for muxers. --- fftools/ffmpeg.c | 5 ++--- fftools/ffmpeg.h | 3 +++ fftools/ffmpeg_demux.c | 2 ++ 3 files changed,

[FFmpeg-devel] [PATCH 16/22] fftools/ffmpeg_demux: reindent after previous commit

2023-05-08 Thread Anton Khirnov
--- fftools/ffmpeg_demux.c | 20 ++-- 1 file changed, 10 insertions(+), 10 deletions(-) diff --git a/fftools/ffmpeg_demux.c b/fftools/ffmpeg_demux.c index 1b268719aa..4b1e7e5254 100644 --- a/fftools/ffmpeg_demux.c +++ b/fftools/ffmpeg_demux.c @@ -527,17 +527,17 @@ static void

[FFmpeg-devel] [PATCH 17/22] fftools/ffmpeg: replace print_error() by more meaningful messages

2023-05-08 Thread Anton Khirnov
--- fftools/ffmpeg.c | 5 ++--- fftools/ffmpeg_demux.c | 3 ++- fftools/ffmpeg_mux.c | 4 +++- 3 files changed, 7 insertions(+), 5 deletions(-) diff --git a/fftools/ffmpeg.c b/fftools/ffmpeg.c index 725ffca631..981714d027 100644 --- a/fftools/ffmpeg.c +++ b/fftools/ffmpeg.c @@ -1756,12

[FFmpeg-devel] [PATCH 15/22] fftools/ffmpeg: move discarding unused programs to ffmpeg_demux

2023-05-08 Thread Anton Khirnov
This is a more appropriate place for this code. --- fftools/ffmpeg.c | 16 fftools/ffmpeg_demux.c | 17 + 2 files changed, 17 insertions(+), 16 deletions(-) diff --git a/fftools/ffmpeg.c b/fftools/ffmpeg.c index 40e97ad486..725ffca631 100644 ---

[FFmpeg-devel] [PATCH 10/22] fftools/ffmpeg_demux: move InputStream.[saw_]first_d?ts to private data

2023-05-08 Thread Anton Khirnov
They are no longer used outside of ffmpeg_demux. --- fftools/ffmpeg.h | 2 -- fftools/ffmpeg_demux.c | 22 ++ 2 files changed, 14 insertions(+), 10 deletions(-) diff --git a/fftools/ffmpeg.h b/fftools/ffmpeg.h index 8e96f27d5d..e4a697af54 100644 --- a/fftools/ffmpeg.h

[FFmpeg-devel] [PATCH 06/22] fftools/ffmpeg: move post-demux packet processing to ffmpeg_demux

2023-05-08 Thread Anton Khirnov
That is a more appropriate place for this code and will allow hiding more of InputStream. The value of repeat_pict extracted from libavformat internal parser no longer needs to be trasmitted outside of the demuxing thread. Move readrate handling to the demuxer thread. This has to be done in the

[FFmpeg-devel] [PATCH 04/22] fftools/ffmpeg_demux: move preparing DemuxMsg to separate function

2023-05-08 Thread Anton Khirnov
Will be useful in following commits, which will move more code into this function. --- fftools/ffmpeg_demux.c | 31 +++ 1 file changed, 23 insertions(+), 8 deletions(-) diff --git a/fftools/ffmpeg_demux.c b/fftools/ffmpeg_demux.c index 1872e87d57..b16a20a87b 100644

[FFmpeg-devel] [PATCH 08/22] fftools/ffmpeg: drop unused decode_video() parameter

2023-05-08 Thread Anton Khirnov
--- fftools/ffmpeg.c | 6 ++ 1 file changed, 2 insertions(+), 4 deletions(-) diff --git a/fftools/ffmpeg.c b/fftools/ffmpeg.c index ee92587798..9cf94f2a22 100644 --- a/fftools/ffmpeg.c +++ b/fftools/ffmpeg.c @@ -1026,7 +1026,7 @@ static int64_t video_duration_estimate(const InputStream

[FFmpeg-devel] [PATCH 01/22] fftools/ffmpeg: consolidate InputStream.[next_]dts updates

2023-05-08 Thread Anton Khirnov
Move them to a separate function called right after timestamp discontinuity processing. This is now possible, since these values have no interaction with decoding anymore. --- This set goes on top of the repeat_pict/ticks_per_frame set, and can also be obtained from branch 'ffmpeg_ist_dts' at

[FFmpeg-devel] [PATCH 03/22] fftools/ffmpeg: stop using decoder properties in ist_dts_update()

2023-05-08 Thread Anton Khirnov
This code runs post-demuxing and is not synchronized with the decoder output (which may be delayed with respect to its input by arbitrary and unknowable amounts), so accessing any decoder properties is incorrect. --- fftools/ffmpeg.c | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-)