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

2017-12-27 Thread 刘歧
> On 28 Dec 2017, at 07:04, Michael Niedermayer wrote: > > On Mon, Nov 20, 2017 at 03:58:59PM +0800, Steven Liu wrote: >> fix ticket id: #6846 >> >> Signed-off-by: Steven Liu >> --- >> libavformat/avio.c | 10 ++ >> 1 file changed, 6 insertions(+), 4 deletions(-) > > LGTM > > thx Pu

Re: [FFmpeg-devel] [PATCH] avfilter/vidstab: set bytesPerPixel only for packed formats.

2017-12-27 Thread Gyan Doshi
On 12/24/2017 10:36 AM, Gyan Doshi wrote: On 12/23/2017 4:39 PM, Gyan Doshi wrote: Patch for ticket #6736. Ping. ___ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org http://ffmpeg.org/mailman/listinfo/ffmpeg-devel

Re: [FFmpeg-devel] [PATCH]lavc/jpeg2000dec: Support reading 64-bit atom size.

2017-12-27 Thread Carl Eugen Hoyos
2017-12-28 1:51 GMT+01:00 Michael Niedermayer : > On Wed, Dec 27, 2017 at 11:29:33PM +0100, Carl Eugen Hoyos wrote: >> Hi! >> >> Attached patch fixes ticket #6935 for me, files with actual atom size >> >= 2^32 cannot be read by the current bytestream system afaict. >> >> Please comment, Carl Eugen

Re: [FFmpeg-devel] [PATCH]lavc/jpeg2000dec: Support reading 64-bit atom size.

2017-12-27 Thread Michael Niedermayer
On Wed, Dec 27, 2017 at 11:29:33PM +0100, Carl Eugen Hoyos wrote: > Hi! > > Attached patch fixes ticket #6935 for me, files with actual atom size > >= 2^32 cannot be read by the current bytestream system afaict. > > Please comment, Carl Eugen > jpeg2000dec.c | 11 ++- > 1 file changed

Re: [FFmpeg-devel] [PATCH] h264: add AVOption to set x264_build default

2017-12-27 Thread Carl Eugen Hoyos
2017-12-28 1:35 GMT+01:00 James Almer : > On 12/27/2017 9:31 PM, Carl Eugen Hoyos wrote: >> 2017-12-28 0:58 GMT+01:00 wm4 : >>> On Thu, 28 Dec 2017 00:56:05 +0100 >>> Carl Eugen Hoyos wrote: >>> 2017-12-26 3:02 GMT+01:00 wm4 : > On Sat, 23 Dec 2017 03:14:49 +0100 > wm4 wrote: > >

Re: [FFmpeg-devel] [PATCH] h264: add AVOption to set x264_build default

2017-12-27 Thread James Almer
On 12/27/2017 9:31 PM, Carl Eugen Hoyos wrote: > 2017-12-28 0:58 GMT+01:00 wm4 : >> On Thu, 28 Dec 2017 00:56:05 +0100 >> Carl Eugen Hoyos wrote: >> >>> 2017-12-26 3:02 GMT+01:00 wm4 : On Sat, 23 Dec 2017 03:14:49 +0100 wm4 wrote: > This provides a generic way to the API user t

Re: [FFmpeg-devel] [PATCH] h264: add AVOption to set x264_build default

2017-12-27 Thread Carl Eugen Hoyos
2017-12-28 0:58 GMT+01:00 wm4 : > On Thu, 28 Dec 2017 00:56:05 +0100 > Carl Eugen Hoyos wrote: > >> 2017-12-26 3:02 GMT+01:00 wm4 : >> > On Sat, 23 Dec 2017 03:14:49 +0100 >> > wm4 wrote: >> > >> >> This provides a generic way to the API user to deal with files that >> >> either lack this SEI, or

Re: [FFmpeg-devel] [PATCH]lavfi/minterpolate: Split struct Pixel to allow higher resolutions

2017-12-27 Thread Carl Eugen Hoyos
2017-12-05 1:43 GMT+01:00 Michael Niedermayer : > On Mon, Dec 04, 2017 at 09:26:15PM +0100, Carl Eugen Hoyos wrote: >> 2017-12-04 19:03 GMT+01:00 Michael Niedermayer : >> > On Sun, Dec 03, 2017 at 05:30:32AM +0100, Carl Eugen Hoyos wrote: >> >> Hi! >> >> >> >> Attached patch should fix ticket #6795

Re: [FFmpeg-devel] [RFC][PATCH] simple_idct: Template functions to support an input bitdepth parameter

2017-12-27 Thread Michael Niedermayer
On Wed, Dec 27, 2017 at 10:27:25PM +, Kieran Kunhya wrote: > On Wed, 27 Dec 2017 at 19:53 Michael Niedermayer > wrote: > > > please call the functions ff_simple_idct_int16_10bit or something that > > makes it > > clear what the 2 numbers mean. > > > > done > > > [...] > > is that faster th

Re: [FFmpeg-devel] [PATCH] h264: add AVOption to set x264_build default

2017-12-27 Thread wm4
On Thu, 28 Dec 2017 00:56:05 +0100 Carl Eugen Hoyos wrote: > 2017-12-26 3:02 GMT+01:00 wm4 : > > On Sat, 23 Dec 2017 03:14:49 +0100 > > wm4 wrote: > > > >> This provides a generic way to the API user to deal with files that > >> either lack this SEI, or which have the SEI only in packets not p

Re: [FFmpeg-devel] [PATCH] h264: add AVOption to set x264_build default

2017-12-27 Thread Carl Eugen Hoyos
2017-12-26 3:02 GMT+01:00 wm4 : > On Sat, 23 Dec 2017 03:14:49 +0100 > wm4 wrote: > >> This provides a generic way to the API user to deal with files that >> either lack this SEI, or which have the SEI only in packets not passed >> to the decoder (such as the common case of the SEI being in the ve

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

2017-12-27 Thread Michael Niedermayer
On Mon, Nov 20, 2017 at 03:58:59PM +0800, Steven Liu wrote: > fix ticket id: #6846 > > Signed-off-by: Steven Liu > --- > libavformat/avio.c | 10 ++ > 1 file changed, 6 insertions(+), 4 deletions(-) LGTM thx [...] -- Michael GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B

[FFmpeg-devel] [PATCH]lavc/jpeg2000dec: Support reading 64-bit atom size.

2017-12-27 Thread Carl Eugen Hoyos
Hi! Attached patch fixes ticket #6935 for me, files with actual atom size >= 2^32 cannot be read by the current bytestream system afaict. Please comment, Carl Eugen From db1010aec864cb963fac243907565fdd8c8a1636 Mon Sep 17 00:00:00 2001 From: Carl Eugen Hoyos Date: Wed, 27 Dec 2017 23:18:58 +0100

Re: [FFmpeg-devel] Licensing question.

2017-12-27 Thread Carl Eugen Hoyos
2017-12-27 20:48 GMT+01:00 Lewis Hornsby : > Hey there, I saw on your facebook page that you were giving out this > email address for enquiries regarding FFMPEG licensing. To whoever takes care of our facebook page: Please redirect license questions to the user mailing list. > I am developing a c

Re: [FFmpeg-devel] [RFC][PATCH] simple_idct: Template functions to support an input bitdepth parameter

2017-12-27 Thread Kieran Kunhya
On Wed, 27 Dec 2017 at 19:53 Michael Niedermayer wrote: > please call the functions ff_simple_idct_int16_10bit or something that > makes it > clear what the 2 numbers mean. > done > [...] is that faster than checking row 4-7 for 0 with sparse matrixes as occuring > in > video data ? > added.

Re: [FFmpeg-devel] [PATCH 2/5] mpeg: add experimental support for PSMF audio.

2017-12-27 Thread Michael Niedermayer
On Tue, Dec 26, 2017 at 04:06:49PM +0800, mi...@brew.sh wrote: > From: Maxim Poliakovski > > Changes by Misty De Meo : > > atrac3plus_parser: remove return statements for invalid data > > atrac3plus_parser: use libavcodec's oma > > Signed-off-by: Misty De Meo > --- > libavcodec/Makefile

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

2017-12-27 Thread Tomas Härdin
ons 2017-12-27 klockan 19:21 +0100 skrev Paul B Mahol: > > > +static int filter_frame(AVFilterLink *inlink, AVFrame *in) > +{ > +AVFilterContext *ctx = inlink->dst; > +AVFilterLink *outlink = ctx->outputs[0]; > +EntropyContext *s = ctx->priv; > +int plane, y, x; > + > +for (pla

Re: [FFmpeg-devel] Licensing question.

2017-12-27 Thread Lou Logan
On Wed, 27 Dec 2017 19:48:52 + Lewis Hornsby wrote: > Hey there, I saw on your facebook page that you were giving out this > email address for enquiries regarding FFMPEG licensing. > > I am developing a closed source application in NW.JS. It is a > Wordpress theme builder. There is a part of

Re: [FFmpeg-devel] [PATCH] AIFF fix in case of ANNO

2017-12-27 Thread Moritz Barsnick
On Wed, Dec 27, 2017 at 08:56:17 +, Eduard Sinelnikov wrote: > Subject: [PATCH] AIFF fix in case of ANNO You should be a bit more precise as to what was wrong and how it was fixed. Also please use the prefix "avformat/aiffdec: ", see other commits in the repo. > From: Ubuntu I'm certain th

[FFmpeg-devel] Licensing question.

2017-12-27 Thread Lewis Hornsby
Hey there, I saw on your facebook page that you were giving out this email address for enquiries regarding FFMPEG licensing. I am developing a closed source application in NW.JS. It is a Wordpress theme builder. There is a part of my application that lets people browse their available media. I

[FFmpeg-devel] [PATCH] AIFF fix in case of ANNO

2017-12-27 Thread Eduard Sinelnikov
From: Ubuntu --- libavformat/aiffdec.c | 5 ++--- 1 file changed, 2 insertions(+), 3 deletions(-) diff --git a/libavformat/aiffdec.c b/libavformat/aiffdec.c index 99e05c7..20decc5 100644 --- a/libavformat/aiffdec.c +++ b/libavformat/aiffdec.c @@ -81,11 +81,10 @@ static void get_meta(AVFormatCon

Re: [FFmpeg-devel] [PATCH] Add android_capture indev

2017-12-27 Thread Felix Matouschek
Ping > Am 22.12.2017 um 21:36 schrieb Felix Matouschek : > > Am 22.12.2017 20:50, schrieb Lou Logan: >> I think you forgot to attach the patch. > > Sorry, flaky mail client... attached it again. ___ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org htt

Re: [FFmpeg-devel] [PATCH v2 4/4] avformat/hls: enable http_multiple only for http/1.1 servers

2017-12-27 Thread Michael Niedermayer
On Wed, Dec 27, 2017 at 07:39:12PM +, Aman Gupta wrote: > On Wed, Dec 27, 2017 at 11:35 AM Michael Niedermayer > wrote: > > > On Mon, Dec 25, 2017 at 09:23:01PM -0800, Aman Gupta wrote: > > > From: Aman Gupta > > > > > > Signed-off-by: Aman Gupta > > > --- > > > doc/demuxers.texi | 2 +- >

Re: [FFmpeg-devel] [RFC][PATCH] simple_idct: Template functions to support an input bitdepth parameter

2017-12-27 Thread Michael Niedermayer
On Wed, Dec 27, 2017 at 01:11:56AM +, Kieran Kunhya wrote: > For MPEG-4 Simple Studio Profile, I need to be able to support int32_t > input coeffcients to the IDCT functions. I have attempted to implement this > with the attached patch. > Any comments would be appreciated, I'm pretty sure it is

Re: [FFmpeg-devel] [PATCH v2 4/4] avformat/hls: enable http_multiple only for http/1.1 servers

2017-12-27 Thread Aman Gupta
On Wed, Dec 27, 2017 at 11:35 AM Michael Niedermayer wrote: > On Mon, Dec 25, 2017 at 09:23:01PM -0800, Aman Gupta wrote: > > From: Aman Gupta > > > > Signed-off-by: Aman Gupta > > --- > > doc/demuxers.texi | 2 +- > > libavformat/hls.c | 13 ++--- > > 2 files changed, 11 insertions(+

Re: [FFmpeg-devel] [PATCH v2 4/4] avformat/hls: enable http_multiple only for http/1.1 servers

2017-12-27 Thread Michael Niedermayer
On Mon, Dec 25, 2017 at 09:23:01PM -0800, Aman Gupta wrote: > From: Aman Gupta > > Signed-off-by: Aman Gupta > --- > doc/demuxers.texi | 2 +- > libavformat/hls.c | 13 ++--- > 2 files changed, 11 insertions(+), 4 deletions(-) segfaults fate-filter-hls #0 0x703259a4 in ?? ()

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

2017-12-27 Thread Paul B Mahol
Signed-off-by: Paul B Mahol --- doc/filters.texi | 4 ++ libavfilter/Makefile | 1 + libavfilter/allfilters.c | 1 + libavfilter/vf_entropy.c | 174 +++ 4 files changed, 180 insertions(+) create mode 100644 libavfilter/vf_entropy.c d

Re: [FFmpeg-devel] [RFC][PATCH] simple_idct: Template functions to support an input bitdepth parameter

2017-12-27 Thread Michael Niedermayer
On Wed, Dec 27, 2017 at 01:11:56AM +, Kieran Kunhya wrote: > For MPEG-4 Simple Studio Profile, I need to be able to support int32_t > input coeffcients to the IDCT functions. I have attempted to implement this > with the attached patch. > Any comments would be appreciated, I'm pretty sure it is

Re: [FFmpeg-devel] [PATCH] Adding mkdir option for img2enc (2nd attempt)

2017-12-27 Thread Derek Buitenhuis
Hi, On 12/27/2017 12:27 PM, Dr Alan Barclay wrote: > Resending the two (git format-patch) patches, without the top lines > removed (which I thought I needed to do as some patch emails didn't seem > to have them). > > All comments and help appreciated. [...] > Subject: [PATCH 1/2] Move mkdir_p

Re: [FFmpeg-devel] [PATCH] added reitnerlace filter

2017-12-27 Thread Nicolas George
Vasile Toncu (2017-12-27): > It is because of licencing issues. I wanted the new filter to be LGPL. Thanks for the explanation. That is a valid reason. But as is, it would result would be duplicated code for people who do not worry about licensing. Are there any features that tinterlace has and

Re: [FFmpeg-devel] [PATCH] added reitnerlace filter

2017-12-27 Thread Vasile Toncu
> > The reinterlace filter does various interlace/interleave/merge operations > > between consecutive frames of a video. It tries too behave just like > > tinterlace filter, including some new modes. Why not add the new modes to the tinterlace filter, then? > It is because of licencing issues. I

Re: [FFmpeg-devel] [PATCH] added reitnerlace filter

2017-12-27 Thread Nicolas George
Vasile Toncu (2017-12-27): > The reinterlace filter does various interlace/interleave/merge operations > between consecutive frames of a video. It tries too behave just like > tinterlace filter, including some new modes. Why not add the new modes to the tinterlace filter, then? > On Wed, Dec 27,

Re: [FFmpeg-devel] [PATCH] added reitnerlace filter

2017-12-27 Thread Vasile Toncu
Hello, The reinterlace filter does various interlace/interleave/merge operations between consecutive frames of a video. It tries too behave just like tinterlace filter, including some new modes. On Wed, Dec 27, 2017 at 2:50 PM, Paul B Mahol wrote: > Hi, > > On 12/27/17, Vasile Toncu wrote: > >

Re: [FFmpeg-devel] [PATCH] added reitnerlace filter

2017-12-27 Thread Nicolas George
Thanks for the patch. Initial comments below. Vasile Toncu (2017-12-27): > --- > doc/filters.texi | 87 ++- > libavfilter/Makefile | 2 + > libavfilter/allfilters.c | 2 + > libavfilter/reinterlace.h| 130 ++ > libavfilter/vf_reinterlace.c | 597 > +++

Re: [FFmpeg-devel] [PATCH] added reitnerlace filter

2017-12-27 Thread Paul B Mahol
Hi, On 12/27/17, Vasile Toncu wrote: > --- > doc/filters.texi | 87 ++- > libavfilter/Makefile | 2 + > libavfilter/allfilters.c | 2 + > libavfilter/reinterlace.h| 130 ++ > libavfilter/vf_reinterlace.c | 597 > +++

[FFmpeg-devel] [PATCH] added reitnerlace filter

2017-12-27 Thread Vasile Toncu
--- doc/filters.texi | 87 ++- libavfilter/Makefile | 2 + libavfilter/allfilters.c | 2 + libavfilter/reinterlace.h| 130 ++ libavfilter/vf_reinterlace.c | 597 +++ 5 files changed, 812 insertions(+), 6 deletions

Re: [FFmpeg-devel] [PATCH] Adding mkdir option for img2enc (2nd attempt)

2017-12-27 Thread Dr Alan Barclay
Hi, Resending the two (git format-patch) patches, without the top lines removed (which I thought I needed to do as some patch emails didn't seem to have them). All comments and help appreciated. Thanks and Regards, Alan. On 27/12/17 01:41, Michael Niedermayer wrote: On Tue, Dec 26, 2017 a