Re: [FFmpeg-devel] [PATCH] libavformat/aac: Parse ID3 tags between ADTS frames.

2018-02-07 Thread Richard Shaffer
I did send the sample file and a unit test for this. If there are no issues or other comments, would you guys be willing to merge it? -Richard On Fri, Feb 2, 2018 at 11:15 AM, Richard Shaffer wrote: > On Fri, Feb 2, 2018 at 5:52 AM, James Almer wrote: >>

Re: [FFmpeg-devel] [FFmpeg-cvslog] lavc: add new API for iterating codecs and codec parsers

2018-02-07 Thread Nicolas George
James Almer (2018-02-07): > Since reverting would be dirty, I'd prefer if we keep the discussion > about the desired API going and then apply the needed patches on top of > the current tree. > As long as we don't take weeks to do it (and do it before a release is > tagged), any kind of change to

[FFmpeg-devel] [PATCH] examples: Don't call deprecated functions which don't do anything

2018-02-07 Thread Mark Thompson
--- doc/examples/avio_dir_cmd.c | 2 -- doc/examples/avio_reading.c | 3 --- doc/examples/decode_audio.c | 3 --- doc/examples/decode_video.c | 2 -- doc/examples/demuxing_decoding.c | 3 --- doc/examples/encode_audio.c | 3 --- doc/examples/encode_video.c | 2 --

Re: [FFmpeg-devel] [FFmpeg-cvslog] doc/APIchanges: mention a few more recently introduced and deprecated functions

2018-02-07 Thread Nicolas George
James Almer (2018-02-07): > ffmpeg | branch: master | James Almer | Wed Feb 7 > 17:05:01 2018 -0300| [3f0a41367eb9180ab6d22d43ad42b9bd85a26df0] | committer: > James Almer > > doc/APIchanges: mention a few more recently introduced and deprecated > functions Pushing that

Re: [FFmpeg-devel] [PATCH] cmdutils: fix finding next codec for id

2018-02-07 Thread Mark Thompson
On 07/02/18 12:14, Josh de Kock wrote: > --- > fftools/cmdutils.c | 11 +++ > 1 file changed, 7 insertions(+), 4 deletions(-) > > diff --git a/fftools/cmdutils.c b/fftools/cmdutils.c > index 0b06ccc71a..8bb9952016 100644 > --- a/fftools/cmdutils.c > +++ b/fftools/cmdutils.c > @@ -1424,11

Re: [FFmpeg-devel] [FFmpeg-cvslog] doc/APIchanges: mention a few more recently introduced and deprecated functions

2018-02-07 Thread James Almer
On 2/7/2018 5:11 PM, Nicolas George wrote: > James Almer (2018-02-07): >> ffmpeg | branch: master | James Almer | Wed Feb 7 >> 17:05:01 2018 -0300| [3f0a41367eb9180ab6d22d43ad42b9bd85a26df0] | committer: >> James Almer >> >> doc/APIchanges: mention a few more recently

Re: [FFmpeg-devel] [FFmpeg-cvslog] lavc: add new API for iterating codecs and codec parsers

2018-02-07 Thread wm4
On Wed, 7 Feb 2018 19:33:15 +0100 Nicolas George wrote: > James Almer (2018-02-07): > > Since reverting would be dirty, I'd prefer if we keep the discussion > > about the desired API going and then apply the needed patches on top of > > the current tree. > > As long as we don't

Re: [FFmpeg-devel] [FFmpeg-cvslog] lavc: add new API for iterating codecs and codec parsers

2018-02-07 Thread James Almer
On 2/7/2018 9:52 AM, Nicolas George wrote: > Josh de Kock (2018-02-06): >> ffmpeg | branch: master | Josh de Kock | Fri Dec 22 >> 22:17:00 2017 +| [7e8eba2d8755962d9dca5eade57bf8f591a73c0c] | committer: >> Josh de Kock >> >> lavc: add new API for iterating codecs and codec

Re: [FFmpeg-devel] [PATCH] cmdutils: fix printing of codecs

2018-02-07 Thread Michael Niedermayer
On Wed, Feb 07, 2018 at 09:07:39PM +, Josh de Kock wrote: > Yes, my bad. It looked like it worked initially (was more > worried about getting a fix out quickly), but it didnt. Try this one. > > --- > fftools/cmdutils.c | 104 > + > 1 file

Re: [FFmpeg-devel] [PATCH] cmdutils: fix printing of codecs

2018-02-07 Thread Michael Niedermayer
On Wed, Feb 07, 2018 at 09:07:39PM +, Josh de Kock wrote: > Yes, my bad. It looked like it worked initially (was more > worried about getting a fix out quickly), but it didnt. Try this one. > > --- > fftools/cmdutils.c | 104 > + > 1 file

Re: [FFmpeg-devel] [PATCH] cmdutils: fix printing of codecs

2018-02-07 Thread Michael Niedermayer
On Wed, Feb 07, 2018 at 09:35:24PM -0300, James Almer wrote: > On 2/7/2018 9:32 PM, Michael Niedermayer wrote: > > On Wed, Feb 07, 2018 at 09:07:39PM +, Josh de Kock wrote: > >> Yes, my bad. It looked like it worked initially (was more > >> worried about getting a fix out quickly), but it

Re: [FFmpeg-devel] [PATCH] lavc/vaapi: release buffer before destroy context.

2018-02-07 Thread Jun Zhao
On 2018/2/8 9:35, Ruiling Song wrote: > The common way to use libVA was first destroy the buffer, then the > context. I am not sure whether libVA has clear statement on this. > This patch just make things simple. This would fix an segmentation > fault issue against iHD open source driver. > >

Re: [FFmpeg-devel] tinterlace license

2018-02-07 Thread Stefano Sabatini
No objections from my side for the parts I contributed (but IIRC was mostly a port of code authored by other people). Best regards. Il 06/feb/2018 19:19, "Thomas Mundt" ha scritto: > Hi, > > 2018-02-05 11:58 GMT+01:00 Vasile Toncu : > > > Hello, >

Re: [FFmpeg-devel] [FFmpeg-cvslog] lavc: add new API for iterating codecs and codec parsers

2018-02-07 Thread Muhammad Faiz
On Thu, Feb 8, 2018 at 1:33 AM, Nicolas George wrote: > James Almer (2018-02-07): >> Since reverting would be dirty, I'd prefer if we keep the discussion >> about the desired API going and then apply the needed patches on top of >> the current tree. >> As long as we don't take

[FFmpeg-devel] [PATCH] lavc/vaapi: release buffer before destroy context.

2018-02-07 Thread Ruiling Song
The common way to use libVA was first destroy the buffer, then the context. I am not sure whether libVA has clear statement on this. This patch just make things simple. This would fix an segmentation fault issue against iHD open source driver. Signed-off-by: Ruiling Song

Re: [FFmpeg-devel] [FFmpeg-cvslog] lavf: add new API for iterating muxers and demuxers

2018-02-07 Thread Michael Niedermayer
On Tue, Feb 06, 2018 at 06:57:49PM +, Josh de Kock wrote: > ffmpeg | branch: master | Josh de Kock | Sun Jan 21 > 16:06:54 2018 +| [0694d8702421e7aff1340038559c438b61bb30dd] | committer: > Josh de Kock > > lavf: add new API for iterating muxers and demuxers > > >

Re: [FFmpeg-devel] [PATCH] cmdutils: fix printing of codecs

2018-02-07 Thread Muhammad Faiz
On Thu, Feb 8, 2018 at 7:35 AM, James Almer wrote: > On 2/7/2018 9:32 PM, Michael Niedermayer wrote: >> On Wed, Feb 07, 2018 at 09:07:39PM +, Josh de Kock wrote: >>> Yes, my bad. It looked like it worked initially (was more >>> worried about getting a fix out quickly), but

Re: [FFmpeg-devel] [FFmpeg-cvslog] lavc: add new API for iterating codecs and codec parsers

2018-02-07 Thread Michael Niedermayer
On Wed, Feb 07, 2018 at 01:52:33PM +0100, Nicolas George wrote: > Josh de Kock (2018-02-06): > > ffmpeg | branch: master | Josh de Kock | Fri Dec 22 > > 22:17:00 2017 +| [7e8eba2d8755962d9dca5eade57bf8f591a73c0c] | > > committer: Josh de Kock > > > > lavc: add new API for

Re: [FFmpeg-devel] [PATCH] cmdutils: fix printing of codecs

2018-02-07 Thread James Almer
On 2/7/2018 9:32 PM, Michael Niedermayer wrote: > On Wed, Feb 07, 2018 at 09:07:39PM +, Josh de Kock wrote: >> Yes, my bad. It looked like it worked initially (was more >> worried about getting a fix out quickly), but it didnt. Try this one. >> >> --- >> fftools/cmdutils.c | 104 >>

Re: [FFmpeg-devel] [FFmpeng-devel] Dose ffmpeg plan to support FMO decode in h264

2018-02-07 Thread Michael Niedermayer
On Wed, Feb 07, 2018 at 02:15:26PM +0800, Wei Gao wrote: > 2018-02-07 6:57 GMT+08:00 Michael Niedermayer : > > > On Tue, Feb 06, 2018 at 11:14:49AM +0800, Wei Gao wrote: > > > Hi > > > > > > I have a question. > > > > > > Does ffmpeg plan to support FMO(Flexible Macroblock

[FFmpeg-devel] [PATCH] cmdutils: fix printing of codecs

2018-02-07 Thread Josh de Kock
Yes, my bad. It looked like it worked initially (was more worried about getting a fix out quickly), but it didnt. Try this one. --- fftools/cmdutils.c | 104 + 1 file changed, 58 insertions(+), 46 deletions(-) diff --git a/fftools/cmdutils.c

Re: [FFmpeg-devel] [PATCH] avcodec/codec_desc: sort codec_descriptors

2018-02-07 Thread Muhammad Faiz
On Sat, Feb 3, 2018 at 1:35 AM, Muhammad Faiz wrote: > Use bsearch on avcodec_descriptor_get(). > > Signed-off-by: Muhammad Faiz > --- > libavcodec/codec_desc.c | 1103 > --- > 1 file changed, 563 insertions(+),

Re: [FFmpeg-devel] [PATCH] lavc/vaapi: release buffer before destroy context.

2018-02-07 Thread Song, Ruiling
> -Original Message- > From: ffmpeg-devel [mailto:ffmpeg-devel-boun...@ffmpeg.org] On Behalf Of > Jun Zhao > Sent: Thursday, February 8, 2018 11:08 AM > To: FFmpeg development discussions and patches ; > Song, Ruiling > Subject: Re:

Re: [FFmpeg-devel] [PATCH] lavc/vaapi_encode_h265: add support for low-power encoding

2018-02-07 Thread Mark Thompson
On 06/02/18 08:17, Haihao Xiang wrote: > Although VAEntrypointEncSliceLP was added in old version of VAAPI, we > never implemented it for VAAPI H265 encoder before. so it is reasonable > to require VAAPI 1.0 > > Signed-off-by: Haihao Xiang > --- >

Re: [FFmpeg-devel] [PATCH] lavfi: add a gitignore file for OpenCL compiled kernels

2018-02-07 Thread Mark Thompson
On 05/02/18 11:06, Rostislav Pehlivanov wrote: > Signed-off-by: Rostislav Pehlivanov > --- > libavfilter/opencl/.gitignore | 1 + > 1 file changed, 1 insertion(+) > create mode 100644 libavfilter/opencl/.gitignore > > diff --git a/libavfilter/opencl/.gitignore

Re: [FFmpeg-devel] [PATCH] lavfi: add a gitignore file for OpenCL compiled kernels

2018-02-07 Thread Mark Thompson
On 07/02/18 10:46, Mark Thompson wrote: > On 05/02/18 11:06, Rostislav Pehlivanov wrote: >> Signed-off-by: Rostislav Pehlivanov >> --- >> libavfilter/opencl/.gitignore | 1 + >> 1 file changed, 1 insertion(+) >> create mode 100644 libavfilter/opencl/.gitignore >> >> diff

Re: [FFmpeg-devel] [PATCH] libavformat/avio: NULL check for URLContext in ffurl_write().

2018-02-07 Thread wm4
On Wed, 7 Feb 2018 08:57:26 +0530 rpata...@akamai.com wrote: > From: Ravindra > > Signed-off-by: Ravindra > --- > libavformat/avio.c | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/libavformat/avio.c b/libavformat/avio.c >

Re: [FFmpeg-devel] [PATCH] lavfi: add a gitignore file for OpenCL compiled kernels

2018-02-07 Thread Rostislav Pehlivanov
On 7 February 2018 at 11:09, Mark Thompson wrote: > On 07/02/18 10:46, Mark Thompson wrote: > > On 05/02/18 11:06, Rostislav Pehlivanov wrote: > >> Signed-off-by: Rostislav Pehlivanov > >> --- > >> libavfilter/opencl/.gitignore | 1 + > >> 1 file changed, 1

Re: [FFmpeg-devel] [PATCH] libavformat/hls: Support metadata updates from subdemuxers

2018-02-07 Thread wm4
On Fri, 2 Feb 2018 12:59:45 -0800 rshaf...@tunein.com wrote: > From: Richard Shaffer > > If a subdemuxer has the updated metadata event flag set, the metadata is > copied > to the corresponding stream. The flag is cleared on the subdemuxer and the > appropriate event flag

[FFmpeg-devel] [PATCH] cmdutils: fix finding next codec for id

2018-02-07 Thread Josh de Kock
--- fftools/cmdutils.c | 11 +++ 1 file changed, 7 insertions(+), 4 deletions(-) diff --git a/fftools/cmdutils.c b/fftools/cmdutils.c index 0b06ccc71a..8bb9952016 100644 --- a/fftools/cmdutils.c +++ b/fftools/cmdutils.c @@ -1424,11 +1424,14 @@ static char get_media_type_char(enum

Re: [FFmpeg-devel] [FFmpeg-cvslog] lavc: add new API for iterating codecs and codec parsers

2018-02-07 Thread Nicolas George
Josh de Kock (2018-02-06): > ffmpeg | branch: master | Josh de Kock | Fri Dec 22 > 22:17:00 2017 +| [7e8eba2d8755962d9dca5eade57bf8f591a73c0c] | committer: > Josh de Kock > > lavc: add new API for iterating codecs and codec parsers > > Based on an unfinished patch by