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

2017-11-19 Thread Jun Zhao
On 2017/11/20 15:43, Jun Zhao wrote: > > On 2017/11/20 15:35, 刘歧 wrote: >>> 在 2017年11月20日,15:23,Jun Zhao 写道: >>> >>> >>> >>> On 2017/11/20 14:45, Steven Liu wrote: fix ticket id: #6846 Signed-off-by: Steven Liu --- libavformat/avio.c | 8 1 file changed, 8

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

2017-11-19 Thread 刘歧
> 在 2017年11月20日,15:43,Jun Zhao 写道: > > > > On 2017/11/20 15:35, 刘歧 wrote: >> >>> 在 2017年11月20日,15:23,Jun Zhao 写道: >>> >>> >>> >>> On 2017/11/20 14:45, Steven Liu wrote: fix ticket id: #6846 Signed-off-by: Steven Liu --- libavformat/avio.c | 8 1 fi

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

2017-11-19 Thread Tobias Rapp
On 19.11.2017 17:39, Nicolas George wrote: L'octidi 28 brumaire, an CCXXVI, Michael Roitzsch a écrit : Hi FFmpeg team, I was using af_pan to subtract one audio file from another. I first used an amerge filter and then configured the pan filter with a channel formula like c0=c0-c2|c1=c1-c3. H

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

2017-11-19 Thread Jun Zhao
On 2017/11/20 15:35, 刘歧 wrote: > >> 在 2017年11月20日,15:23,Jun Zhao 写道: >> >> >> >> On 2017/11/20 14:45, Steven Liu wrote: >>> fix ticket id: #6846 >>> >>> Signed-off-by: Steven Liu >>> --- >>> libavformat/avio.c | 8 >>> 1 file changed, 8 insertions(+) >>> >>> diff --git a/libavformat/avi

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

2017-11-19 Thread 刘歧
> 在 2017年11月20日,15:23,Jun Zhao 写道: > > > > On 2017/11/20 14:45, Steven Liu wrote: >> fix ticket id: #6846 >> >> Signed-off-by: Steven Liu >> --- >> libavformat/avio.c | 8 >> 1 file changed, 8 insertions(+) >> >> diff --git a/libavformat/avio.c b/libavformat/avio.c >> index 4dc4683

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

2017-11-19 Thread 刘歧
> 在 2017年11月8日,17:22,Karthick J 写道: > > --- > doc/muxers.texi | 2 ++ > libavformat/dashenc.c | 22 +++--- > 2 files changed, 21 insertions(+), 3 deletions(-) > > diff --git a/doc/muxers.texi b/doc/muxers.texi > index 91bbe67..412fede 100644 > --- a/doc/muxers.texi > +++ b/

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

2017-11-19 Thread Jun Zhao
On 2017/11/20 14:45, Steven Liu wrote: > fix ticket id: #6846 > > Signed-off-by: Steven Liu > --- > libavformat/avio.c | 8 > 1 file changed, 8 insertions(+) > > diff --git a/libavformat/avio.c b/libavformat/avio.c > index 4dc468350c..e719326660 100644 > --- a/libavformat/avio.c > +++

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

2017-11-19 Thread Steven Liu
fix ticket id: #6846 Signed-off-by: Steven Liu --- libavformat/avio.c | 8 1 file changed, 8 insertions(+) diff --git a/libavformat/avio.c b/libavformat/avio.c index 4dc468350c..e719326660 100644 --- a/libavformat/avio.c +++ b/libavformat/avio.c @@ -625,6 +625,8 @@ int64_t ffurl_size(U

Re: [FFmpeg-devel] [PATCH 3/3] avformat/opensrt: add Haivision Open SRT protocol

2017-11-19 Thread nablet developer
> > Thanks for explaining. I think it is not the best decision. > > The reason the socket API resembles TCP is because all the sockets API > resemble each other, since they are based on the old BSD socket API. And > the protocol handlers of libavformat too. > > Therefore, I think all the trampo

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

2017-11-19 Thread Jeyapal, Karthick
Looks like nobody has any objection for the below patch. Can somebody please push this. Regards, Karthick On 11/8/17, 2:53 PM, "Karthick J" wrote: --- doc/muxers.texi | 2 ++ libavformat/dashenc.c | 22 +++--- 2 files changed, 21 insertions(+), 3 deletions(-) diff --gi

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

2017-11-19 Thread Gyan Doshi
On 11/20/2017 5:04 AM, Thilo Borgmann wrote: min_start n_begin start_delay ? Regards, Gyan ___ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org http://ffmpeg.org/mailman/listinfo/ffmpeg-devel

Re: [FFmpeg-devel] [PATCH] avformat/hlsenc: creation of hls variant streams with master playlist in a single hlsenc instance

2017-11-19 Thread Dixit, Vishwanath
>On 11/20/17, 7:38 AM, "刘歧" wrote: >pushed! Thank you!! ( ___ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org http://ffmpeg.org/mailman/listinfo/ffmpeg-devel

Re: [FFmpeg-devel] [PATCH 09/11] avformat/swfenc: use FFABS to instead of abs

2017-11-19 Thread Steven Liu
2017-11-16 23:00 GMT+08:00 Michael Niedermayer : > On Wed, Nov 15, 2017 at 03:45:03PM +0800, Steven Liu wrote: >> Signed-off-by: Steven Liu >> --- >> libavformat/swfenc.c | 2 +- >> 1 file changed, 1 insertion(+), 1 deletion(-) > > LGTM > Pushed Thx > thx > > [...] > -- > Michael GnuPG fing

Re: [FFmpeg-devel] [PATCH 05/11] avformat/mp3dec: use FFABS to instead of abs

2017-11-19 Thread Steven Liu
2017-11-16 22:58 GMT+08:00 Michael Niedermayer : > On Wed, Nov 15, 2017 at 03:44:17PM +0800, Steven Liu wrote: >> Signed-off-by: Steven Liu >> --- >> libavformat/mp3dec.c | 4 ++-- >> 1 file changed, 2 insertions(+), 2 deletions(-) > > LGTM > Pushed Thx > [...] > -- > Michael GnuPG fingerpri

Re: [FFmpeg-devel] [PATCH 07/11] avformat/mxf: use FFABS to instead of abs

2017-11-19 Thread Steven Liu
2017-11-16 22:59 GMT+08:00 Michael Niedermayer : > On Wed, Nov 15, 2017 at 03:44:39PM +0800, Steven Liu wrote: >> Signed-off-by: Steven Liu >> --- >> libavformat/mxf.c | 2 +- >> 1 file changed, 1 insertion(+), 1 deletion(-) > > LGTM > Pushed Thx > thx > > [...] > -- > Michael GnuPG fingerpr

Re: [FFmpeg-devel] [PATCH] avformat/hlsenc: creation of hls variant streams with master playlist in a single hlsenc instance

2017-11-19 Thread 刘歧
> 在 2017年11月16日,17:01,Dixit, Vishwanath 写道: > > > >> On 11/16/17, 12:09 PM, "刘歧" wrote: >> >> FATE test passed: Ubuntu, OS X, qemu+MIPS Linux, wine MingW, qemu+ARM >> Linux, Thanks. >> >> +if (p) { >> +strcpy(extension, p); >> What about use av_strlcpy ? >> >> +

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

2017-11-19 Thread Michael Niedermayer
On Sun, Nov 19, 2017 at 10:24:41PM -0300, James Almer wrote: > 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

[FFmpeg-devel] [PATCH V3] examples/vaapi_encode: Add a VA-API encode example.

2017-11-19 Thread Jun Zhao
V3: Use av_hwframe_transfer_date load YUV to HW surface From c0edef97f4f123e74e62396b45db01236ae6477d Mon Sep 17 00:00:00 2001 From: Jun Zhao Date: Mon, 6 Nov 2017 14:45:27 +0800 Subject: [PATCH V3] examples/vaapi_encode: Add a VA-API encode example. add a VA-API encode example, only support NV12

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

2017-11-19 Thread 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 > mov > MIME-Version: 1.0 > Content-Type: tex

Re: [FFmpeg-devel] [PATCH] libavformat/mov:the "stco" box parse no need return eof.

2017-11-19 Thread Michael Niedermayer
On Sun, Nov 19, 2017 at 10:00:29PM +0800, tiejun.peng wrote: > if return eof, will cause to play the whole mp4 file fail. > For example, currently the whole file will play fail > when a few bytes missed in the "stco" box > and the "stco" box is the last box of MP4 file, > Actually the almost media

Re: [FFmpeg-devel] [PATCH 2/4] avformat/movenc: write clap atom for uncompressed yuv in mov

2017-11-19 Thread Michael Niedermayer
On Sat, Nov 18, 2017 at 09:19:23PM -0500, Dave Rice wrote: > From e9079167c199791e652fe9cd3550333b819a8a9a Mon Sep 17 00:00:00 2001 > From: Dave Rice > Date: Thu, 16 Nov 2017 11:29:06 -0500 > Subject: [PATCH 2/4] avformat/movenc: write clap atom for uncompressed yuv in > mov > > fixes 6145 > ---

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

2017-11-19 Thread Michael Niedermayer
On Sun, Nov 19, 2017 at 10:05:56AM +0100, Reto Kromer wrote: > Dave Rice wrote: > > >As required by Apple’s TN2162. > >--- > > libavformat/movenc.c | 2 +- > > 1 file changed, 1 insertion(+), 1 deletion(-) > > LGTM will apply thanks [...] -- Michael GnuPG fingerprint: 9FF2128B147EF6730BADF

Re: [FFmpeg-devel] [PATCH 1/4] avformat/movenc: correct ImageDescription for uncompressed ycbcr

2017-11-19 Thread Michael Niedermayer
On Sat, Nov 18, 2017 at 09:19:17PM -0500, Dave Rice wrote: > This patch set updates movenc to write uncompressed ycbcr in mov following > requirements in Apple’s TN2162, > https://developer.apple.com/library/content/technotes/tn2162/_index.html. > Thanks to Carl and Michael for comments on an ea

Re: [FFmpeg-devel] [PATCH 2/3] checkasm/jpeg2000dsp: add test for ict_float

2017-11-19 Thread Michael Niedermayer
On Sun, Nov 19, 2017 at 12:39:36AM -0300, James Almer wrote: > Signed-off-by: James Almer > --- > tests/checkasm/jpeg2000dsp.c | 33 + > 1 file changed, 33 insertions(+) LGTM thx [...] -- Michael GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB

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

2017-11-19 Thread Jun Zhao
From 01a66eb4559fb3fec0765ee03417a65ef5e06fe1 Mon Sep 17 00:00:00 2001 From: Jun Zhao Date: Sat, 18 Nov 2017 13:24:24 +0800 Subject: [PATCH] ffmpeg: add return value check to supress the build warning. add return value check to supress the build warning message like "warning: ignoring return val

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

2017-11-19 Thread Jun Zhao
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. Signed-off-by: Jun Zhao --- libavutil/hwcon

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

2017-11-19 Thread Jun Zhao
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 unimplemented when query surface attributes, we just ignore

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

2017-11-19 Thread Carl Eugen Hoyos
2017-11-19 23:35 GMT+01:00 Carl Eugen Hoyos : > 2017-11-19 20:52 GMT+01:00 Philip Langdale : > >> gmc_enabled: This seems to map to mc_sel being non-zero, but I also >> have no samples where that is true. > > Iirc, global motion compensation is not supported by hardware decoders. Or m

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

2017-11-19 Thread Philip Langdale
On Sun, 19 Nov 2017 11:52:28 -0800 Philip Langdale wrote: > This was predictably nightmarish, given how ridiculous mpeg4 is. > I had to stare at the cuvid parser output for a long time to work > out what each field was supposed to be, and even then, I still don't > fully understand some of them,

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

2017-11-19 Thread Philip Langdale
On Sun, 19 Nov 2017 23:35:24 +0100 Carl Eugen Hoyos wrote: > 2017-11-19 20:52 GMT+01:00 Philip Langdale : > > > gmc_enabled: This seems to map to mc_sel being non-zero, but I also > > have no samples where that is true. > > Iirc, global motion compensation is not supported by har

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

2017-11-19 Thread Thilo Borgmann
> Am 19.11.2017 um 23:25 schrieb Nicolas George : > > Thilo Borgmann (2017-11-19): >> Based on Dave's example I'd say the queue parameter defines the >> minimum of available tiles before output is generated. >> >> 'queue' -> 'min' >> 'queue' -> 'min_tiles' >> 'queue' -> 'min_available' >> >>

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

2017-11-19 Thread Nick Lewycky
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, given an atomic_int, should I assign to it with equality, atomic_stor

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

2017-11-19 Thread Carl Eugen Hoyos
2017-11-19 20:52 GMT+01:00 Philip Langdale : > gmc_enabled: This seems to map to mc_sel being non-zero, but I also > have no samples where that is true. Iirc, global motion compensation is not supported by hardware decoders. Carl Eugen ___

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

2017-11-19 Thread Carl Eugen Hoyos
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 to indicate packets that contain frames that can >>> +

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

2017-11-19 Thread 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 to indicate packets that contain frames that can >> + * be discarded by the decoder. I.e. Non-reference

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

2017-11-19 Thread Nicolas George
Thilo Borgmann (2017-11-19): > Based on Dave's example I'd say the queue parameter defines the > minimum of available tiles before output is generated. > > 'queue' -> 'min' > 'queue' -> 'min_tiles' > 'queue' -> 'min_available' > > I'd suggest something like that. "min" would imply a global min

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

2017-11-19 Thread John Stebbins
On 11/19/2017 01:28 PM, Marton Balint wrote: > On Sun, 19 Nov 2017, John Stebbins wrote: > >> --- >> fftools/ffplay.c | 10 ++ >> 1 file changed, 10 insertions(+) >> >> diff --git a/fftools/ffplay.c b/fftools/ffplay.c >> index 10a917194d..97555d5047 100644 >> --- a/fftools/ffplay.c >> +++ b/

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

2017-11-19 Thread Carl Eugen Hoyos
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 to indicate packets that contain frames that can > + * be discarded by the decoder. I.e. Non-reference frames. > + */ > +#define AV_PKT_FLAG_DISPOSABLE 0x0

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

2017-11-19 Thread Thilo Borgmann
> Am 19.11.2017 um 21:22 schrieb Nicolas George : > > Dave Rice (2017-11-19): >> IMHO, ‘queue' is a good name for what it does. > > Based on your explanation, I would say not at all. It rather seems to > say the opposite. I would suggest something like "init_padding". Based on Dave's example I

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

2017-11-19 Thread Marton Balint
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 deletion(-) Breaks ABI by moving all the fields b

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

2017-11-19 Thread Marton Balint
On Sun, 19 Nov 2017, John Stebbins wrote: --- fftools/ffplay.c | 10 ++ 1 file changed, 10 insertions(+) diff --git a/fftools/ffplay.c b/fftools/ffplay.c index 10a917194d..97555d5047 100644 --- a/fftools/ffplay.c +++ b/fftools/ffplay.c @@ -257,6 +257,7 @@ typedef struct VideoState {

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

2017-11-19 Thread James Almer
On 11/19/2017 5:33 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 +

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

2017-11-19 Thread John Stebbins
--- fftools/ffplay.c | 10 ++ 1 file changed, 10 insertions(+) diff --git a/fftools/ffplay.c b/fftools/ffplay.c index 10a917194d..97555d5047 100644 --- a/fftools/ffplay.c +++ b/fftools/ffplay.c @@ -257,6 +257,7 @@ typedef struct VideoState { struct SwrContext *swr_ctx; int frame

[FFmpeg-devel] [PATCH 5/6] lavf/mov: fix disposable flags in mov_fix_index

2017-11-19 Thread John Stebbins
Merges ctts_data into mov_index_entries. This allows a considerable amount of code simplification. --- libavformat/isom.h | 1 + libavformat/mov.c | 254 +++-- 2 files changed, 93 insertions(+), 162 deletions(-) diff --git a/libavformat/isom.h b

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

2017-11-19 Thread John Stebbins
--- fftools/ffplay.c | 10 ++ 1 file changed, 10 insertions(+) diff --git a/fftools/ffplay.c b/fftools/ffplay.c index 10a917194d..97555d5047 100644 --- a/fftools/ffplay.c +++ b/fftools/ffplay.c @@ -257,6 +257,7 @@ typedef struct VideoState { struct SwrContext *swr_ctx; int frame

[FFmpeg-devel] [PATCH 5/6] lavf/mov: fix disposable flags in mov_fix_index

2017-11-19 Thread John Stebbins
Merges ctts_data into mov_index_entries. This allows a considerable amount of code simplification. --- libavformat/isom.h | 1 + libavformat/mov.c | 254 +++-- 2 files changed, 93 insertions(+), 162 deletions(-) diff --git a/libavformat/isom.h b

Re: [FFmpeg-devel] [PATCH 4/4] fate: add v210 mov test

2017-11-19 Thread Michael Niedermayer
On Sat, Nov 18, 2017 at 09:19:29PM -0500, Dave Rice wrote: > From fe9e3aa13ecf3b4cb81f279696bcc21602662e7a Mon Sep 17 00:00:00 2001 > From: Dave Rice > Date: Sat, 18 Nov 2017 20:32:33 -0500 > Subject: [PATCH 4/4] fate: add v210 mov test > > --- > tests/fate/vcodec.mak | 4 > tes

[FFmpeg-devel] [PATCH 4/6] lavf/mov: set AV_PKT_FLAG_DISPOSABLE from sdtp box

2017-11-19 Thread John Stebbins
--- libavformat/isom.h | 10 ++ libavformat/mov.c | 92 ++--- libavformat/utils.c | 3 +- 3 files changed, 99 insertions(+), 6 deletions(-) diff --git a/libavformat/isom.h b/libavformat/isom.h index 65676fb0f5..646b31ccc7 100644 --- a/libavf

Re: [FFmpeg-devel] RPM library deployments

2017-11-19 Thread Moritz Barsnick
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? > > It's not from us, a third-party makes that, but I'm not sure who. FFm

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

2017-11-19 Thread John Stebbins
The sdtp is required by the AppleTV 4K in order to play 2160p60 video. --- libavcodec/avcodec.h | 6 ++ libavformat/isom.h | 5 + libavformat/movenc.c | 30 ++ libavformat/movenc.h | 2 ++ 4 files changed, 43 insertions(+) diff --git a/libavcodec/avcodec.h

[FFmpeg-devel] [PATCH 2/6] lavc/libx264: mark disposable frames

2017-11-19 Thread John Stebbins
Used by movenc to fill sdtp box --- libavcodec/libx264.c | 5 + 1 file changed, 5 insertions(+) diff --git a/libavcodec/libx264.c b/libavcodec/libx264.c index 9c67c91f33..1831f9b30e 100644 --- a/libavcodec/libx264.c +++ b/libavcodec/libx264.c @@ -116,6 +116,7 @@ static int encode_nals(AVCodec

[FFmpeg-devel] mp4 sdtp (sample dependency) box

2017-11-19 Thread John Stebbins
Patch 4 & 5 should be squashed before commit. Patch 4 "works" on it's own, but doesn't account for edit list fix-ups. I seperated them to make reviewing easier. Patch 5 merges ctts_data into the new mov_index_entries struct I created, so it touches quite a lot, but results in a fair amount of cod

[FFmpeg-devel] [PATCH 3/6] lavc/libx265: mark disposable frames

2017-11-19 Thread John Stebbins
Used by movenc to fill sdtp box --- libavcodec/libx265.c | 4 1 file changed, 4 insertions(+) diff --git a/libavcodec/libx265.c b/libavcodec/libx265.c index 4456e300f2..8279a99ddd 100644 --- a/libavcodec/libx265.c +++ b/libavcodec/libx265.c @@ -329,6 +329,10 @@ FF_DISABLE_DEPRECATION_WARNING

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

2017-11-19 Thread Michael Niedermayer
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] avfilter/vf_tile: add queue option

2017-11-19 Thread Nicolas George
Dave Rice (2017-11-19): > IMHO, ‘queue' is a good name for what it does. Based on your explanation, I would say not at all. It rather seems to say the opposite. I would suggest something like "init_padding". Regards, -- Nicolas George signature.asc Description: Digital signature ___

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

2017-11-19 Thread Dave Rice
> On Nov 19, 2017, at 2:19 PM, Nicolas George wrote: > > Paul B Mahol (2017-11-19): >> So what you propose? > > I do not know. It is your patch, and I am not even sure I understood > your explanation correctly. IMHO, ‘queue' is a good name for what it does. Perhaps an additional example woul

Re: [FFmpeg-devel] [DEVEL][PATCH 2/2] ffmpeg: fix ticket 6706

2017-11-19 Thread pkv.stream
Le 19/11/2017 à 8:28 PM, Michael Niedermayer a écrit : On Sun, Nov 19, 2017 at 11:01:37AM +0100, pkv.stream wrote: [...] @@ -3674,6 +3697,10 @@ const OptionDef options[] = { { "channel_layout", OPT_AUDIO | HAS_ARG | OPT_EXPERT | OPT_PERFILE | OPT_INPUT | OPT_OUT

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

2017-11-19 Thread Philip Langdale
Once I remembered that there's a separate decoder type for mpeg1, even though params struct is shared with mpeg2, everything worked. Signed-off-by: Philip Langdale --- Changelog | 2 +- configure | 2 ++ libavcodec/Makefile | 1 + libavcodec/allcodecs.c

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

2017-11-19 Thread Philip Langdale
This was predictably nightmarish, given how ridiculous mpeg4 is. I had to stare at the cuvid parser output for a long time to work out what each field was supposed to be, and even then, I still don't fully understand some of them, particularly: vop_coded: I think this means whether the vop has a p

[FFmpeg-devel] [PATCH 0/3] nvdec hwaccels for mpeg1 and mpeg4

2017-11-19 Thread Philip Langdale
Consolidating the existing outstanding patches along with a new mpeg4 patch. I have working vp8 and mjpeg implementations waiting in the hwaccel hooks getting merged. Philip Langdale (3): avcodec: Refactor common nvdec hwaccel logic avcodec: Implement mpeg1 nvdec hwaccel avcodec: Implement

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

2017-11-19 Thread Philip Langdale
The 'simple' hwaccels (not h.264 and hevc) all use the same bitstream management and reference lookup logic so let's refactor all that into common functions. I verified that casting a signed int -1 to unsigned char produces 255 according to the C language specification. Signed-off-by: Philip Lang

Re: [FFmpeg-devel] [DEVEL][PATCH 2/2] ffmpeg: fix ticket 6706

2017-11-19 Thread Michael Niedermayer
On Sun, Nov 19, 2017 at 11:01:37AM +0100, pkv.stream wrote: [...] > @@ -3674,6 +3697,10 @@ const OptionDef options[] = { > { "channel_layout", OPT_AUDIO | HAS_ARG | OPT_EXPERT | OPT_PERFILE | > OPT_INPUT | OPT_OUTPUT, > { .func_arg

Re: [FFmpeg-devel] [DEVEL][PATCH 1/2] ffmpeg: add ui64 type to SpecifierOpt

2017-11-19 Thread Michael Niedermayer
On Sun, Nov 19, 2017 at 11:01:33AM +0100, pkv.stream wrote: > Hi Michael > > >>Tell me the best course; or if you see a way to make my > >>MATCH_PER_STREAM_OPT() code less hacky. > >iam sure theres a way to do this less hacky > >why do you need a 2nd table ? or rather why does it not work if you >

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

2017-11-19 Thread Nicolas George
Paul B Mahol (2017-11-19): > So what you propose? I do not know. It is your patch, and I am not even sure I understood your explanation correctly. Regards, -- Nicolas George signature.asc Description: Digital signature ___ ffmpeg-devel mailing lis

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

2017-11-19 Thread Paul B Mahol
On 11/19/17, Nicolas George wrote: > Le nonidi 29 brumaire, an CCXXVI, Paul B Mahol a ecrit : >> It allows user to start outputing video frames immediately with first >> input frame. >> Instead of after nb_frames. > > I think I understand what it means, but only because I have read the > code, and

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

2017-11-19 Thread Marton Balint
On Thu, 16 Nov 2017, Jeyapal, Karthick wrote: On 11/16/17, 1:11 AM, "Marton Balint" wrote: Thanks for the reply. For the first patch, I think you forgot to call ff_decklink_set_configs in decklink_enc. ff_decklink_set_configs is not doing anything related to decklink_enc In any case I hav

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

2017-11-19 Thread James Almer
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 insertions(+), 22 deletions(-) diff --git a/configure b/configure index 8b7b7

Re: [FFmpeg-devel] [PATCH] avformat/subfile: allow to extract till EOF

2017-11-19 Thread Nicolas George
Le quintidi 25 brumaire, an CCXXVI, Gyan Doshi a écrit : > Disregard patch in first message. Didn't remove residue of aborted doc > change earlier. Patch applied. Thanks. Regards, -- Nicolas George signature.asc Description: Digital signature ___

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

2017-11-19 Thread Nicolas George
Le nonidi 29 brumaire, an CCXXVI, Paul B Mahol a écrit : > It allows user to start outputing video frames immediately with first > input frame. > Instead of after nb_frames. I think I understand what it means, but only because I have read the code, and even like that I am not entirely sure. This n

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

2017-11-19 Thread Paul B Mahol
On 11/19/17, Nicolas George wrote: > Le septidi 27 brumaire, an CCXXVI, Paul B Mahol a ecrit : >> Signed-off-by: Paul B Mahol >> --- >> doc/filters.texi | 4 >> libavfilter/vf_tile.c | 18 +- >> 2 files changed, 21 insertions(+), 1 deletion(-) >> >> diff --git a/doc/fi

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

2017-11-19 Thread Nicolas George
L'octidi 28 brumaire, an CCXXVI, Michael Roitzsch a écrit : > Hi FFmpeg team, > > I was using af_pan to subtract one audio file from another. I first used an > amerge filter and then configured the pan filter with a channel formula like > c0=c0-c2|c1=c1-c3. > > However, this does not work as ex

Re: [FFmpeg-devel] [PATCH 2/3] avfilter/vf_cropdetect: don't ignore frames, export crop parameters

2017-11-19 Thread Nicolas George
Le septidi 27 brumaire, an CCXXVI, Aman Gupta a écrit : > From: Aman Gupta > > --- > libavfilter/vf_cropdetect.c | 9 +++-- > 1 file changed, 7 insertions(+), 2 deletions(-) I think the changes belong in different commits, since they both change the behaviour of the filter in different ways

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

2017-11-19 Thread Nicolas George
Le septidi 27 brumaire, an CCXXVI, Paul B Mahol a écrit : > Signed-off-by: Paul B Mahol > --- > doc/filters.texi | 4 > libavfilter/vf_tile.c | 18 +- > 2 files changed, 21 insertions(+), 1 deletion(-) > > diff --git a/doc/filters.texi b/doc/filters.texi > index 5d9943

Re: [FFmpeg-devel] [PATCH] avfilter/avfiltergraph: pass correct audio/video flags

2017-11-19 Thread Nicolas George
Le nonidi 29 brumaire, an CCXXVI, Paul B Mahol a écrit : > Signed-off-by: Paul B Mahol > --- > libavfilter/avfiltergraph.c | 14 -- > 1 file changed, 8 insertions(+), 6 deletions(-) LGTM. Maybe add in the commit message the real reason for the change (a flag that was previously ignor

Re: [FFmpeg-devel] [PATCH] 8-bit hevc decoding optimization on aarch64 with neon

2017-11-19 Thread Rafal Dabrowa
On 11/18/2017 07:41 PM, James Almer wrote: On 11/18/2017 3:31 PM, Rostislav Pehlivanov wrote: On 18 November 2017 at 17:35, Rafal Dabrowa wrote: This is a proposal of performance optimizations for 8-bit hevc video decoding on aarch64 platform with neon (simd) extension. I'm testing my optim

[FFmpeg-devel] [PATCH] avfilter/avfiltergraph: pass correct audio/video flags

2017-11-19 Thread Paul B Mahol
Signed-off-by: Paul B Mahol --- libavfilter/avfiltergraph.c | 14 -- 1 file changed, 8 insertions(+), 6 deletions(-) diff --git a/libavfilter/avfiltergraph.c b/libavfilter/avfiltergraph.c index a009e0a760..a959a9ef3f 100644 --- a/libavfilter/avfiltergraph.c +++ b/libavfilter/avfilter

Re: [FFmpeg-devel] order T-shirts

2017-11-19 Thread Thilo Borgmann
Hi, >> My suggestion would be, that we could order Thomas' design for all >> the developers and requests I've recieved by now and that we take >> Lou's for our stock for give-aways during conferences. Just my >> thinking... > > no further comments so I did order alike. Should recieve them next we

Re: [FFmpeg-devel] [PATCH] configure: add audio_frame_queue dependency for aptx codec

2017-11-19 Thread Jan Ekstrom
On Sun, Nov 19, 2017 at 4:01 PM, James Darnley wrote: > --- > configure | 2 ++ > 1 file changed, 2 insertions(+) > LGTM, fixes shared linkage reported on IRC. ___ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org http://ffmpeg.org/mailman/listinfo/ffm

[FFmpeg-devel] [PATCH] configure: add audio_frame_queue dependency for aptx codec

2017-11-19 Thread James Darnley
--- configure | 2 ++ 1 file changed, 2 insertions(+) diff --git a/configure b/configure index 8b7b7e164b..48761934be 100755 --- a/configure +++ b/configure @@ -2439,6 +2439,8 @@ amv_encoder_select="aandcttables jpegtables mpegvideoenc" ape_decoder_select="bswapdsp llauddsp" apng_decoder_select

Re: [FFmpeg-devel] [Patch] Fix for ticket 6658 (Dash demuxer segfault)

2017-11-19 Thread Moritz Barsnick
On Sat, Nov 18, 2017 at 00:16:33 +, Colin NG wrote: > Cleaned the code up. Please do read the ffmpeg style guide. > +static char * ishttp(char *url) { Asterisk attaches to the right hand side. > -ret = s->io_open(s, pb, url, AVIO_FLAG_READ, &tmp); > +av_freep(pb); > +ret = avio_

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

2017-11-19 Thread James Almer
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 insertions(+), 10 deletions(-) > > diff --git a/configure b

[FFmpeg-devel] [PATCH] libavformat/mov:the "stco" box parse no need return eof.

2017-11-19 Thread tiejun.peng
if return eof, will cause to play the whole mp4 file fail. For example, currently the whole file will play fail when a few bytes missed in the "stco" box and the "stco" box is the last box of MP4 file, Actually the almost media frames can play success at most time. A few words missed in "stco" box

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

2017-11-19 Thread James Almer
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 1eca97a081..36f915a8e5 100644 > --- a/libavcodec/libvpx.c > ++

[FFmpeg-devel] [PATCH] libavformat/id3v2: add some ID3v2.3 tags convertion

2017-11-19 Thread tiejun.peng
Signed-off-by: tiejun.peng --- libavformat/id3v2.c | 5 + 1 file changed, 5 insertions(+) diff --git a/libavformat/id3v2.c b/libavformat/id3v2.c index 6c216ba..1cf759b 100644 --- a/libavformat/id3v2.c +++ b/libavformat/id3v2.c @@ -56,6 +56,11 @@ const AVMetadataConv ff_id3v2_34_metadata_conv

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

2017-11-19 Thread Paul B Mahol
On 11/19/17, Richard Ling wrote: > OK, trying again. I've worked out how to send to myself without corruption > (I think). Still broken. Post patch as attachment. ___ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org http://ffmpeg.org/mailman/listinfo/f

Re: [FFmpeg-devel] avcodec/qsvenc Question for Intel QSV low latency

2017-11-19 Thread Natsuki Kai
I sent it as another thread, but yes it's ok. Please check the attachment patch file. Thank you. On Wed, Nov 15, 2017 at 7:55 AM, Carl Eugen Hoyos wrote: > 2017-11-14 17:02 GMT+01:00 Natsuki Kai : >> Hi Moritz, >> >> Thank you for explaining in details. It really helped me. >> It seems gmail or

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

2017-11-19 Thread Timo Rothenpieler
Usually if a discussion comes up the mail from cvslog is replied to on this list, so no actual discussion happens on the automatic cvslog list. smime.p7s Description: S/MIME Cryptographic Signature ___ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.or

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

2017-11-19 Thread Jim DeLaHunt
Hello, all: Continuing my reading of the FFmpeg developer instructions, I find this at , *Developer Guide*, /1.4.3 Documentation/Other/ : *Subscribe to the ffmpeg-cvslog mailing list.* It is important to do this as the diffs of

Re: [FFmpeg-devel] Source for http://ffmpeg.org/contact.html and documentation.html ?

2017-11-19 Thread Jim DeLaHunt
On 2017-11-18 14:53, Mark Thompson wrote: On 18/11/17 22:32, Jim DeLaHunt wrote: Hello, all: From what sources are the web pages http://ffmpeg.org/contact.html and https://ffmpeg.org/documentation.html generated? Sister web pages, like https://ffmpeg.org/ffmpeg.html and http://ffmpeg.org/de

[FFmpeg-devel] [DEVEL][PATCH 1/2] ffmpeg: add ui64 type to SpecifierOpt

2017-11-19 Thread pkv.stream
Hi Michael Tell me the best course; or if you see a way to make my MATCH_PER_STREAM_OPT() code less hacky. iam sure theres a way to do this less hacky why do you need a 2nd table ? or rather why does it not work if you put the entry in the main table ? (so there are 2 entries one for OPT_SPEC a

[FFmpeg-devel] [DEVEL][PATCH 2/2] ffmpeg: fix ticket 6706

2017-11-19 Thread pkv.stream
From f94f2e8c8878d6dbda540b19d90c2b8f1ba00850 Mon Sep 17 00:00:00 2001 From: pkviet Date: Sat, 18 Nov 2017 00:26:50 +0100 Subject: [PATCH 2/2] ffmpeg: fix ticket 6706 Fix regression with channel_layout option which is not passed correctly from output streams to filters when the channel layout

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

2017-11-19 Thread Reto Kromer
Dave Rice wrote: >As required by Apple’s TN2162. >--- > libavformat/movenc.c | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) LGTM ___ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org http://ffmpeg.org/mailman/listinfo/ffmpeg-devel

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

2017-11-19 Thread Richard Ling
OK, trying again. I've worked out how to send to myself without corruption (I think). From 590b3bc8e2675c75c2ff7e75f7fc1fbb1e1a8f71 Mon Sep 17 00:00:00 2001 From: Richard Ling Date: Thu, 16 Nov 2017 23:00:01 +1100 Subject: [PATCH] avfilter: add normalize filter --- doc/filters.texi |