Re: [FFmpeg-devel] [PATCH 7/30] avformat/matroskaenc: Avoid allocations for SeekHead

2020-03-05 Thread Andreas Rheinhardt
Andreas Rheinhardt: > On Fri, Feb 28, 2020 at 6:10 AM Andreas Rheinhardt < > andreas.rheinha...@gmail.com> wrote: > >> On Tue, Feb 25, 2020 at 2:24 PM Andreas Rheinhardt < >> andreas.rheinha...@gmail.com> wrote: >> >>> Andreas Rheinhardt: Andreas Rheinhardt: > Andreas Rheinhardt: >>

Re: [FFmpeg-devel] [PATCH V2 3/3] avfilter/vf_dnn_processing.c: add frame size change support for planar yuv format

2020-03-05 Thread Guo, Yejun
> -Original Message- > From: ffmpeg-devel [mailto:ffmpeg-devel-boun...@ffmpeg.org] On Behalf Of > myp...@gmail.com > Sent: Friday, March 06, 2020 10:49 AM > To: FFmpeg development discussions and patches > Subject: Re: [FFmpeg-devel] [PATCH V2 3/3] avfilter/vf_dnn_processing.c: add >

[FFmpeg-devel] [PATCH v2] avcodec/exr: add cineon lin2log trc

2020-03-05 Thread mindmark
From: Mark Reid Hi, The following patch adds a cineon lin2log color transfer characteristic to exr. The purpose of this patch is to allow preserving of the dynamic range of an exr file when converting to DPX or when using video filter such as 3d luts. I wasn't sure if adding it to the

[FFmpeg-devel] [PATCH v2] avformat/hlsenc: Fix initial setting for start_pts

2020-03-05 Thread Hongcheng Zhong
This patch fixes Bug #8469 If x264 baseline profile is used with other profiles, start_pts will be initialized to audio stream's first pts, while the duration is calculated based on video stream's pts. In this patch the start_pts is initialized with the correct stream's first pts. Signed-off-by:

Re: [FFmpeg-devel] [PATCH V2 3/3] avfilter/vf_dnn_processing.c: add frame size change support for planar yuv format

2020-03-05 Thread myp...@gmail.com
On Tue, Feb 25, 2020 at 5:24 PM Guo, Yejun wrote: > > The Y channel is handled by dnn, and also resized by dnn. The UV channels > are resized with swscale. For me, this is a little weird to resize Y with dnn backend but resize UV channel with FFmpeg swscale, is it used the same scale algorithm ?

Re: [FFmpeg-devel] [PATCH] avformat/hlsenc: Fix initial setting for start_pts

2020-03-05 Thread Hongcheng Zhong
- On Mar 6, 2020, at 11:08 AM, Andreas Rheinhardt andreas.rheinha...@gmail.com wrote: > Hongcheng Zhong: >> - On Mar 6, 2020, at 12:55 AM, Andreas Rheinhardt >> andreas.rheinha...@gmail.com wrote: >> >>> Hongcheng Zhong: - On Mar 5, 2020, at 11:38 PM, lq l...@chinaffmpeg.org

Re: [FFmpeg-devel] [PATCH v2 7/7] avformat/audiointerleave: use a fixed frame duration for non-audio packets

2020-03-05 Thread Andreas Rheinhardt
Marton Balint: > The packet durations might not be set properly which can cause the MXF muxer > to write more than one packet of a stream to an edit unit messing up the > constant byte per element index... > > Also use nb_samples directly to calculate dts of audio packets because adding > packet

Re: [FFmpeg-devel] [PATCH] avformat/hlsenc: Fix initial setting for start_pts

2020-03-05 Thread Andreas Rheinhardt
Hongcheng Zhong: > - On Mar 6, 2020, at 12:55 AM, Andreas Rheinhardt > andreas.rheinha...@gmail.com wrote: > >> Hongcheng Zhong: >>> - On Mar 5, 2020, at 11:38 PM, lq l...@chinaffmpeg.org wrote: >>> > 2020年3月5日 下午9:41,Hongcheng Zhong 写道: > > This patch fixes Bug #8469 >

Re: [FFmpeg-devel] [PATCH 2/2] libavcodec/libaomenc.c: Add super-resolution options to libaom wrapper

2020-03-05 Thread Wang Cao
On Wed, Mar 4, 2020 at 1:38 AM Moritz Barsnick wrote: > On Wed, Mar 04, 2020 at 05:59:03 +0800, Wang Cao wrote: > > Signed-off-by: Wang Cao > > --- > > doc/encoders.texi | 39 +++ > > libavcodec/libaomenc.c | 38 ++ >

[FFmpeg-devel] [PATCH v2 1/2] avcodec/libaomenc.c: Add a libaom command-line option 'tune'

2020-03-05 Thread Wang Cao
Signed-off-by: Wang Cao --- Made changes according to the review. - Bump the MICRO version of libavcodec. - Use enum for 'tune' option consts for better consistency doc/encoders.texi | 11 +++ libavcodec/libaomenc.c | 7 +++ libavcodec/version.h | 2 +- 3 files changed, 19

[FFmpeg-devel] [PATCH v2 2/2] avcodec/libaomenc.c: Add super-resolution options to libaom wrapper

2020-03-05 Thread Wang Cao
Signed-off-by: Wang Cao --- The changes are made according to the code review - Bump the MICRO version - Use enum for Super resolution mode consts. The original enum in libaom is not public so a enum is defined and matched the original enum doc/encoders.texi | 39

[FFmpeg-devel] [PATCH] avcodec/vp9: use a buffer pool to allocate VP9Frame extradata

2020-03-05 Thread James Almer
Signed-off-by: James Almer --- libavcodec/vp9.c| 13 - libavcodec/vp9dec.h | 4 2 files changed, 16 insertions(+), 1 deletion(-) diff --git a/libavcodec/vp9.c b/libavcodec/vp9.c index 7aaae9b792..7ee375d4d0 100644 --- a/libavcodec/vp9.c +++ b/libavcodec/vp9.c @@ -112,10

Re: [FFmpeg-devel] [PATCH V2 3/3] avfilter/vf_dnn_processing.c: add frame size change support for planar yuv format

2020-03-05 Thread Guo, Yejun
> -Original Message- > From: Guo, Yejun > Sent: Tuesday, February 25, 2020 5:15 PM > To: ffmpeg-devel@ffmpeg.org > Cc: Guo, Yejun > Subject: [PATCH V2 3/3] avfilter/vf_dnn_processing.c: add frame size change > support for planar yuv format > > The Y channel is handled by dnn, and also

Re: [FFmpeg-devel] [PATCH v2 01/14] h264_mp4toannexb: Remove unnecessary check

2020-03-05 Thread Michael Niedermayer
On Sun, Dec 15, 2019 at 11:56:00PM +0100, Michael Niedermayer wrote: > On Sat, Dec 14, 2019 at 11:19:13PM +0100, Andreas Rheinhardt wrote: > > There can be at most 31 SPS and 255 PPS in the mp4/Matroska extradata. > > Given that each has a size of at most 2^16-1, the length of the output > >

[FFmpeg-devel] [PATCH] avcodec/exr: add cineon lin2log trc

2020-03-05 Thread mindmark
From: Mark Reid Hi, The following patch adds a cineon lin2log color transfer characteristic to exr. The purpose of this patch is to allow preserving of the dynamic range of an exr file when converting to DPX or when using video filter such as 3d luts. I wasn't sure if adding it to the

Re: [FFmpeg-devel] [PATCH 1/2] avcodec/cbs_av1_syntax_template: Set seen_frame_header only after successfull uncompressed_header()

2020-03-05 Thread Michael Niedermayer
On Sat, Dec 14, 2019 at 03:21:57PM -0300, James Almer wrote: > On 12/14/2019 2:43 PM, Michael Niedermayer wrote: > > Fixes: assertion failure > > Fixes: > > 19301/clusterfuzz-testcase-minimized-ffmpeg_BSF_AV1_FRAME_MERGE_fuzzer-5743212006473728 > > > > Found-by: continuous fuzzing process > >

[FFmpeg-devel] [PATCH] fftools/ffplay: fix possible memory leak in decoder

2020-03-05 Thread Marton Balint
Fixes ticket #8549. Signed-off-by: Marton Balint --- fftools/ffplay.c | 5 - 1 file changed, 4 insertions(+), 1 deletion(-) diff --git a/fftools/ffplay.c b/fftools/ffplay.c index fee0619f7c..f6511e4afd 100644 --- a/fftools/ffplay.c +++ b/fftools/ffplay.c @@ -644,7 +644,10 @@ static int

[FFmpeg-devel] [PATCH] lavd/avfoundation: Add basic transport control observation, for capable devices.

2020-03-05 Thread Thilo Borgmann
Hi, fixes infinite loop waiting for input using e.g. tape devices that suport transport control. -Thilo From 656f1fe2a91bda6099d0ed3848dce9aec55c8932 Mon Sep 17 00:00:00 2001 From: Thilo Borgmann Date: Thu, 5 Mar 2020 12:32:49 +0100 Subject: [PATCH] lavd/avfoundation: Add basic transport

[FFmpeg-devel] [PATCH v2 6/7] avformat/mxfenc: allow all frame rates if -strict mode is set to unofficial or lower

2020-03-05 Thread Marton Balint
There was no consensus wheter or not to allow unofficial frame rates due to possible interoperability issues, a compromise is to only allow it if -strict mode is set to unofficial. Signed-off-by: Marton Balint --- libavformat/mxfenc.c | 8 ++-- 1 file changed, 6 insertions(+), 2

[FFmpeg-devel] [PATCH v2 7/7] avformat/audiointerleave: use a fixed frame duration for non-audio packets

2020-03-05 Thread Marton Balint
The packet durations might not be set properly which can cause the MXF muxer to write more than one packet of a stream to an edit unit messing up the constant byte per element index... Also use nb_samples directly to calculate dts of audio packets because adding packet durations might not be

[FFmpeg-devel] [PATCH v2 3/7] avformat/mxfenc: move content package rates and timebase combinations to a separate struct

2020-03-05 Thread Marton Balint
Signed-off-by: Marton Balint --- libavformat/mxf.c | 32 libavformat/mxf.h | 5 + 2 files changed, 21 insertions(+), 16 deletions(-) diff --git a/libavformat/mxf.c b/libavformat/mxf.c index 10ccd770e3..80626e2a16 100644 --- a/libavformat/mxf.c +++

[FFmpeg-devel] [PATCH v2 1/7] avformat/audiointerleave: disallow using a samples_per_frame array

2020-03-05 Thread Marton Balint
Only MXF used an actual sample array, and that is unneeded there because simple rounding rules can be used instead. Signed-off-by: Marton Balint --- libavformat/audiointerleave.c | 24 +++- libavformat/audiointerleave.h | 7 --- libavformat/gxfenc.c | 2 +-

[FFmpeg-devel] [PATCH v2 5/7] avformat/mxfenc: factorize timecode checking and setting

2020-03-05 Thread Marton Balint
Signed-off-by: Marton Balint --- libavformat/mxfenc.c | 32 +--- 1 file changed, 13 insertions(+), 19 deletions(-) diff --git a/libavformat/mxfenc.c b/libavformat/mxfenc.c index 93d840ebbf..51e2dc5f31 100644 --- a/libavformat/mxfenc.c +++ b/libavformat/mxfenc.c @@

[FFmpeg-devel] [PATCH v2 2/7] avformat/mxf: get rid of samples per frame array usage

2020-03-05 Thread Marton Balint
Signed-off-by: Marton Balint --- libavformat/mxf.c| 44 libavformat/mxf.h| 6 -- libavformat/mxfdec.c | 23 +++ libavformat/mxfenc.c | 22 ++ 4 files changed, 17 insertions(+), 78 deletions(-) diff

[FFmpeg-devel] [PATCH v2 4/7] avformat/mxfenc: add some missing content package rates

2020-03-05 Thread Marton Balint
Fixes ticket #8523. Signed-off-by: Marton Balint --- libavformat/mxf.c | 13 + 1 file changed, 13 insertions(+) diff --git a/libavformat/mxf.c b/libavformat/mxf.c index 80626e2a16..7d154ca9d3 100644 --- a/libavformat/mxf.c +++ b/libavformat/mxf.c @@ -139,10 +139,23 @@ static const

Re: [FFmpeg-devel] [PATCH] avformat/hlsenc: Fix initial setting for start_pts

2020-03-05 Thread Hongcheng Zhong
- On Mar 6, 2020, at 12:55 AM, Andreas Rheinhardt andreas.rheinha...@gmail.com wrote: > Hongcheng Zhong: >> - On Mar 5, 2020, at 11:38 PM, lq l...@chinaffmpeg.org wrote: >> 2020年3月5日 下午9:41,Hongcheng Zhong 写道: This patch fixes Bug #8469 If x264 baseline profile is

Re: [FFmpeg-devel] [PATCH] avformat/hlsenc: Fix initial setting for start_pts

2020-03-05 Thread Andreas Rheinhardt
Hongcheng Zhong: > - On Mar 5, 2020, at 11:38 PM, lq l...@chinaffmpeg.org wrote: > >>> 2020年3月5日 下午9:41,Hongcheng Zhong 写道: >>> >>> This patch fixes Bug #8469 >>> If x264 baseline profile is used with other profiles, >>> start_pts will be initialized to audio stream's first pts, >>> while

Re: [FFmpeg-devel] [PATCH] avformat/hlsenc: Fix initial setting for start_pts

2020-03-05 Thread Hongcheng Zhong
- On Mar 5, 2020, at 11:38 PM, lq l...@chinaffmpeg.org wrote: >> 2020年3月5日 下午9:41,Hongcheng Zhong 写道: >> >> This patch fixes Bug #8469 >> If x264 baseline profile is used with other profiles, >> start_pts will be initialized to audio stream's first pts, >> while the duration is calculated

Re: [FFmpeg-devel] [PATCH] dashenc: more accurate time values in playlist.mpd

2020-03-05 Thread Alfred E. Heggestad
On 05/03/2020 15:37, Carl Eugen Hoyos wrote: Am Do., 5. März 2020 um 11:08 Uhr schrieb Alfred E. Heggestad : -int64_t update_period = c->last_duration / AV_TIME_BASE; +double update_period = (double)c->last_duration / AV_TIME_BASE; Can't you instead do int64 update_period =

Re: [FFmpeg-devel] [PATCH] avformat/hlsenc: Fix initial setting for start_pts

2020-03-05 Thread Steven Liu
> 2020年3月5日 下午9:41,Hongcheng Zhong 写道: > > This patch fixes Bug #8469 > If x264 baseline profile is used with other profiles, > start_pts will be initialized to audio stream's first pts, > while the duration is calculated based on video stream's pts. > In this patch the start_pts is

Re: [FFmpeg-devel] [PATCH 2/3] avformat: add raw AC-4 demuxer

2020-03-05 Thread James Almer
On 3/5/2020 9:42 AM, Paul B Mahol wrote: > On 3/5/20, Lynne wrote: >> Mar 5, 2020, 11:57 by one...@gmail.com: >> >>> On 3/5/20, Paul B Mahol wrote: >>> On 3/5/20, Andreas Rheinhardt wrote: > Paul B Mahol: > >> On 3/5/20, Andreas Rheinhardt wrote: >> >>> Am

Re: [FFmpeg-devel] [PATCH] dashenc: more accurate time values in playlist.mpd

2020-03-05 Thread Carl Eugen Hoyos
Am Do., 5. März 2020 um 11:08 Uhr schrieb Alfred E. Heggestad : > -int64_t update_period = c->last_duration / AV_TIME_BASE; > +double update_period = (double)c->last_duration / AV_TIME_BASE; Can't you instead do int64 update_period = last_duration * 1000 / AV_TIME_BASE to avoid

[FFmpeg-devel] [PATCH] avformat/hlsenc: Fix initial setting for start_pts

2020-03-05 Thread Hongcheng Zhong
This patch fixes Bug #8469 If x264 baseline profile is used with other profiles, start_pts will be initialized to audio stream's first pts, while the duration is calculated based on video stream's pts. In this patch the start_pts is initialized with the correct stream's first pts. Signed-off-by:

Re: [FFmpeg-devel] [PATCH 2/3] avformat: add raw AC-4 demuxer

2020-03-05 Thread Lynne
Mar 5, 2020, 11:57 by one...@gmail.com: > On 3/5/20, Paul B Mahol wrote: > >> On 3/5/20, Andreas Rheinhardt wrote: >> >>> Paul B Mahol: >>> On 3/5/20, Andreas Rheinhardt wrote: > Am 05.03.2020 um 00:05 schrieb James Almer: > >> On 3/4/2020 7:51 PM, Paul B Mahol wrote:

Re: [FFmpeg-devel] [PATCH 2/3] avformat: add raw AC-4 demuxer

2020-03-05 Thread Paul B Mahol
On 3/5/20, Lynne wrote: > Mar 5, 2020, 11:57 by one...@gmail.com: > >> On 3/5/20, Paul B Mahol wrote: >> >>> On 3/5/20, Andreas Rheinhardt wrote: >>> Paul B Mahol: > On 3/5/20, Andreas Rheinhardt wrote: > >> Am 05.03.2020 um 00:05 schrieb James Almer: >> >>> On

Re: [FFmpeg-devel] [PATCH 2/3] avformat: add raw AC-4 demuxer

2020-03-05 Thread Paul B Mahol
On 3/5/20, Paul B Mahol wrote: > On 3/5/20, Andreas Rheinhardt wrote: >> Paul B Mahol: >>> On 3/5/20, Andreas Rheinhardt wrote: Am 05.03.2020 um 00:05 schrieb James Almer: > On 3/4/2020 7:51 PM, Paul B Mahol wrote: >> On 3/4/20, James Almer wrote: >>> On 3/4/2020 7:26 PM, Paul

Re: [FFmpeg-devel] [PATCH 2/3] avformat: add raw AC-4 demuxer

2020-03-05 Thread Paul B Mahol
On 3/5/20, Andreas Rheinhardt wrote: > Paul B Mahol: >> On 3/5/20, Andreas Rheinhardt wrote: >>> Am 05.03.2020 um 00:05 schrieb James Almer: On 3/4/2020 7:51 PM, Paul B Mahol wrote: > On 3/4/20, James Almer wrote: >> On 3/4/2020 7:26 PM, Paul B Mahol wrote: >>> Signed-off-by:

Re: [FFmpeg-devel] [PATCH 2/3] avformat: add raw AC-4 demuxer

2020-03-05 Thread Andreas Rheinhardt
Paul B Mahol: > On 3/5/20, Andreas Rheinhardt wrote: >> Am 05.03.2020 um 00:05 schrieb James Almer: >>> On 3/4/2020 7:51 PM, Paul B Mahol wrote: On 3/4/20, James Almer wrote: > On 3/4/2020 7:26 PM, Paul B Mahol wrote: >> Signed-off-by: Paul B Mahol >> --- >>

Re: [FFmpeg-devel] [PATCH 2/3] avformat: add raw AC-4 demuxer

2020-03-05 Thread Paul B Mahol
On 3/5/20, Andreas Rheinhardt wrote: > Am 05.03.2020 um 00:05 schrieb James Almer: >> On 3/4/2020 7:51 PM, Paul B Mahol wrote: >>> On 3/4/20, James Almer wrote: On 3/4/2020 7:26 PM, Paul B Mahol wrote: > Signed-off-by: Paul B Mahol > --- > libavformat/Makefile | 1 +

Re: [FFmpeg-devel] [PATCH] dashenc: more accurate time values in playlist.mpd

2020-03-05 Thread Alfred E. Heggestad
On 05/03/2020 11:40, Moritz Barsnick wrote: On Thu, Mar 05, 2020 at 11:01:19 +0100, Alfred E. Heggestad wrote: -avio_printf(out, "\tsuggestedPresentationDelay=\"PT%"PRId64"S\"\n", c->last_duration / AV_TIME_BASE); +avio_printf(out,

Re: [FFmpeg-devel] [PATCH] dashenc: more accurate time values in playlist.mpd

2020-03-05 Thread Moritz Barsnick
On Thu, Mar 05, 2020 at 11:01:19 +0100, Alfred E. Heggestad wrote: > -avio_printf(out, > "\tsuggestedPresentationDelay=\"PT%"PRId64"S\"\n", c->last_duration / > AV_TIME_BASE); > +avio_printf(out, > "\tsuggestedPresentationDelay=\"PT%.3fS\"\n", (double)c->last_duration / >

Re: [FFmpeg-devel] [PATCH 2/3] avformat: add raw AC-4 demuxer

2020-03-05 Thread Paul B Mahol
On 3/5/20, James Almer wrote: > On 3/4/2020 7:51 PM, Paul B Mahol wrote: >> On 3/4/20, James Almer wrote: >>> On 3/4/2020 7:26 PM, Paul B Mahol wrote: Signed-off-by: Paul B Mahol --- libavformat/Makefile | 1 + libavformat/ac4dec.c | 104

[FFmpeg-devel] [PATCH] dashenc: more accurate time values in playlist.mpd

2020-03-05 Thread Alfred E. Heggestad
use 3 decimals for the following items: - minBufferTime - minimumUpdatePeriod - timeShiftBufferDepth - suggestedPresentationDelay This improves buffering with shaka player Signed-off-by: Alfred E. Heggestad --- libavformat/dashenc.c | 8 1 file changed, 4 insertions(+), 4

[FFmpeg-devel] [PATCH v8 4/4] avcodec/mpeg12dec: Add CPB coded side data

2020-03-05 Thread Nicolas Gaullier
This fixes mpeg2video stream copies to mpeg muxer like this: ffmpeg -i xdcamhd.mxf -c:v copy output.mpg --- libavcodec/mpeg12dec.c | 7 +++ tests/ref/fate/mxf-probe-d10 | 3 +++ tests/ref/fate/ts-demux | 2 +- 3 files changed, 11 insertions(+), 1 deletion(-) diff --git

[FFmpeg-devel] [PATCH v8 3/4] avcodec/utils: Fix ff_add_cpb_side_data() add twice

2020-03-05 Thread Nicolas Gaullier
Makes it behave similarly to av_stream_add_side_data(). --- libavcodec/utils.c | 5 + 1 file changed, 5 insertions(+) diff --git a/libavcodec/utils.c b/libavcodec/utils.c index c4dc136d3c..08e2d5b68b 100644 --- a/libavcodec/utils.c +++ b/libavcodec/utils.c @@ -1980,6 +1980,11 @@

[FFmpeg-devel] [PATCH v8 1/4] avcodec/mpeg12dec: Do not alter avctx->rc_buffer_size

2020-03-05 Thread Nicolas Gaullier
--- libavcodec/mpeg12dec.c | 9 + 1 file changed, 5 insertions(+), 4 deletions(-) diff --git a/libavcodec/mpeg12dec.c b/libavcodec/mpeg12dec.c index 17f9495a1d..2945728edd 100644 --- a/libavcodec/mpeg12dec.c +++ b/libavcodec/mpeg12dec.c @@ -64,6 +64,7 @@ typedef struct Mpeg1Context {

[FFmpeg-devel] [PATCH v8 2/4] avformat/utils: Make find_stream_info get side data from codec context

2020-03-05 Thread Nicolas Gaullier
This will allow probing input coded side data, and also forwarding them to the output. --- libavformat/utils.c | 18 ++ 1 file changed, 18 insertions(+) diff --git a/libavformat/utils.c b/libavformat/utils.c index cb15f6a4b3..a58e47fabc 100644 --- a/libavformat/utils.c +++

[FFmpeg-devel] [PATCH v8 0/4] Fix mpeg1/2 stream copy

2020-03-05 Thread Nicolas Gaullier
I have not received any feedback yet. I repost here with all 4 patches, maybe it is easier for the review - 1/4: fix mpeg12 decoder use of avctx->rc_buffer_size with is forbidden by API doc - 2/4,3/4,4/4: fix mpeg1/2 stream copy Thank you Nicolas Gaullier (4): avcodec/mpeg12dec: Do not alter