Re: [FFmpeg-devel] PATCH[0/4]

Amit Kale 于2018年9月26日周三 下午1:50写道： > > Hi, > > I am sending an HLS manifest file output patch containing fixes broken down > in four parts as follows > 0001-Fix-computation-of-vs-start_pos.patch > 0002-Add-a-new-hls_flag-peak_segment_bw.patch > 0003-Adds-a-new-hls_flag-avg_bw.patch > 0004-fix-mast

Re: [FFmpeg-devel] [PATCH] libaomdec: add row_mt option

On 9/26/18, Tristan Matthews wrote: > Partially fixes #7456 > --- > libavcodec/libaomdec.c | 62 ++ > 1 file changed, 62 insertions(+) > > diff --git a/libavcodec/libaomdec.c b/libavcodec/libaomdec.c > index 2530c9f76b..15cda2940a 100644 > --- a/libavcodec/

Re: [FFmpeg-devel] [PATCH 4/4] Delay freeing of hls stream data

> Am 26.09.2018 um 08:00 schrieb Amit Kale : > > This patch delays freeing of hls stream data, so that it's available for > bandwidth calculation. > Otherwise the previous patches would cause a segfault in this code. This means that your patchset was not split correctly: No part of your patch

Re: [FFmpeg-devel] [PATCH] libavformat/segment: strftime date sub-directories

On 26 September 2018 at 00:18, Steven Liu wrote: > > > > On Sep 26, 2018, at 06:49, James Courtier-Dutton > wrote: > > On 25 September 2018 at 23:24, Steven Liu wrote: > yes, the error message is confusing, maybe change it here is better. > > > > +av_log(oc, AV_LOG_ERROR, "Could not

[FFmpeg-devel] [PATCH 1/2] lavc/h264dec: don't error out when receiving multiple IDR slices

From: Josh de Kock This error isn't particularly helpful as checking for mixed IDR/non-IDR NALUs would need to be done at a higher level to actually be accurate. Removing the error allows an API user to send individual slice NALUs (i.e. incomplete frames) so they can take advantage of slice threa

[FFmpeg-devel] [PATCH 2/2] fate: add api-h264-slice test

From: Josh de Kock This test ensures that you are able to send N number of slice NALUs in slice threaded mode to be decoded simultaneously --- tests/api/Makefile | 1 + tests/api/api-h264-slice-test.c | 208 + tests/fate/api.mak | 4 + tests/ref

Re: [FFmpeg-devel] [PATCH] libavformat/segment: strftime date sub-directories

James Courtier-Dutton 于2018年9月26日周三 下午5:08写道： > > On 26 September 2018 at 00:18, Steven Liu wrote: > > > > > > > > On Sep 26, 2018, at 06:49, James Courtier-Dutton > > wrote: > > > On 25 September 2018 at 23:24, Steven Liu wrote: > > yes, the error message is confusing, maybe change it here is

[FFmpeg-devel] [PATCH V1] ffmpeg: fix video_delay warning for HEVC/MPEG4 decoding

For HEVC/MPEG4, we also need video_delay from the decoder, when decoding some HEVC/MPEG4 clips, got numerous log like: "video_delay is larger in decoder than demuxer", similar ticket: #3711 fix ticket: #6109 Signed-off-by: Lin Xie Signed-off-by: Jun Zhao --- fftools/ffmpeg.c |4 +++- 1 file

Re: [FFmpeg-devel] [PATCH 4/4] Delay freeing of hls stream data

Carl Eugen Hoyos 于2018年9月26日周三 下午4:22写道： > > > > > Am 26.09.2018 um 08:00 schrieb Amit Kale : > > > > This patch delays freeing of hls stream data, so that it's available for > > bandwidth calculation. > > > Otherwise the previous patches would cause a segfault in this code. > > This means that y

[FFmpeg-devel] [PATCH V2] ffmpeg: fix video_delay warning for HEVC/MPEG4 decoding

For HEVC/MPEG4, we also need video_delay from the decoder, when decoding some HEVC/MPEG4 clips, got numerous log like: "video_delay is larger in decoder than demuxer", similar ticket: #3711 fix ticket: #6019 Signed-off-by: Lin Xie Signed-off-by: Jun Zhao --- fftools/ffmpeg.c |4 +++- 1 file

Re: [FFmpeg-devel] [PATCH 2/2] fate: add api-h264-slice test

On Wed, Sep 26, 2018 at 10:35:05AM +0100, jos...@ob-encoder.com wrote: > From: Josh de Kock > > This test ensures that you are able to send N number of slice NALUs in slice > threaded mode to be decoded simultaneously > --- > tests/api/Makefile | 1 + > tests/api/api-h264-slice-t

[FFmpeg-devel] [PATCH 0/1] mov.c: improvement to fMP4 bitrate estimation

Current code estimates bitrate by taking size of a single fragment and dividing by the total duration. This patch changes calculation to be based on duration used for framerate estimation. Example ffprobes BEFORE patch (note video stream 15 kb/s, audio stream 0 kb/s): ffprobe version N-91996-g9

[FFmpeg-devel] [PATCH 0/1] mov.c: improvement to fMP4 bitrate estimation

Current code estimates bitrate by taking size of a single fragment and dividing by the total duration. This patch changes calculation to be based on duration used for framerate estimation. Nick Ryan (1): possible fix to correct (improve) bitrate estimation for streams in fragmented MP4 whe

[FFmpeg-devel] [PATCH 1/1] possible fix to correct (improve) bitrate estimation for streams in fragmented MP4 when calculation is based on trex_data

Signed-off-by: Nick Ryan --- libavformat/mov.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/libavformat/mov.c b/libavformat/mov.c index 540e5ca057..67015a72a1 100644 --- a/libavformat/mov.c +++ b/libavformat/mov.c @@ -7459,14 +7459,14 @@ static int mov_read_header(AVFor

[FFmpeg-devel] [PATCH] libaomdec: add row_mt option

Partially fixes #7456 --- libavcodec/libaomdec.c | 62 ++ 1 file changed, 62 insertions(+) diff --git a/libavcodec/libaomdec.c b/libavcodec/libaomdec.c index 2530c9f76b..6740f98be1 100644 --- a/libavcodec/libaomdec.c +++ b/libavcodec/libaomdec.c @@ -23,20 +

[FFmpeg-devel] [PATCH 1/1] Trying to fix trac ticket #7359

Signed-off-by: Nick Ryan --- libavformat/mov.c | 19 --- 1 file changed, 16 insertions(+), 3 deletions(-) diff --git a/libavformat/mov.c b/libavformat/mov.c index 67015a72a1..587513e06e 100644 --- a/libavformat/mov.c +++ b/libavformat/mov.c @@ -7605,15 +7605,15 @@ static int mov_

[FFmpeg-devel] [PATCH 0/1] Trying to fix trac ticket #7359

Hello, With reference to: https://trac.ffmpeg.org/ticket/7359 I believe another way this issue manifests itself is using ffplay and trying to seek forward 10 seconds with the right arrow key: playback freezes. I have dug into this and developed a hack which seems to resolve the issue, BUT I

Re: [FFmpeg-devel] [PATCH] libaomdec: add row_mt option

On Wed, Sep 26, 2018 at 2:27 PM Tristan Matthews wrote: > > Partially fixes #7456 > --- row-mt is enabled by default in recent libaom versions, and they even removed the option for it from aomdec, so do we really need that option? - Hendrik ___ ffmpeg-

Re: [FFmpeg-devel] [PATCH] avformat/matroskaenc: add reserve free space option

> On Sep 12, 2018, at 11:56 AM, Sigríður Regína Sigurþórsdóttir > wrote: > > On Thu, Sep 6, 2018 at 3:31 PM James Almer > wrote: >> >> On 9/6/2018 4:18 PM, James Darnley wrote: >>> On 2018-09-06 19:39, Sigríður Regína Sigurþórsdóttir wrote: +if (s->metadata_

Re: [FFmpeg-devel] [PATCH 2/9] avcodec/gdv: Replace divisions by shifts in rescale()

Michael Niedermayer (2018-09-24): > CC-ing you so you dont miss this (though its not important) > > if i hear nothing from anyone i will apply the patchset as is. > IIUC james is ok with that. > But as said i can redo it with unsigned if people want. > Just dont want to spend time redoing a patch

Re: [FFmpeg-devel] [PATCH V1] ffmpeg: fix video_delay warning for HEVC/MPEG4 decoding

2018-09-26 11:49 GMT+02:00, Jun Zhao : > For HEVC/MPEG4, we also need video_delay from the decoder, when decoding > some HEVC/MPEG4 clips, got numerous log like: > "video_delay is larger in decoder than demuxer", similar ticket: #3711 Can you provide such a hevc sample? Carl Eugen ___

Re: [FFmpeg-devel] [PATCH 1/4] avformat/matroskaenc: remove unnecessary additional codec tags

On 9/21/2018 8:01 PM, James Almer wrote: > On 9/21/2018 7:44 PM, Carl Eugen Hoyos wrote: >> >> >>> Am 21.09.2018 um 23:29 schrieb James Almer : >>> >>> They are listed in riff.c already. >>> >>> Signed-off-by: James Almer >>> --- >>> libavformat/matroskaenc.c | 3 --- >>> 1 file changed, 3 deletion

Re: [FFmpeg-devel] [PATCH] libaomdec: add row_mt option

On Wed, Sep 26, 2018 at 9:43 AM Hendrik Leppkes wrote: > On Wed, Sep 26, 2018 at 2:27 PM Tristan Matthews > wrote: > > > > Partially fixes #7456 > > --- > > row-mt is enabled by default in recent libaom versions, and they even > removed the option for it from aomdec, so do we really need that >

Re: [FFmpeg-devel] [PATCH 1/4] avformat/matroskaenc: remove unnecessary additional codec tags

2018-09-21 23:29 GMT+02:00, James Almer : > They are listed in riff.c already. > > Signed-off-by: James Almer > --- > libavformat/matroskaenc.c | 3 --- > 1 file changed, 3 deletions(-) > > diff --git a/libavformat/matroskaenc.c b/libavformat/matroskaenc.c > index a0e2f426f7..61efe2e3f8 100644 >

Re: [FFmpeg-devel] [PATCH V2] ffmpeg: fix video_delay warning for HEVC/MPEG4 decoding

On Wed, Sep 26, 2018 at 06:03:19PM +0800, Jun Zhao wrote: > For HEVC/MPEG4, we also need video_delay from the decoder, when decoding > some HEVC/MPEG4 clips, got numerous log like: > "video_delay is larger in decoder than demuxer", similar ticket: #3711 for mpeg4 this looks incorrect. What your

[FFmpeg-devel] [PATCH]lavf/mpeg: Allow demuxing hevc

Hi! A HandBrake user provided a program stream with hevc, attached patch allows demuxing without auto-detection. Please comment, Carl Eugen From c0d3208edbf2243fd7c554e1ac0fa0afe0591778 Mon Sep 17 00:00:00 2001 From: Carl Eugen Hoyos Date: Wed, 26 Sep 2018 22:39:18 +0200 Subject: [PATCH] lavf/mp

Re: [FFmpeg-devel] [PATCH] avdevice/decklink: Add option to align Capture start time

On Tue, 25 Sep 2018, Jeyapal, Karthick wrote: On 9/24/18 7:42 PM, Devin Heitmueller wrote: Hello Karthick, On Sep 24, 2018, at 7:49 AM, Karthick J wrote: From: Karthick Jeyapal This option is useful for maintaining input synchronization across N different hardware devices deployed for

Re: [FFmpeg-devel] [PATCH]lavf/mpeg: Allow demuxing hevc

On Wed, Sep 26, 2018 at 11:46 PM, Carl Eugen Hoyos wrote: > Hi! > > A HandBrake user provided a program stream with hevc, attached > patch allows demuxing without auto-detection. > > Please comment, Carl Eugen > Verified that the identifier matches ITU-T H.222 2014/10 (and what's in mpegts.{c,h})

Re: [FFmpeg-devel] [PATCH]lavf/matroskadec: Simplify string length calculation slightly

2018-09-19 19:18 GMT+02:00, Carl Eugen Hoyos : > Attached patch removes useless sizeof's, we require sizeof(char)==1 afaict. I will push this if there are no objections. Carl Eugen ___ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org http://ffmpeg.org

Re: [FFmpeg-devel] [FFmpeg-cvslog] lavc/h264_levels: Avoid integer overflow in bitrate

2018-09-25 0:16 GMT+02:00, Mark Thompson : > ffmpeg | branch: master | Mark Thompson | Mon Sep 24 23:03:32 > 2018 +0100| [581b4125aa187f2cf848d7a27e6128573c80dc64] | committer: Mark > Thompson > > lavc/h264_levels: Avoid integer overflow in bitrate > > Fixes CID #1439656. > >> http://git.videolan.

Re: [FFmpeg-devel] [FFmpeg-cvslog] h264_metadata: Avoid integer overflow in bitrate

2018-09-25 0:16 GMT+02:00, Mark Thompson : > ffmpeg | branch: master | Mark Thompson | Mon Sep 24 22:45:50 > 2018 +0100| [321294adb788b5e143fcec776cdf1daf79ed921c] | committer: Mark > Thompson > > h264_metadata: Avoid integer overflow in bitrate > > Fixes CID #1439664. > >> http://git.videolan.org

[FFmpeg-devel] [PATCH 1/3] avformat/utils: Do not ignore failure in extract_extradata_init()

We check for the documented explanation of the "Ignore code" in extract_extradata_check() already Signed-off-by: Michael Niedermayer --- libavformat/utils.c | 8 ++-- 1 file changed, 2 insertions(+), 6 deletions(-) diff --git a/libavformat/utils.c b/libavformat/utils.c index c973a7e0c5..7e

[FFmpeg-devel] [PATCH 3/3] avformat/utils: Fix potential integer overflow in extract_extradata()

Signed-off-by: Michael Niedermayer --- libavformat/utils.c | 4 +++- 1 file changed, 3 insertions(+), 1 deletion(-) diff --git a/libavformat/utils.c b/libavformat/utils.c index c1835b1ab5..3e99478ad9 100644 --- a/libavformat/utils.c +++ b/libavformat/utils.c @@ -3544,7 +3544,9 @@ static int extr

[FFmpeg-devel] [PATCH 2/3] avformat/utils: Do not use "i" as a context pointer, "i" is normally the integer counter in loops

This avoids surprising developers. Its bad to surprise developers with such unexpected things. Signed-off-by: Michael Niedermayer --- libavformat/utils.c | 42 +- 1 file changed, 21 insertions(+), 21 deletions(-) diff --git a/libavformat/utils.c b/libavfo

Re: [FFmpeg-devel] [PATCH] libavformat/segment: strftime date sub-directories

On Wed, Sep 26, 2018 at 10:07:58AM +0100, James Courtier-Dutton wrote: > On 26 September 2018 at 00:18, Steven Liu wrote: > > > > > > > > On Sep 26, 2018, at 06:49, James Courtier-Dutton > > wrote: > > > On 25 September 2018 at 23:24, Steven Liu wrote: > > yes, the error message is confusing, m

Re: [FFmpeg-devel] [PATCH 3/3] avformat/utils: Fix potential integer overflow in extract_extradata()

On 9/26/2018 7:00 PM, Michael Niedermayer wrote: > Signed-off-by: Michael Niedermayer > --- > libavformat/utils.c | 4 +++- > 1 file changed, 3 insertions(+), 1 deletion(-) > > diff --git a/libavformat/utils.c b/libavformat/utils.c > index c1835b1ab5..3e99478ad9 100644 > --- a/libavformat/utils.

Re: [FFmpeg-devel] [PATCH 2/3] avformat/utils: Do not use "i" as a context pointer, "i" is normally the integer counter in loops

On 9/26/2018 7:00 PM, Michael Niedermayer wrote: > This avoids surprising developers. Its bad to surprise developers with > such unexpected things. > > Signed-off-by: Michael Niedermayer > --- > libavformat/utils.c | 42 +- > 1 file changed, 21 insertions(

Re: [FFmpeg-devel] [FFmpeg-cvslog] lavc/h264_levels: Avoid integer overflow in bitrate

On 26/09/18 22:43, Carl Eugen Hoyos wrote: > 2018-09-25 0:16 GMT+02:00, Mark Thompson : >> ffmpeg | branch: master | Mark Thompson | Mon Sep 24 >> 23:03:32 >> 2018 +0100| [581b4125aa187f2cf848d7a27e6128573c80dc64] | committer: Mark >> Thompson >> >> lavc/h264_levels: Avoid integer overflow in bit

Re: [FFmpeg-devel] [FFmpeg-cvslog] lavc/h264_levels: Avoid integer overflow in bitrate

2018-09-27 0:15 GMT+02:00, Mark Thompson : > On 26/09/18 22:43, Carl Eugen Hoyos wrote: >> 2018-09-25 0:16 GMT+02:00, Mark Thompson : >>> ffmpeg | branch: master | Mark Thompson | Mon Sep 24 >>> 23:03:32 >>> 2018 +0100| [581b4125aa187f2cf848d7a27e6128573c80dc64] | committer: Mark >>> Thompson >>>

Re: [FFmpeg-devel] [PATCH 2/3] avformat/utils: Do not use "i" as a context pointer, "i" is normally the integer counter in loops

2018-09-27 0:00 GMT+02:00, Michael Niedermayer : > This avoids surprising developers. Its bad to surprise developers with > such unexpected things. +1 Carl Eugen ___ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org http://ffmpeg.org/mailman/listinfo/ff

Re: [FFmpeg-devel] [FFmpeg-cvslog] h264_metadata: Avoid integer overflow in bitrate

On 26/09/18 22:50, Carl Eugen Hoyos wrote: > 2018-09-25 0:16 GMT+02:00, Mark Thompson : >> ffmpeg | branch: master | Mark Thompson | Mon Sep 24 >> 22:45:50 >> 2018 +0100| [321294adb788b5e143fcec776cdf1daf79ed921c] | committer: Mark >> Thompson >> >> h264_metadata: Avoid integer overflow in bitrat

Re: [FFmpeg-devel] [FFmpeg-cvslog] lavc/h264_levels: Avoid integer overflow in bitrate

On 26/09/18 23:18, Carl Eugen Hoyos wrote: > 2018-09-27 0:15 GMT+02:00, Mark Thompson : >> On 26/09/18 22:43, Carl Eugen Hoyos wrote: >>> 2018-09-25 0:16 GMT+02:00, Mark Thompson : ffmpeg | branch: master | Mark Thompson | Mon Sep 24 23:03:32 2018 +0100| [581b4125aa187f2cf848d7a27e6

Re: [FFmpeg-devel] [PATCH] libavformat/segment: strftime date sub-directories

> On Sep 27, 2018, at 06:10, Michael Niedermayer wrote: > > On Wed, Sep 26, 2018 at 10:07:58AM +0100, James Courtier-Dutton wrote: >> On 26 September 2018 at 00:18, Steven Liu wrote: >> >>> >>> On Sep 26, 2018, at 06:49, James Courtier-Dutton >>> wrote: On 25 September 2018 at 23

Re: [FFmpeg-devel] [PATCH v2 2/3] lavc: Add coded bitstream read/write support for AV1

On 23/09/18 23:33, Mark Thompson wrote: > On 18/09/18 01:25, James Almer wrote: >> On 9/17/2018 8:47 PM, Mark Thompson wrote: >>> ... >> >> Looks good now. Thanks! > > Great. Does anyone else have any comments? I'll push this in two days if > there isn't anything further. And pushed. Thank yo

[FFmpeg-devel] [PATCH 1/4] fate/libavcodec: Fix config dependency of h264_levels test

--- tests/fate/libavcodec.mak | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/tests/fate/libavcodec.mak b/tests/fate/libavcodec.mak index aa4c36b112..5dde1243fa 100644 --- a/tests/fate/libavcodec.mak +++ b/tests/fate/libavcodec.mak @@ -46,7 +46,7 @@ fate-dct8x8: libavcodec/test

[FFmpeg-devel] [PATCH 3/4] lavc/h265_profile_level: Do not allow high tier at level < 4

--- libavcodec/h265_profile_level.c | 18 -- 1 file changed, 12 insertions(+), 6 deletions(-) diff --git a/libavcodec/h265_profile_level.c b/libavcodec/h265_profile_level.c index 692252bb4d..126f8dbc2a 100644 --- a/libavcodec/h265_profile_level.c +++ b/libavcodec/h265_profile_leve

[FFmpeg-devel] [PATCH 2/4] lavc/h265_profile_level: Fix DPB size calculation

The maxDpbPicBuf value which is used in the DPB size calculation depends on the profile (it's usually 6, but 7 for screen-extended profiles). --- libavcodec/h265_profile_level.c | 86 - libavcodec/h265_profile_level.h | 1 + 2 files changed, 44 insertions(+), 43 de

[FFmpeg-devel] [PATCH 4/4] lavc/h265_profile_level: Add unit test

Operates in the same way as the h264-levels test. --- libavcodec/Makefile| 1 + libavcodec/tests/h265_levels.c | 297 + tests/fate/libavcodec.mak | 5 + 3 files changed, 303 insertions(+) create mode 100644 libavcodec/tests/h265_levels.c diff

Re: [FFmpeg-devel] [PATCH V2] ffmpeg: fix video_delay warning for HEVC/MPEG4 decoding

On Thu, Sep 27, 2018 at 4:09 AM Michael Niedermayer wrote: > On Wed, Sep 26, 2018 at 06:03:19PM +0800, Jun Zhao wrote: > > For HEVC/MPEG4, we also need video_delay from the decoder, when decoding > > some HEVC/MPEG4 clips, got numerous log like: > > "video_delay is larger in decoder than demuxer"

Re: [FFmpeg-devel] [PATCH V1] ffmpeg: fix video_delay warning for HEVC/MPEG4 decoding

On Wed, Sep 26, 2018 at 11:36 PM Carl Eugen Hoyos wrote: > > 2018-09-26 11:49 GMT+02:00, Jun Zhao : > > For HEVC/MPEG4, we also need video_delay from the decoder, when decoding > > some HEVC/MPEG4 clips, got numerous log like: > > "video_delay is larger in decoder than demuxer", similar ticket: #3

[FFmpeg-devel] [PATCH] libavcodec/libaomenc.c: Added code for computing PSNR/SSIM for libaom encoder.Updated the patch to read the AV_PICTURE_TYPE_I flag for AOM.

--- libavcodec/libaomenc.c | 83 ++ 1 file changed, 75 insertions(+), 8 deletions(-) diff --git a/libavcodec/libaomenc.c b/libavcodec/libaomenc.c index 6a79d9b873..4e75ca0b18 100644 --- a/libavcodec/libaomenc.c +++ b/libavcodec/libaomenc.c @@ -50,6 +50,9 @@

Re: [FFmpeg-devel] [PATCH] libavcodec/libaomenc.c: Added code for computing PSNR/SSIM for libaom encoder.Updated the patch to read the AV_PICTURE_TYPE_I flag for AOM.

On 9/26/2018 10:42 PM, Sam John wrote: > --- > libavcodec/libaomenc.c | 83 ++ > 1 file changed, 75 insertions(+), 8 deletions(-) > > diff --git a/libavcodec/libaomenc.c b/libavcodec/libaomenc.c > index 6a79d9b873..4e75ca0b18 100644 > --- a/libavcodec/libao

Re: [FFmpeg-devel] [PATCH] libavformat/segment: strftime date sub-directories

Steven Liu 于2018年9月27日周四 上午6:37写道： > > > > > On Sep 27, 2018, at 06:10, Michael Niedermayer > > wrote: > > > > On Wed, Sep 26, 2018 at 10:07:58AM +0100, James Courtier-Dutton wrote: > >> On 26 September 2018 at 00:18, Steven Liu wrote: > >> > >>> > >>> > On Sep 26, 2018, at 06:49, James Co

Re: [FFmpeg-devel] [PATCH] libavformat/segment: strftime date sub-directories

Steven Liu 于2018年9月27日周四 下午2:53写道： > > Steven Liu 于2018年9月27日周四 上午6:37写道： > > > > > > > > > On Sep 27, 2018, at 06:10, Michael Niedermayer > > > wrote: > > > > > > On Wed, Sep 26, 2018 at 10:07:58AM +0100, James Courtier-Dutton wrote: > > >> On 26 September 2018 at 00:18, Steven Liu wrote: > >