[FFmpeg-devel] [PATCH] avformat/avio: check input URLContext value NULL

fix ticket id: #6846 Signed-off-by: Steven Liu --- libavformat/avio.c | 10 ++ 1 file changed, 6 insertions(+), 4 deletions(-) diff --git a/libavformat/avio.c b/libavformat/avio.c index 4dc468350c..63e82872f7 100644 --- a/libavformat/avio.c +++ b/libavformat/avio.c @@ -625,13 +625,15 @@

Re: [FFmpeg-devel] [PATCH 1/1] avformat/dashenc: Added configuration to override HTTP User-Agent

>On 11/20/17, 1:01 PM, "刘歧" wrote: >> 在 2017年11月8日，17:22，Karthick J 写道： >> +{ "http_user_agent", "override User-Agent field in HTTP header", >> OFFSET(user_agent), AV_OPT_TYPE_STRING, {.str = NULL}, 0, 0, E}, >What about dash_user_agent？ The reason is http_user_agent maybe get mean HTTP >Pr

Re: [FFmpeg-devel] [PATCH] avfilter: add normalize filter

Patch attached. R. 0001-avfilter-add-normalize-filter.patch Description: Binary data ___ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org http://ffmpeg.org/mailman/listinfo/ffmpeg-devel

Re: [FFmpeg-devel] [PATCH 1/1] avformat/dashenc: Added configuration to override HTTP User-Agent

> 在 2017年11月20日，15:59，Jeyapal, Karthick 写道： > >> On 11/20/17, 1:01 PM, "刘歧" wrote: >>> 在 2017年11月8日，17:22，Karthick J 写道： >>> +{ "http_user_agent", "override User-Agent field in HTTP header", >>> OFFSET(user_agent), AV_OPT_TYPE_STRING, {.str = NULL}, 0, 0, E}, >> What about dash_user_agen

Re: [FFmpeg-devel] [PATCH] ffmpeg: add return value check to suppress the build warning.

> 在 2017年11月20日，08:40，Jun Zhao 写道： > > > <0001-ffmpeg-add-return-value-check-to-supress-the-build-w.patch>___ > ffmpeg-devel mailing list > ffmpeg-devel@ffmpeg.org > http://ffmpeg.org/mailman/listinfo/ffmpeg-devel From 01a66eb4559fb3fec0765ee03417a65

Re: [FFmpeg-devel] [PATCH 2/2] hwcontext_vaapi: add the fourcc of I420 format map.

> 在 2017年11月20日，08:36，Jun Zhao 写道： > > <0002-hwcontext_vaapi-add-the-fourcc-of-I420-format-map.patch> From f31e492201b7a8c59bdf7c266c2df016057897c4 Mon Sep 17 00:00:00 2001 From: Jun Zhao Date: Sat, 18 Nov 2017 11:57:41 +0800 Subject: [PATCH 2/2] hwcontext_vaapi: add the fourcc of I420 format m

[FFmpeg-devel] [PATCH 1/3] lavu/qsv: remove unused variable

suppres build warning of "variable 's' is declared but not used" Signed-off-by: Zhong Li --- libavutil/hwcontext_qsv.c | 1 - 1 file changed, 1 deletion(-) diff --git a/libavutil/hwcontext_qsv.c b/libavutil/hwcontext_qsv.c index 8e43420..9b6040b 100644 --- a/libavutil/hwcontext_qsv.c +++ b/liba

[FFmpeg-devel] [PATCH v2 2/3] lavf/qsv_overlay: check the return value of ff_formats_ref()

suppress the build warning of "ignoring return value of ‘ff_formats_ref’, declared with attribute warn_unused_result" Signed-off-by: Zhong Li Reviewed-by: Carl Eugen Hoyos --- libavfilter/vf_overlay_qsv.c | 12 +--- 1 file changed, 9 insertions(+), 3 deletions(-) diff --git a/libavfilt

[FFmpeg-devel] [PATCH v2 3/3] lavf/qsv_vpp: check the return value of ff_formats_ref()

suppress the build warning of "ignoring return value of ‘ff_formats_ref’, declared with attribute warn_unused_result" Signed-off-by: Zhong Li Reviewed-by: Carl Eugen Hoyos --- libavfilter/vf_vpp_qsv.c | 9 +++-- 1 file changed, 7 insertions(+), 2 deletions(-) diff --git a/libavfilter/vf_vp

Re: [FFmpeg-devel] [PATCH] avfilter/vf_tile: add queue option

> Am 20.11.2017 um 06:25 schrieb Gyan Doshi : > > >> On 11/20/2017 5:04 AM, Thilo Borgmann wrote: >> >> min_start >> n_begin > > start_delay ? Just to complete my thoughts... init_padding would be good and for code-viewing people it is way better than queue. My thoughts are about the name o

Re: [FFmpeg-devel] [PATCH] avcodec: Implement mpeg1 nvdec hwaccel

simple enough, lgtm smime.p7s Description: S/MIME Cryptographic Signature ___ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org http://ffmpeg.org/mailman/listinfo/ffmpeg-devel

Re: [FFmpeg-devel] [PATCH] avcodec: Refactor common nvdec hwaccel logic

lgtm smime.p7s Description: S/MIME Cryptographic Signature ___ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org http://ffmpeg.org/mailman/listinfo/ffmpeg-devel

Re: [FFmpeg-devel] [PATCH 1/3] avcodec: Refactor common nvdec hwaccel logic

this version is good as well smime.p7s Description: S/MIME Cryptographic Signature ___ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org http://ffmpeg.org/mailman/listinfo/ffmpeg-devel

Re: [FFmpeg-devel] [PATCH 2/3] avcodec: Implement mpeg1 nvdec hwaccel

nvdec part looks good again smime.p7s Description: S/MIME Cryptographic Signature ___ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org http://ffmpeg.org/mailman/listinfo/ffmpeg-devel

Re: [FFmpeg-devel] [PATCH 3/3] avcodec: Implement mpeg4 nvdec hwaccel

Duplicate. Will remove. Can't really tell if the parameters are in any way correct, but I guess time will tell if some sample comes up that breaks it. lgtm with that duplicate removed smime.p7s Description: S/MIME Cryptographic Signature ___ ffmp

Re: [FFmpeg-devel] [PATCH 11/11] avformat/wavdecuse FFABS to instead of fabs

On 11/15/17, Steven Liu wrote: > Signed-off-by: Steven Liu > --- > libavformat/wavdec.c | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > NACK FFABS is slower than fabs. ___ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org http://ffmpeg.org/

[FFmpeg-devel] [PATCH] avfilter/drawbox: rename variable for maximum thickness

At present, the value name 'max' for maximum thickness in drawbox (and drawgrid) filter leads to a parse error if the thickness expression contains 'max(val1,val2)' i.e. [Eval @ ...] Invalid chars '(20,30)' at the end of expression 'max(20,30)' Renamed to 'fill'; tested & documented. Re

[FFmpeg-devel] [PATCH] Added Turing codec to ffmpeg

This patch contains the changes required to interface the Turing codec (http://turingcodec.org/) to ffmpeg --- LICENSE.md | 1 + configure | 6 + libavcodec/Makefile| 1 + libavcodec/allcodecs.c | 1 + libavcodec/libturing.c | 318 ++

Re: [FFmpeg-devel] [PATCH] avfilter: add normalize filter

On Mon, Nov 20, 2017 at 19:01:12 +1100, Richard Ling wrote: > Patch attached. You code nicely sticks to floats throughout, but at points like these: > +min[c].smoothed = (min[c].smoothed *s->independence) > ++ (rgb_min_smoothed * (1.0 - s->independence));

Re: [FFmpeg-devel] [PATCH 03/11] avutil: detect when AVX-512 is available

On 2017-11-10 03:11, James Almer wrote: > On 11/9/2017 8:58 AM, James Darnley wrote: >> @@ -154,6 +155,13 @@ int ff_get_cpu_flags_x86(void) >> if (ebx & 0x0100) >> rval |= AV_CPU_FLAG_BMI2; >> } >> +#if HAVE_AVX512 /* F, CD, BW, DQ, VL */ > > Nit: Maybe m

Re: [FFmpeg-devel] [PATCH 1/1] avformat/dashenc: Added configuration to override HTTP User-Agent

>On 11/20/17, 1:47 PM, "刘歧" wrote: >> 在 2017年11月20日，15:59，Jeyapal, Karthick 写道： >> >>> On 11/20/17, 1:01 PM, "刘歧" wrote: 在 2017年11月8日，17:22，Karthick J 写道： +{ "http_user_agent", "override User-Agent field in HTTP header", OFFSET(user_agent), AV_OPT_TYPE_STRING, {.str = NUL

Re: [FFmpeg-devel] [PATCH 1/2] hwcontext_vaapi: add support when driver return unimplemented.

On 20/11/17 00:36, Jun Zhao wrote: > From 5cbcd032de46e6a3f9563d1781776ea26728079d Mon Sep 17 00:00:00 2001 > From: Jun Zhao > Date: Sat, 18 Nov 2017 10:44:44 +0800 > Subject: [PATCH 1/2] hwcontext_vaapi: add support when driver return > unimplemented. > > iHD driver sometime return unimplemente

Re: [FFmpeg-devel] [PATCH 2/2] hwcontext_vaapi: add the fourcc of I420 format map.

On 20/11/17 00:36, Jun Zhao wrote: > From f31e492201b7a8c59bdf7c266c2df016057897c4 Mon Sep 17 00:00:00 2001 > From: Jun Zhao > Date: Sat, 18 Nov 2017 11:57:41 +0800 > Subject: [PATCH 2/2] hwcontext_vaapi: add the fourcc of I420 format map. > > VA-API 2.0 have enable the I420, so enable this map.

Re: [FFmpeg-devel] [PATCH 1/1] avdevice/decklink_dec: Autodetect the video input format

>On 11/20/17, 12:39 AM, "Marton Balint" wrote: >Thanks, there is one more thing I still don't get: [...] >> +// 1 second timeout >> +for (i = 0; i < 10; i++) { >> +av_usleep(10); >> +// Sometimes VideoInputFrameArrived is called before >> VideoInputFormatChanged >> +

Re: [FFmpeg-devel] [PATCH] avfilter: add normalize filter

Thanks Moritz. I'll update and repost a new patch. I also noticed another error of mine: I hadn't fully updated the examples in the documentation, after changes to the parameter. Regards R. ___ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org http://ffmp

Re: [FFmpeg-devel] [PATCH 1/1] avformat/dashenc: Added configuration to override HTTP User-Agent

> 在 2017年11月20日，19:31，Jeyapal, Karthick 写道： > >> On 11/20/17, 1:47 PM, "刘歧" wrote: > > >>> 在 2017年11月20日，15:59，Jeyapal, Karthick 写道： >>> On 11/20/17, 1:01 PM, "刘歧" wrote: > 在 2017年11月8日，17:22，Karthick J 写道： > +{ "http_user_agent", "override User-Agent field in HTTP heade

Re: [FFmpeg-devel] RPM library deployments

On Sunday, 19 November 2017 at 21:50, Moritz Barsnick wrote: > On Sat, Nov 18, 2017 at 10:45:44 -0900, Lou Logan wrote: > > On Fri, Nov 17, 2017, at 01:10 PM, Gardner, Greg - 0995 - MITLL wrote: > > > Hey folks, > > > > > > Could someone put me in touch with whoever builds the ffmpeg-libs RPM? > >

Re: [FFmpeg-devel] [PATCH] Added Turing codec to ffmpeg

On Mon, Nov 20, 2017 at 10:35:39 +, Matteo Naccari wrote: I don't recall all of the previous review remarks, but these may be new: > LICENSE.md | 1 + > configure | 6 + Please add a changelog entry. > librubberband > + libturing > libvidstab You

Re: [FFmpeg-devel] Refund request of travel costs for GSoC Summit 2017

2017-11-18 18:26 GMT+01:00 Thilo Borgmann : > some time ago Carl Eugen and me went to the GSoC Summit. > See review mail for details. > > Here comes the refund request for my flight. Costs were 478.82€ total I have sent my invoice over 932,27€ to Stefano. Thank you, Carl Eugen __

Re: [FFmpeg-devel] Refund request of travel costs for GSoC Summit 2017

On 11/20/17, Carl Eugen Hoyos wrote: > 2017-11-18 18:26 GMT+01:00 Thilo Borgmann : > >> some time ago Carl Eugen and me went to the GSoC Summit. >> See review mail for details. >> >> Here comes the refund request for my flight. Costs were 478.82EUR total > > I have sent my invoice over 932,27EUR t

[FFmpeg-devel] [PATCH] avfilter/af_asetnsamples: add missing error check

Signed-off-by: Paul B Mahol --- libavfilter/af_asetnsamples.c | 7 +-- 1 file changed, 5 insertions(+), 2 deletions(-) diff --git a/libavfilter/af_asetnsamples.c b/libavfilter/af_asetnsamples.c index 3c2f66b30a..ecb76e64db 100644 --- a/libavfilter/af_asetnsamples.c +++ b/libavfilter/af_asetn

Re: [FFmpeg-devel] [PATCH 1/1] avformat/dashenc: Added configuration to override HTTP User-Agent

2017-11-20 9:17 GMT+01:00 刘歧 : > >> 在 2017年11月20日，15:59，Jeyapal, Karthick 写道： >> >>> On 11/20/17, 1:01 PM, "刘歧" wrote: 在 2017年11月8日，17:22，Karthick J 写道： +{ "http_user_agent", "override User-Agent field in HTTP header", OFFSET(user_agent), AV_OPT_TYPE_STRING, {.str = NULL}, 0,

Re: [FFmpeg-devel] Refund request of travel costs for GSoC Summit 2017

2017-11-20 14:04 GMT+01:00 Paul B Mahol : > On 11/20/17, Carl Eugen Hoyos wrote: >> 2017-11-18 18:26 GMT+01:00 Thilo Borgmann : >> >>> some time ago Carl Eugen and me went to the GSoC Summit. >>> See review mail for details. >>> >>> Here comes the refund request for my flight. Costs were 478.82EUR

Re: [FFmpeg-devel] Subscribe to ffmpeg-cvslog for patch discussion? Really?

2017-11-19 11:27 GMT+01:00 Jim DeLaHunt : > Looking in the ffmpeg-cvslog archives > , it's clear that diffs of many, > possibly all, commits are sent there. But I see no evidence there of > discussion since 2011 > (though I might have missed something).

[FFmpeg-devel] [PATCH] avformat/avienc: fix fields-per-frame value for interlaced video streams

Fixes ticket #6383. Signed-off-by: Tobias Rapp --- libavformat/avienc.c | 10 -- libavformat/version.h| 2 +- tests/ref/fate/copy-trac2211-avi | 2 +- 3 files changed, 10 insertions(+), 4 deletions(-) diff --git a/libavformat/avienc.c b/libavformat/avienc.c ind

[FFmpeg-devel] [PATCH] avcodec/h264idct_template: Fix integer overflow in ff_h264_idct8_add

Fixes: signed integer overflow: 452986184 - -2113885312 cannot be represented in type 'int' Fixes: 4196/clusterfuzz-testcase-minimized-5580648594014208 Found-by: continuous fuzzing process https://github.com/google/oss-fuzz/tree/master/projects/ffmpeg Signed-off-by: Michael Niedermayer --- lib

Re: [FFmpeg-devel] [PATCH 1/1] avformat/dashenc: Added configuration to override HTTP User-Agent

> 在 2017年11月20日，下午9:03，Carl Eugen Hoyos 写道： > > 2017-11-20 9:17 GMT+01:00 刘歧 : >> > 在 2017年11月20日，15:59，Jeyapal, Karthick 写道： > On 11/20/17, 1:01 PM, "刘歧" wrote: > 在 2017年11月8日，17:22，Karthick J 写道： > +{ "http_user_agent", "override User-Agent field in HTTP header",

Re: [FFmpeg-devel] [PATCH 3/4] avformat/movenc: force colr atom for uncompressed yuv in mov

2017-11-20 2:24 GMT+01:00 James Almer : > On 11/18/2017 11:19 PM, Dave Rice wrote: >> From 41da5e48f8788b85dd7a382030bb2866c506cc03 Mon Sep 17 00:00:00 2001 >> From: Dave Rice >> Date: Sat, 18 Nov 2017 20:31:27 -0500 >> Subject: [PATCH 3/4] avformat/movenc: force colr atom for uncompressed yuv in

Re: [FFmpeg-devel] [PATCH 1/1] avformat/dashenc: Added configuration to override HTTP User-Agent

2017-11-20 15:00 GMT+01:00 Steven Liu : > > >> 在 2017年11月20日，下午9:03，Carl Eugen Hoyos 写道： >> >> 2017-11-20 9:17 GMT+01:00 刘歧 : >>> >> 在 2017年11月20日，15:59，Jeyapal, Karthick 写道： > >> On 11/20/17, 1:01 PM, "刘歧" wrote: >> 在 2017年11月8日，17:22，Karthick J 写道： >> +{ "http_user_age

Re: [FFmpeg-devel] [PATCH] Added Turing codec to ffmpeg

On 11/20/2017 10:35 AM, Matteo Naccari wrote: > +enabled libturing && require_pkg_config libturing libturing turing.h > turing_version && > + { check_cpp_condition turing.h > "TURING_API_VERSION > 1" || > + die "ERROR: libturing re

Re: [FFmpeg-devel] [PATCH 1/1] avformat/dashenc: Added configuration to override HTTP User-Agent

> 在 2017年11月20日，下午10:03，Carl Eugen Hoyos 写道： > > 2017-11-20 15:00 GMT+01:00 Steven Liu : >> >> >>> 在 2017年11月20日，下午9:03，Carl Eugen Hoyos 写道： >>> >>> 2017-11-20 9:17 GMT+01:00 刘歧 : 在 2017年11月20日，15:59，Jeyapal, Karthick 写道： >>> >>> On 11/20/17, 1:01 PM, "刘歧" wrote: >

Re: [FFmpeg-devel] [PATCH 3/4] avformat/movenc: force colr atom for uncompressed yuv in mov

> On Nov 20, 2017, at 9:01 AM, Carl Eugen Hoyos wrote: > > 2017-11-20 2:24 GMT+01:00 James Almer >: >> On 11/18/2017 11:19 PM, Dave Rice wrote: >>> From 41da5e48f8788b85dd7a382030bb2866c506cc03 Mon Sep 17 00:00:00 2001 >>> From: Dave Rice >>> Date: Sat, 18 Nov 2017 20

Re: [FFmpeg-devel] [PATCH 3/4] avformat/movenc: force colr atom for uncompressed yuv in mov

On 11/20/2017 2:50 PM, Dave Rice wrote: > I am hesitant to see write_colr option removed since the guesswork used here > https://github.com/FFmpeg/FFmpeg/blob/8f4702a93f87f9f76563e80f1ae2141a40029d9d/libavformat/movenc.c#L1747-L1775 > would then be applied to all movs of standard HD and SD size a

Re: [FFmpeg-devel] [PATCH 1/1] avformat/dashenc: Added configuration to override HTTP User-Agent

On Mon, Nov 20, 2017 at 3:11 PM, Steven Liu wrote: > > >> 在 2017年11月20日，下午10:03，Carl Eugen Hoyos 写道： >> >> 2017-11-20 15:00 GMT+01:00 Steven Liu : >>> >>> 在 2017年11月20日，下午9:03，Carl Eugen Hoyos 写道： 2017-11-20 9:17 GMT+01:00 刘歧 : > > 在 2017年11月20日，15:59，Jeyapal, Karthick

Re: [FFmpeg-devel] [PATCH 3/4] avformat/movenc: force colr atom for uncompressed yuv in mov

> On Nov 20, 2017, at 9:55 AM, Derek Buitenhuis > wrote: > > On 11/20/2017 2:50 PM, Dave Rice wrote: >> I am hesitant to see write_colr option removed since the guesswork used here >> https://github.com/FFmpeg/FFmpeg/blob/8f4702a93f87f9f76563e80f1ae2141a40029d9d/libavformat/movenc.c#L1747-L177

Re: [FFmpeg-devel] [PATCH 3/3] avcodec: Implement mpeg4 nvdec hwaccel

On Mon, 20 Nov 2017 10:59:52 +0100 Timo Rothenpieler wrote: > > Duplicate. Will remove. > > Can't really tell if the parameters are in any way correct, but I > guess time will tell if some sample comes up that breaks it. > > lgtm with that duplicate removed > Thanks. I've pushed all three.

Re: [FFmpeg-devel] [PATCH 1/6] lavf/movenc: add sdtp (sample dependency) box

On 11/19/2017 02:29 PM, Carl Eugen Hoyos wrote: > 2017-11-19 23:25 GMT+01:00 John Stebbins : >> On 11/19/2017 02:12 PM, Carl Eugen Hoyos wrote: >>> 2017-11-19 21:46 GMT+01:00 John Stebbins : The sdtp is required by the AppleTV 4K in order to play 2160p60 video. +/** + * Flag is used

Re: [FFmpeg-devel] [PATCH] avformat/avienc: fix fields-per-frame value for interlaced video streams

On 20.11.2017 14:47, Tobias Rapp wrote: Fixes ticket #6383. Signed-off-by: Tobias Rapp --- libavformat/avienc.c | 10 -- libavformat/version.h| 2 +- tests/ref/fate/copy-trac2211-avi | 2 +- 3 files changed, 10 insertions(+), 4 deletions(-) [...] Please

Re: [FFmpeg-devel] [FFmpeg-cvslog] tests/fate: addition of test case for hls variant stream creation with master playlist

On 20/11/17 02:06, Vishwanath Dixit wrote: > ffmpeg | branch: master | Vishwanath Dixit | Mon Nov 20 > 10:05:33 2017 +0800| [218ce1f6237b39090015809468f8e933bb09d9b8] | committer: > Steven Liu > > tests/fate: addition of test case for hls variant stream creation with master > playlist > > Rev

Re: [FFmpeg-devel] [PATCH 1/1] avformat/dashenc: Added configuration to override HTTP User-Agent

2017-11-20 23:12 GMT+08:00 Hendrik Leppkes : > On Mon, Nov 20, 2017 at 3:11 PM, Steven Liu wrote: >> >> >>> 在 2017年11月20日，下午10:03，Carl Eugen Hoyos 写道： >>> >>> 2017-11-20 15:00 GMT+01:00 Steven Liu : > 在 2017年11月20日，下午9:03，Carl Eugen Hoyos 写道： > > 2017-11-20 9:17 GMT+01:00 刘

Re: [FFmpeg-devel] [FFmpeg-cvslog] tests/fate: addition of test case for hls variant stream creation with master playlist

On 11/20/2017 12:54 PM, Mark Thompson wrote: > On 20/11/17 02:06, Vishwanath Dixit wrote: >> ffmpeg | branch: master | Vishwanath Dixit | Mon Nov 20 >> 10:05:33 2017 +0800| [218ce1f6237b39090015809468f8e933bb09d9b8] | committer: >> Steven Liu >> >> tests/fate: addition of test case for hls varia

Re: [FFmpeg-devel] [PATCH 1/3] lavu/qsv: remove unused variable

On 20/11/17 09:14, Zhong Li wrote: > suppres build warning of "variable 's' is declared but not used" > > Signed-off-by: Zhong Li > --- > libavutil/hwcontext_qsv.c | 1 - > 1 file changed, 1 deletion(-) > > diff --git a/libavutil/hwcontext_qsv.c b/libavutil/hwcontext_qsv.c > index 8e43420..9b60

Re: [FFmpeg-devel] [PATCH 09/13] compat/cuda: Pass a logging context to load functions

On 18/11/17 23:23, Timo Rothenpieler wrote: > lgtm And applied. Thanks! - Mark ___ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org http://ffmpeg.org/mailman/listinfo/ffmpeg-devel

Re: [FFmpeg-devel] [PATCH] fix bug in af_pan channel coefficient parser

Tobias Rapp (2017-11-20): > Would it be OK to backport the fix into release/3.4? I can do the > cherry-picking but am unsure about the policies and what other older > releases should possibly be adapted, too. I do not know how backporting to releases work either, but I have of course no objection

Re: [FFmpeg-devel] [FFmpeg-cvslog] tests/fate: addition of test case for hls variant stream creation with master playlist

> 在 2017年11月21日，上午12:04，James Almer 写道： > > On 11/20/2017 12:54 PM, Mark Thompson wrote: >> On 20/11/17 02:06, Vishwanath Dixit wrote: >>> ffmpeg | branch: master | Vishwanath Dixit | Mon Nov 20 >>> 10:05:33 2017 +0800| [218ce1f6237b39090015809468f8e933bb09d9b8] | >>> committer: Steven Liu >>

Re: [FFmpeg-devel] [PATCH 3/4] avformat/movenc: force colr atom for uncompressed yuv in mov

On 11/20/2017 3:19 PM, Dave Rice wrote: > TN2162 requires a colr atom for uncompressed yuv (including v210, v308, v408, > etc) in mov, so I'd prefer to write it in this case. Note that the colr atom > provides an option for unspecified for each of the color values, so there's a > method to write

Re: [FFmpeg-devel] [PATCH] lavf/mov.c: Guess video codec delay based on PTS while parsing MOV header.

On Sun, Nov 19, 2017 at 1:17 AM, Michael Niedermayer wrote: > On Sat, Nov 18, 2017 at 11:12:17AM -0800, Sasi Inguva wrote: > > Signed-off-by: Sasi Inguva > > --- > > libavformat/mov.c| 54 ++ > ++ > > tests/fate/mov.mak | 5

[FFmpeg-devel] [PATCH] lavf/mov.c: Guess video codec delay based on PTS while parsing MOV header.

Signed-off-by: Sasi Inguva --- libavformat/mov.c| 55 tests/fate/mov.mak | 7 + tests/ref/fate/mov-guess-delay-1 | 3 +++ tests/ref/fate/mov-guess-delay-2 | 3 +++ tests/ref/fate/mov-guess-delay-3 | 3 +++ 5 files chan

Re: [FFmpeg-devel] Refund request of travel costs for GSoC Summit 2017

Hi, On Mon, Nov 20, 2017 at 8:04 AM, Paul B Mahol wrote: > On 11/20/17, Carl Eugen Hoyos wrote: > > 2017-11-18 18:26 GMT+01:00 Thilo Borgmann : > > > >> some time ago Carl Eugen and me went to the GSoC Summit. > >> See review mail for details. > >> > >> Here comes the refund request for my flig

Re: [FFmpeg-devel] [PATCH] lavf/mov.c: Guess video codec delay based on PTS while parsing MOV header.

Attaching the fate sample On Mon, Nov 20, 2017 at 8:28 AM, Sasi Inguva wrote: > Signed-off-by: Sasi Inguva > --- > libavformat/mov.c| 55 ++ > ++ > tests/fate/mov.mak | 7 + > tests/ref/fate/mov-guess-delay-1 | 3 +++ > te

Re: [FFmpeg-devel] [PATCH 3/4] avformat/movenc: force colr atom for uncompressed yuv in mov

> On Nov 20, 2017, at 11:22 AM, Derek Buitenhuis > wrote: > > On 11/20/2017 3:19 PM, Dave Rice wrote: >> TN2162 requires a colr atom for uncompressed yuv (including v210, v308, >> v408, etc) in mov, so I'd prefer to write it in this case. Note that the >> colr atom provides an option for unsp

Re: [FFmpeg-devel] [PATCH 3/4] avformat/movenc: force colr atom for uncompressed yuv in mov

On 11/20/2017 4:45 PM, Dave Rice wrote: > What do you propose as the default for guessed_hacky_crap? Also are there > supporters for the need of a guessed_hacky_crap optio? Is there precedence in > ffmpeg to enable/disable guesswork via a user option? I personally would never use the guesswork s

[FFmpeg-devel] [PATCH] tools/target_dec_fuzzer: Limit error concealment to max_pixels

Error concealment is time consuming, limiting it in a pixel dependant way allows the decoder to be fuzzed with more frames quicker Signed-off-by: Michael Niedermayer --- tools/target_dec_fuzzer.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/tools/target_dec_fuzzer.c b/tool

Re: [FFmpeg-devel] [PATCH] configure: fix module dependencies on zlib

On Sun, Nov 19, 2017 at 04:00:11PM -0300, James Almer wrote: > select should not be used with external libraries. It's mean to soft > enable internal modules/features. > > Signed-off-by: James Almer > --- > configure | 48 ++-- > 1 file changed, 26 ins

Re: [FFmpeg-devel] [PATCH] configure: fix module dependencies on zlib

On 11/20/2017 2:34 PM, Michael Niedermayer wrote: > On Sun, Nov 19, 2017 at 04:00:11PM -0300, James Almer wrote: >> select should not be used with external libraries. It's mean to soft >> enable internal modules/features. >> >> Signed-off-by: James Almer >> --- >> configure | 48 +

Re: [FFmpeg-devel] [PATCH] tools/target_dec_fuzzer: Limit error concealment to max_pixels

> > On Mon, 20 Nov 2017 at 17:26 Michael Niedermayer > wrote: > Error concealment is time consuming, limiting it in a pixel dependant way > allows > the decoder to be fuzzed with more frames quicker > How is this a reasonable fix considering you have submitted numerous patches to fix "long-runnin

[FFmpeg-devel] [PATCH 6/6] ffplay: use AV_PKT_FLAG_DISPOSABLE in frame drop logic

--- fftools/ffplay.c | 21 - 1 file changed, 16 insertions(+), 5 deletions(-) diff --git a/fftools/ffplay.c b/fftools/ffplay.c index 10a917194d..f1e0522528 100644 --- a/fftools/ffplay.c +++ b/fftools/ffplay.c @@ -198,6 +198,8 @@ typedef struct Decoder { int64_t next_pts;

[FFmpeg-devel] ffplay: use AV_PKT_FLAG_DISPOSABLE in frame drop logic

This update to the patch incorporates Marton's suggestion to eliminate the effects of buffering on the selection of frames to drop. It ties the decision to start dropping disposable frames to the "early" frame drop logic that happens immediately after avcodec_receive_frame. And disposable frames

Re: [FFmpeg-devel] [PATCH 11/11] avformat/wavdecuse FFABS to instead of fabs

> FFABS is slower than fabs. Not generally. That is highly dependent on compiler/flags/architecture. In some cases, FFABS may actually be faster, in others maybe not. The expected result would be on-par performance. ___ ffmpeg-devel mailing list ffmpeg

Re: [FFmpeg-devel] [PATCH 1/4] configure: require libvpx-1.4.0 for vp[89] support

On Sun, Nov 19, 2017 at 6:03 AM, James Almer wrote: > On 11/17/2017 10:50 PM, James Zern wrote: >> this will simplify libvpxenc/dec.c and ensure more stable versions of >> the codecs are present. >> >> Signed-off-by: James Zern >> --- >> configure | 20 ++-- >> 1 file changed, 10

Re: [FFmpeg-devel] [PATCH 2/4] libvpx: remove pre-1.4.0 checks

On Sun, Nov 19, 2017 at 5:57 AM, James Almer wrote: > On 11/17/2017 10:50 PM, James Zern wrote: >> Signed-off-by: James Zern >> --- >> libavcodec/libvpx.c | 12 +--- >> 1 file changed, 1 insertion(+), 11 deletions(-) >> >> diff --git a/libavcodec/libvpx.c b/libavcodec/libvpx.c >> index 1

Re: [FFmpeg-devel] [PATCH] configure: fix module dependencies on zlib

On Mon, Nov 20, 2017 at 02:58:58PM -0300, James Almer wrote: > On 11/20/2017 2:34 PM, Michael Niedermayer wrote: > > On Sun, Nov 19, 2017 at 04:00:11PM -0300, James Almer wrote: > >> select should not be used with external libraries. It's mean to soft > >> enable internal modules/features. > >> > >

Re: [FFmpeg-devel] [PATCH] tools/target_dec_fuzzer: Limit error concealment to max_pixels

On Mon, Nov 20, 2017 at 06:30:28PM +, Kieran Kunhya wrote: > > > > On Mon, 20 Nov 2017 at 17:26 Michael Niedermayer > > wrote: > > Error concealment is time consuming, limiting it in a pixel dependant way > > allows > > the decoder to be fuzzed with more frames quicker > > > > How is this a r

Re: [FFmpeg-devel] [PATCH 1/4] configure: require libvpx-1.4.0 for vp[89] support

On Mon, Nov 20, 2017 at 11:17 AM, James Zern wrote: > On Sun, Nov 19, 2017 at 6:03 AM, James Almer wrote: >> On 11/17/2017 10:50 PM, James Zern wrote: >>> this will simplify libvpxenc/dec.c and ensure more stable versions of >>> the codecs are present. >>> >>> Signed-off-by: James Zern >>> --- >

Re: [FFmpeg-devel] [PATCH 1/4] configure: require libvpx-1.4.0 for vp[89] support

On 11/20/2017 4:38 PM, James Zern wrote: > On Mon, Nov 20, 2017 at 11:17 AM, James Zern wrote: >> On Sun, Nov 19, 2017 at 6:03 AM, James Almer wrote: >>> On 11/17/2017 10:50 PM, James Zern wrote: this will simplify libvpxenc/dec.c and ensure more stable versions of the codecs are presen

Re: [FFmpeg-devel] [mpeg4video] Fix undefined shift on assumed 8-bit input.

On Fri, Nov 17, 2017 at 6:22 PM, Michael Niedermayer wrote: > > Not sure what is best but > throwing part of the version silently away is not correct > most likely erroring out and asking for a sample video to be uploaded > would make sense if such a file is encountered I'm using the same workar

Re: [FFmpeg-devel] [mpeg4video] Fix undefined shift on assumed 8-bit input.

Derp, that's the wrong comment for this; I meant that for another thread. Returning an error for optional user data seems a bit harsh, but if that's what you want, I'm happy to change it. Please let me know if this is the route you want to take. - dale On Mon, Nov 20, 2017 at 11:55 AM, Dale Curti

[FFmpeg-devel] [PATCH] avfilter: add mix filter

Signed-off-by: Paul B Mahol --- doc/filters.texi | 28 + libavfilter/Makefile | 1 + libavfilter/allfilters.c | 1 + libavfilter/vf_mix.c | 283 +++ 4 files changed, 313 insertions(+) create mode 100644 libavfilter/vf_mix.c di

[FFmpeg-devel] [PATCH] avformat/mov: Propagate errors in mov_switch_root.

Signed-off-by: Jacob Trimble --- libavformat/mov.c | 7 +-- 1 file changed, 5 insertions(+), 2 deletions(-) diff --git a/libavformat/mov.c b/libavformat/mov.c index 0b8f8ad24b..b6cdf3a52a 100644 --- a/libavformat/mov.c +++ b/libavformat/mov.c @@ -6753,6 +6753,7 @@ static int should_retry(AVI

Re: [FFmpeg-devel] [avformat] Prevent undefined shift with wrap_bits > 63.

On Sat, Nov 18, 2017 at 2:44 AM, Michael Niedermayer wrote: > > this would skip the code for wrap_bits >= 63, this does not look > correct > Why do you think that's incorrect? The max int64_t value is 1 << 63 and 2 << 63 == 1 << 64 ? - dale ___ ffmpeg-

[FFmpeg-devel] [ogg] Close ogg stream upon error when using AV_EF_EXPLODE.

Without this there can be multiple memory leaks with unknown ogg streams. - dale From 4fc9d288c50891a9740facc549c895da066af71b Mon Sep 17 00:00:00 2001 From: Dale Curtis Date: Mon, 20 Nov 2017 12:07:57 -0800 Subject: [PATCH] [ogg] Close ogg stream upon error when using AV_EF_EXPLODE. Without thi

Re: [FFmpeg-devel] Subscribe to ffmpeg-cvslog for patch discussion? Really?

On 2017-11-20 05:18, Carl Eugen Hoyos wrote: 2017-11-19 11:27 GMT+01:00 Jim DeLaHunt : Looking in the ffmpeg-cvslog archives , it's clear that diffs of many, possibly all, commits are sent there. But I see no evidence there of discussion since 2011 (th

Re: [FFmpeg-devel] Subscribe to ffmpeg-cvslog for patch discussion? Really?

On 11/20/17, Jim DeLaHunt wrote: > On 2017-11-20 05:18, Carl Eugen Hoyos wrote: >> 2017-11-19 11:27 GMT+01:00 Jim DeLaHunt : >>> Looking in the ffmpeg-cvslog archives >>> , it's clear that diffs of >>> many, >>> possibly all, commits are sent there. But

[FFmpeg-devel] [PATCH] avcodec/mlpdsp: Fix signed integer overflow, 2nd try

The outputted bits should match what is used in the lossless check Fixes: runtime error: signed integer overflow: -538697856 * 256 cannot be represented in type 'int' Fixes: 4326/clusterfuzz-testcase-minimized-5689449645080576 Found-by: continuous fuzzing process https://github.com/google/oss-f

Re: [FFmpeg-devel] [PATCH 1/2] avcodec/hevcdsp_template: Fix invalid shift in put_hevc_epel_bi_w_v()

On Sat, Nov 18, 2017 at 01:33:17AM +0100, Michael Niedermayer wrote: > Fixes: runtime error: left shift of negative value -255 > Fixes: 4037/clusterfuzz-testcase-minimized-5290998163832832 > > Found-by: continuous fuzzing process > https://github.com/google/oss-fuzz/tree/master/projects/ffmpeg >

Re: [FFmpeg-devel] [PATCH 11/13] vp8: Add hwaccel hooks

On 2017-11-18 10:47, Mark Thompson wrote: Also adds some extra fields to the main context structure that may be needed by a hwaccel decoder. The current behaviour of the WebP decoder is maintained by adding an additional field to the VP8 decoder private context to indicate that it is actually be

Re: [FFmpeg-devel] [mpeg4video] Fix undefined shift on assumed 8-bit input.

On Mon, Nov 20, 2017 at 12:01:11PM -0800, Dale Curtis wrote: > Derp, that's the wrong comment for this; I meant that for another thread. > Returning an error for optional user data seems a bit harsh, but if that's > what you want, I'm happy to change it. Please let me know if this is the > route yo

[FFmpeg-devel] [PATCH] avformat: rename AVOutputFormat.deinit to AVOutputFormat.close

This is more in line with other callbacks, like AVCodec.close and AVInputFormat.read_close Signed-off-by: James Almer --- libavformat/avformat.h | 4 ++-- libavformat/dashenc.c | 6 +++--- libavformat/fifo.c | 4 ++-- libavformat/movenc.c| 18 +- libavformat/mpegts

Re: [FFmpeg-devel] [avformat] Prevent undefined shift with wrap_bits > 63.

On Mon, Nov 20, 2017 at 12:05:05PM -0800, Dale Curtis wrote: > On Sat, Nov 18, 2017 at 2:44 AM, Michael Niedermayer > wrote: > > > > this would skip the code for wrap_bits >= 63, this does not look > > correct > > > > Why do you think that's incorrect? The max int64_t value is 1 << 63 and 2 > <<

Re: [FFmpeg-devel] [PATCH] avformat: rename AVOutputFormat.deinit to AVOutputFormat.close

James Almer (2017-11-20): > This is more in line with other callbacks, like AVCodec.close and > AVInputFormat.read_close Unless I am mistaken, unlike the "close" callbacks, this one is always called, including when init fails. Regards, -- Nicolas George signature.asc Description: Digital si

[FFmpeg-devel] [PATCH 1/2] avocdec/ttaenc: buffer packets directly

This is a bit more robust in case of OOM. Signed-off-by: James Almer --- libavformat/ttaenc.c | 42 +- 1 file changed, 33 insertions(+), 9 deletions(-) diff --git a/libavformat/ttaenc.c b/libavformat/ttaenc.c index fdce1e3dc1..b05705b6b6 100644 --- a/liba

[FFmpeg-devel] [PATCH 2/2] avformat/ttaenc: add tta_init()

Signed-off-by: James Almer --- libavformat/ttaenc.c | 32 ++-- 1 file changed, 22 insertions(+), 10 deletions(-) diff --git a/libavformat/ttaenc.c b/libavformat/ttaenc.c index b05705b6b6..add15873d0 100644 --- a/libavformat/ttaenc.c +++ b/libavformat/ttaenc.c @@ -35,1

Re: [FFmpeg-devel] [PATCH] libavcodec/mpeg12dec: Use atomic addition for value updated in multiple threads.

On Sun, Nov 19, 2017 at 03:23:47PM -0800, Nick Lewycky wrote: [...] > error_resilience.c | 22 -- > error_resilience.h |3 ++- > h264_slice.c |6 -- > mpeg12dec.c| 11 ++- > mpegvideo_enc.c|5 - > 5 files changed, 28 insertions(

Re: [FFmpeg-devel] [PATCH] avformat/avio: remove must_flush from AVIOContext

On Sun, 19 Nov 2017, Marton Balint wrote: On Wed, 15 Nov 2017, Marton Balint wrote: On Wed, 15 Nov 2017, Nicolas George wrote: Le quintidi 25 brumaire, an CCXXVI, Marton Balint a écrit : It is unused. Signed-off-by: Marton Balint --- libavformat/avio.h | 1 - 1 file changed, 1 delet

Re: [FFmpeg-devel] [PATCH] libavcodec/mpeg12dec: Use atomic addition for value updated in multiple threads.

On Sun, Nov 19, 2017 at 03:23:47PM -0800, Nick Lewycky wrote: > Sorry-- what should I do now? Wait for another patch to go in first then > rebase on top of it? Attempt to migrate error_count to C11 atomics myself? > If I'm migrating, is there a primer on how ffmpeg uses C11 atomics? For > instance,

Re: [FFmpeg-devel] libavcodec/huffyuvdsp(enc) : add avx2 version for add(diff)_int16

2017-11-04 19:31 GMT+01:00 Martin Vignali : > > > 2017-10-22 0:26 GMT+02:00 Martin Vignali : > >> Hello, >> >> In attach patch to add avx2 version for huffyuv dsp and huffyuvdsp enc >> for add_int16 and diff_int16 func >> >> Check asm result for add_int16 (Kaby Lake, os 10.12) >> add_int16_128_c:

Re: [FFmpeg-devel] libavcodec/utvideodsp : add avx2 version

2017-11-04 19:33 GMT+01:00 Martin Vignali : > > > 2017-10-25 21:53 GMT+02:00 Martin Vignali : > >> >> >> 2017-10-22 14:05 GMT+02:00 Martin Vignali : >> >>> Hello, >>> >>> In attach patch to add AVX2 version for the utvideodsp >>> >>> Checkasm result (Kaby Lake, os 10.12) >>> restore_rgb_planes_c:

Re: [FFmpeg-devel] [PATCH 01/13] lavc: Add codec metadata to indicate hardware support

On 2017-11-18 10:47, Mark Thompson wrote: --- doc/APIchanges | 3 +++ libavcodec/avcodec.h | 74 libavcodec/hwaccel.h | 18 + libavcodec/utils.c | 12 + 4 files changed, 107 insertions(+) diff --git a/doc/APIcha

Re: [FFmpeg-devel] [PATCH] avformat: rename AVOutputFormat.deinit to AVOutputFormat.close

On 11/20/2017 5:45 PM, Nicolas George wrote: > James Almer (2017-11-20): >> This is more in line with other callbacks, like AVCodec.close and >> AVInputFormat.read_close > > Unless I am mistaken, unlike the "close" callbacks, this one is always > called, including when init fails. > > Regards, T

Re: [FFmpeg-devel] libavcodec/hapdec : add support for hapqa decoding

2017-11-13 20:06 GMT+01:00 Martin Vignali : > > > 2017-10-21 21:37 GMT+02:00 Carl Eugen Hoyos : > >> 2017-10-21 21:32 GMT+02:00 Martin Vignali : >> > 2017-10-21 21:23 GMT+02:00 Carl Eugen Hoyos : >> > >> >> 2017-10-21 19:35 GMT+02:00 Martin Vignali : >> >> > Hello, >> >> > >> >> > I split the prev

  1   2   >