[FFmpeg-devel] [PATCH] lavf/http: Fix incorrectly placed parenthesis.

2015-09-03 Thread Stephan Holljes
Signed-off-by: Stephan Holljes --- libavformat/http.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) The assignment had incorrectly placed parentheses which resulted in ret always being > 0. diff --git a/libavformat/http.c b/libavformat/http.c index

[FFmpeg-devel] [PATCH][RFC] libavformat/mov Extend metadata handling to read in the keys from the 'keys' atom

2015-09-03 Thread Kevin Wheatley
Looking for feedback on this patch, in particular the use of the 'keys' dictionary within the MOVContext structure. With the patch FFmpeg can now 'find' the metadata in ARRI's sample footage see here: http://www.arri.com/camera/alexa/learn/alexa_sample_footage/ Thanks Kevin From

Re: [FFmpeg-devel] [PATCH] lavf/http: Fix incorrectly placed parenthesis.

2015-09-03 Thread wm4
On Thu, 3 Sep 2015 15:55:10 +0200 Stephan Holljes wrote: > Signed-off-by: Stephan Holljes > --- > libavformat/http.c | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > The assignment had incorrectly placed parentheses which

[FFmpeg-devel] [PATCH 2/2] avfilter/vf_waveform: reduce number of operations a little

2015-09-03 Thread Paul B Mahol
Signed-off-by: Paul B Mahol --- libavfilter/vf_waveform.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/libavfilter/vf_waveform.c b/libavfilter/vf_waveform.c index dbf06a0..858c718 100644 --- a/libavfilter/vf_waveform.c +++ b/libavfilter/vf_waveform.c

Re: [FFmpeg-devel] [PATCH 3/3] avfilter/af_dynaudnorm: Fix typo in assert

2015-09-03 Thread Paul B Mahol
On 9/3/15, Michael Niedermayer wrote: > From: Michael Niedermayer > > Fixes: CID1322303 > > Signed-off-by: Michael Niedermayer > --- > libavfilter/af_dynaudnorm.c |2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) >

[FFmpeg-devel] [PATCH] libvpxdec: explicitly signal RGB.

2015-09-03 Thread Ronald S. Bultje
--- libavcodec/libvpxdec.c | 15 +++ 1 file changed, 15 insertions(+) diff --git a/libavcodec/libvpxdec.c b/libavcodec/libvpxdec.c index 52a89ca..eb05f26 100644 --- a/libavcodec/libvpxdec.c +++ b/libavcodec/libvpxdec.c @@ -90,7 +90,12 @@ static int set_pix_fmt(AVCodecContext *avctx,

Re: [FFmpeg-devel] [PATCH 2/3] avfilter/avf_showfreqs: Use floating point division in WFUNC_BHANN

2015-09-03 Thread Paul B Mahol
On 9/3/15, Michael Niedermayer wrote: > From: Michael Niedermayer > > Fixes: CID1322365 > > Signed-off-by: Michael Niedermayer > --- > libavfilter/avf_showfreqs.c |2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) >

[FFmpeg-devel] [PATCH] lavf/http: Remove superfluous parenthesis.

2015-09-03 Thread Stephan Holljes
Signed-off-by: Stephan Holljes --- libavformat/http.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/libavformat/http.c b/libavformat/http.c index c148a29..20e942e 100644 --- a/libavformat/http.c +++ b/libavformat/http.c @@ -401,7 +401,7 @@ static

Re: [FFmpeg-devel] [PATCH] lavf/http: Remove superfluous parenthesis.

2015-09-03 Thread Ganesh Ajjanagadde
On Thu, Sep 3, 2015 at 6:56 AM, Stephan Holljes wrote: > Signed-off-by: Stephan Holljes > --- > libavformat/http.c | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/libavformat/http.c b/libavformat/http.c > index

Re: [FFmpeg-devel] [PATCH 1/3] avformat/concatdec: Check file variable before dereferencing

2015-09-03 Thread Ganesh Ajjanagadde
On Thu, Sep 3, 2015 at 6:03 AM, Michael Niedermayer wrote: > From: Michael Niedermayer > > Fixes CID1322328 > > Signed-off-by: Michael Niedermayer > --- > libavformat/concatdec.c |5 + > 1 file changed, 5 insertions(+) >

[FFmpeg-devel] [PATCH] avfilter/vf_transpose: use the name 's' for the pointer to the private context

2015-09-03 Thread Ganesh Ajjanagadde
Signed-off-by: Ganesh Ajjanagadde --- libavfilter/vf_transpose.c | 46 +++--- 1 file changed, 23 insertions(+), 23 deletions(-) diff --git a/libavfilter/vf_transpose.c b/libavfilter/vf_transpose.c index e4de31b..484b8e8 100644 ---

Re: [FFmpeg-devel] [PATCH 2/2] avfilter/vf_waveform: reduce number of operations a little

2015-09-03 Thread Ganesh Ajjanagadde
On Thu, Sep 3, 2015 at 7:32 AM, Paul B Mahol wrote: > Signed-off-by: Paul B Mahol > --- > libavfilter/vf_waveform.c | 4 ++-- > 1 file changed, 2 insertions(+), 2 deletions(-) > > diff --git a/libavfilter/vf_waveform.c b/libavfilter/vf_waveform.c > index

Re: [FFmpeg-devel] [PATCH] avfilter/avf_showwaves: Check max_samples

2015-09-03 Thread Ganesh Ajjanagadde
On Thu, Sep 3, 2015 at 5:52 AM, Michael Niedermayer wrote: > From: Michael Niedermayer > > Fixes potential division by zero > Fixes: CID1292295 > > Signed-off-by: Michael Niedermayer > --- > libavfilter/avf_showwaves.c |5

Re: [FFmpeg-devel] MIPS and PNS

2015-09-03 Thread Claudio Freire
I'm also looking into it. It's simply that it tends to get out of sync with the rest really. I will remove some duplicated code (namely the twoloop and trellis functions which are verbatim copies of their counterparts in aaccoder), while somehow keeping the fact that they used the optimized

Re: [FFmpeg-devel] [PATCH] libvpxdec: explicitly signal RGB.

2015-09-03 Thread James Zern
On Thu, Sep 3, 2015 at 6:38 AM, Ronald S. Bultje wrote: > --- > libavcodec/libvpxdec.c | 15 +++ > 1 file changed, 15 insertions(+) > lgtm > diff --git a/libavcodec/libvpxdec.c b/libavcodec/libvpxdec.c > index 52a89ca..eb05f26 100644 > ---

Re: [FFmpeg-devel] [PATCH] lavf/http: Fix incorrectly placed parenthesis.

2015-09-03 Thread Michael Niedermayer
On Thu, Sep 03, 2015 at 04:19:06PM +0200, wm4 wrote: > On Thu, 3 Sep 2015 15:55:10 +0200 > Stephan Holljes wrote: > > > Signed-off-by: Stephan Holljes > > --- > > libavformat/http.c | 2 +- > > 1 file changed, 1 insertion(+), 1 deletion(-) >

Re: [FFmpeg-devel] [PATCH]Remove experimental flag from j2k encoder

2015-09-03 Thread Michael Niedermayer
On Thu, Sep 03, 2015 at 12:38:40PM +, Carl Eugen Hoyos wrote: > Carl Eugen Hoyos ag.or.at> writes: > > > The encoder produces conforming and - if requested - > > lossless files, it should not be marked experimental imo. > > Ping? no objections from me [...] -- Michael GnuPG

Re: [FFmpeg-devel] [PATCH] lavf/http: Remove superfluous parenthesis.

2015-09-03 Thread Michael Niedermayer
On Thu, Sep 03, 2015 at 08:00:58AM -0700, Ganesh Ajjanagadde wrote: > On Thu, Sep 3, 2015 at 6:56 AM, Stephan Holljes > wrote: > > Signed-off-by: Stephan Holljes > > --- > > libavformat/http.c | 2 +- > > 1 file changed, 1 insertion(+), 1

Re: [FFmpeg-devel] Help for patch: Sending last full, non-corrupt frame to output

2015-09-03 Thread Carl Eugen Hoyos
Chris Wiggins chriswiggins.co.nz> writes: > Should I just disable threads when I'm testing to > see if this alleviates the problem? I would expect that the output frame looks different with single-threaded decoding. In any case, a real-world h264 stream may help improving the issue. Carl

Re: [FFmpeg-devel] [PATCH]Remove experimental flag from j2k encoder

2015-09-03 Thread Ganesh Ajjanagadde
On Thu, Sep 3, 2015 at 10:57 AM, Michael Niedermayer wrote: > On Thu, Sep 03, 2015 at 12:38:40PM +, Carl Eugen Hoyos wrote: >> Carl Eugen Hoyos ag.or.at> writes: >> >> > The encoder produces conforming and - if requested - >> > lossless files, it should not be marked

Re: [FFmpeg-devel] [PATCH] vp9: reset segmentation information on context reset.

2015-09-03 Thread James Zern
On Tue, Sep 1, 2015 at 1:44 PM, Ronald S. Bultje wrote: > --- > libavcodec/vp9.c | 1 + > 1 file changed, 1 insertion(+) > lgtm > diff --git a/libavcodec/vp9.c b/libavcodec/vp9.c > index 4503ed7..7035512 100644 > --- a/libavcodec/vp9.c > +++ b/libavcodec/vp9.c > @@ -707,6

Re: [FFmpeg-devel] [PATCH]Remove experimental flag from j2k encoder

2015-09-03 Thread Carl Eugen Hoyos
Carl Eugen Hoyos ag.or.at> writes: > I completely agree with you that marking the > jpeg2000 decoder might have been premature! ... marking it non-experimental... Carl Eugen ___ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org

Re: [FFmpeg-devel] [PATCH]Remove experimental flag from j2k encoder

2015-09-03 Thread Carl Eugen Hoyos
Ganesh Ajjanagadde mit.edu> writes: > >> > The encoder produces conforming and - if requested - > >> > lossless files, it should not be marked experimental imo. > Sorry for the late comment, but I see quite a > few tickets related to jpeg2000 on trac (e.g > 4653, 4679, 4669, 4601). You

Re: [FFmpeg-devel] [PATCH]Remove probesize32 and max_analyze_duration32 on version bump

2015-09-03 Thread James Almer
On 8/31/2015 6:41 AM, Carl Eugen Hoyos wrote: > diff --git a/libavformat/avformat.h b/libavformat/avformat.h > index e83a374..4068ab6 100644 > --- a/libavformat/avformat.h > +++ b/libavformat/avformat.h > @@ -1396,6 +1396,7 @@ typedef struct AVFormatContext { > #define AVFMT_FLAG_KEEP_SIDE_DATA

[FFmpeg-devel] [PATCH] vp9_parse: fix parsing of pskip and profile 2/3.

2015-09-03 Thread Ronald S. Bultje
--- libavcodec/vp9_parser.c | 29 - 1 file changed, 24 insertions(+), 5 deletions(-) diff --git a/libavcodec/vp9_parser.c b/libavcodec/vp9_parser.c index ab33c33..a880570 100644 --- a/libavcodec/vp9_parser.c +++ b/libavcodec/vp9_parser.c @@ -22,6 +22,7 @@ */

Re: [FFmpeg-devel] [PATCH][RFC] libavformat/mov Extend metadata handling to read in the keys from the 'keys' atom

2015-09-03 Thread Michael Niedermayer
On Thu, Sep 03, 2015 at 04:04:53PM +0100, Kevin Wheatley wrote: > Looking for feedback on this patch, in particular the use of the > 'keys' dictionary within the MOVContext structure. > > With the patch FFmpeg can now 'find' the metadata in ARRI's sample > footage see here: > >

[FFmpeg-devel] [PATCH] swscale: re enable gamma conversion

2015-09-03 Thread Pedro Arthur
Hi, This patch implement gamma conversion in the refactored scaling code, and re-enable it in old code. If it is good let me know and I'll push it. Thanks, Pedro. From 4ac30e1e08cbe91f88a2d9efd1a100429e387112 Mon Sep 17 00:00:00 2001 From: Pedro Arthur Date: Thu, 3 Sep 2015

[FFmpeg-devel] Sample (libavcodec/h264_sei.c): Subtitles with data type 0x06

2015-09-03 Thread Moritz Barsnick
Hi, I have uploaded ffmpeg.sample.subtitles_with_data_type_0x06.ts to ftp://upload.ffmpeg.org/incoming/. It exposes the request "[h264 @ 0xb25bb60] Subtitles with data type 0x06 is not implemented.", as issued in libavcodec/h264_sei.c:

Re: [FFmpeg-devel] [PATCH 3/3] avfilter/af_dynaudnorm: Fix typo in assert

2015-09-03 Thread Michael Niedermayer
On Thu, Sep 03, 2015 at 01:37:38PM +, Paul B Mahol wrote: > On 9/3/15, Michael Niedermayer wrote: > > From: Michael Niedermayer > > > > Fixes: CID1322303 > > > > Signed-off-by: Michael Niedermayer > > --- > >

Re: [FFmpeg-devel] [PATCH] swscale: re enable gamma conversion

2015-09-03 Thread compn
On Thu, 3 Sep 2015 13:01:34 -0300 Pedro Arthur wrote: > Hi, > > This patch implement gamma conversion in the refactored scaling code, > and re-enable it in old code. > If it is good let me know and I'll push it. did you run any benchmarks? :) -compn

Re: [FFmpeg-devel] [PATCH 1/3] avformat/concatdec: Check file variable before dereferencing

2015-09-03 Thread Nicolas George
Le septidi 17 fructidor, an CCXXIII, Michael Niedermayer a écrit : > From: Michael Niedermayer > > Fixes CID1322328 > > Signed-off-by: Michael Niedermayer > --- > libavformat/concatdec.c |5 + > 1 file changed, 5 insertions(+) Ok, of

Re: [FFmpeg-devel] [PATCH 1/3] avformat/concatdec: Check file variable before dereferencing

2015-09-03 Thread Michael Niedermayer
On Thu, Sep 03, 2015 at 03:03:25PM +0200, Michael Niedermayer wrote: > From: Michael Niedermayer > > Fixes CID1322328 > > Signed-off-by: Michael Niedermayer > --- > libavformat/concatdec.c |5 + > 1 file changed, 5 insertions(+) applied

Re: [FFmpeg-devel] [PATCH] avutil/common: Add FFNABS()

2015-09-03 Thread Michael Niedermayer
On Wed, Sep 02, 2015 at 05:33:04PM -0700, Ganesh Ajjanagadde wrote: > On Wed, Sep 2, 2015 at 5:17 PM, Michael Niedermayer wrote: > > From: Michael Niedermayer > > > > This function avoids the undefined corner cases which affects other absolute > > value

Re: [FFmpeg-devel] extract and change pixels of AVFrame

2015-09-03 Thread Paul B Mahol
Dana 3. 9. 2015. 03:13 osoba "Mehdi Hosseini" napisala je: > > Hello, > > I want to do some pixel operations on a decoded frame (AVFrame structure) > in function "ff_thread_decode_frame" of the file > "ffmpeg/libavcodec/pthread.c". > > For example. I wrote a simple code

Re: [FFmpeg-devel] MIPS and PNS

2015-09-03 Thread Nedeljko Babic
Hi, Thanks Michael. I'll look in to it. -Nedeljko Od: Michael Niedermayer [michae...@gmx.at] Poslato: 2. septembar 2015 13:11 Za: FFmpeg development discussions and patches Cc: Nedeljko Babic Tema: MIPS and PNS Hi Enabling pns by default

Re: [FFmpeg-devel] [PATCH] avfilter/vf_sab: use the name 's' for the pointer to the private context

2015-09-03 Thread Paul B Mahol
On 9/1/15, Ganesh Ajjanagadde wrote: > Signed-off-by: Ganesh Ajjanagadde > --- > libavfilter/vf_sab.c | 58 > ++-- > 1 file changed, 29 insertions(+), 29 deletions(-) > applied

[FFmpeg-devel] Merging the major bump from Libav

2015-09-03 Thread Hendrik Leppkes
Hey, I plan to finally merge the major bump from Libav this weekend. I just wanted to give everyone a heads-up in case they didn't actually notice that its coming, or are waiting for it to happen. If anyone has some changes that require a bump (ABI changes), you still have a few days to prepare

[FFmpeg-devel] [PATCH] avfilter/vf_stereo3d: add checkerboard output format

2015-09-03 Thread Paul B Mahol
Signed-off-by: Paul B Mahol --- doc/filters.texi | 6 ++ libavfilter/vf_stereo3d.c | 50 ++- 2 files changed, 55 insertions(+), 1 deletion(-) diff --git a/doc/filters.texi b/doc/filters.texi index 162958f..cdd73fc

Re: [FFmpeg-devel] [PATCH]doc: Improve fieldmatch documentation

2015-09-03 Thread Nicholas Robbins
> On Wednesday, September 2, 2015 5:43 PM, Carl Eugen Hoyos > wrote: > I didn't test because it works fine afaict. Great >> Or are you offering a suggestion for mixed 24fps >> progressive and 24->30 telecined? > > Yes. > >> In that case it seems like this should

Re: [FFmpeg-devel] [PATCH] avfilter/af_amerge: avoid undefined shift (<<64) in outlayout setup

2015-09-03 Thread Ganesh Ajjanagadde
On Thu, Sep 3, 2015 at 3:33 PM, Michael Niedermayer wrote: > From: Michael Niedermayer > > Fixes CID1322306 > > Signed-off-by: Michael Niedermayer > --- > libavfilter/af_amerge.c |4 ++-- > 1 file changed, 2 insertions(+), 2

[FFmpeg-devel] [PATCH] avfilter/af_amerge: avoid undefined shift (<<64) in outlayout setup

2015-09-03 Thread Michael Niedermayer
From: Michael Niedermayer Fixes CID1322306 Signed-off-by: Michael Niedermayer --- libavfilter/af_amerge.c |4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/libavfilter/af_amerge.c b/libavfilter/af_amerge.c index

Re: [FFmpeg-devel] [PATCH] swscale: re enable gamma conversion

2015-09-03 Thread compn
On Thu, 3 Sep 2015 23:02:03 -0300 Pedro Arthur wrote: > did you run any benchmarks? :) > > > No, but it is only enabled explicitly by flag, so it can only penalize > performance if the user explicitly enables it. no objections from me then, thank you. -compn

Re: [FFmpeg-devel] [PATCH] Extract FFV1 FATE tests to own Makefile, independent of vcodec.mak

2015-09-03 Thread Peter B.
I've now combined my changes into one patch. They're basically moving ffv1 tests to ffv1.mak, and making that independent of vcodec.mak. Hope it works now :) Thanks and regards, Pb >From 5ba0e0510269e0ee591a9505fcb4654044002743 Mon Sep 17 00:00:00 2001 From: Peter B.

Re: [FFmpeg-devel] [PATCH]doc: Improve fieldmatch documentation

2015-09-03 Thread Nicholas Robbins
> On Thursday, September 3, 2015 8:45 AM, Carl Eugen Hoyos > wrote: > > Nicholas Robbins ffmpeg.org> writes: > >> >> You are using dejudder & fps to produce a stream of >> >> frames lies ABCDDEFGHH... then decimate drops the >> >> dups. Seems brutal. >> > >> >

Re: [FFmpeg-devel] [PATCH]doc: Improve fieldmatch documentation

2015-09-03 Thread Carl Eugen Hoyos
Nicholas Robbins ffmpeg.org> writes: > >> You are using dejudder & fps to produce a stream of > >> frames lies ABCDDEFGHH... then decimate drops the > >> dups. Seems brutal. > > > > Why / how? > > It just seems strange to me to make those extra frames > just to throw them away. Seems

Re: [FFmpeg-devel] [PATCH] avcodec: Switch bitrate to 64bit with the bump

2015-09-03 Thread Carl Eugen Hoyos
Michael Niedermayer gmx.at> writes: > a 32bit bitrate is insufficient for high resolution, > high framerate material an example would be rawvideo Please mention ticket #2089 if it is fixed by this commit. Thanks, Carl Eugen ___ ffmpeg-devel mailing

[FFmpeg-devel] [PATCH] avfilter/avf_showwaves: Check max_samples

2015-09-03 Thread Michael Niedermayer
From: Michael Niedermayer Fixes potential division by zero Fixes: CID1292295 Signed-off-by: Michael Niedermayer --- libavfilter/avf_showwaves.c |5 + 1 file changed, 5 insertions(+) diff --git a/libavfilter/avf_showwaves.c

[FFmpeg-devel] [PATCH 1/3] avformat/concatdec: Check file variable before dereferencing

2015-09-03 Thread Michael Niedermayer
From: Michael Niedermayer Fixes CID1322328 Signed-off-by: Michael Niedermayer --- libavformat/concatdec.c |5 + 1 file changed, 5 insertions(+) diff --git a/libavformat/concatdec.c b/libavformat/concatdec.c index 969b001..88b6dbe 100644

[FFmpeg-devel] [PATCH 3/3] avfilter/af_dynaudnorm: Fix typo in assert

2015-09-03 Thread Michael Niedermayer
From: Michael Niedermayer Fixes: CID1322303 Signed-off-by: Michael Niedermayer --- libavfilter/af_dynaudnorm.c |2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/libavfilter/af_dynaudnorm.c b/libavfilter/af_dynaudnorm.c index

[FFmpeg-devel] [PATCH 2/3] avfilter/avf_showfreqs: Use floating point division in WFUNC_BHANN

2015-09-03 Thread Michael Niedermayer
From: Michael Niedermayer Fixes: CID1322365 Signed-off-by: Michael Niedermayer --- libavfilter/avf_showfreqs.c |2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/libavfilter/avf_showfreqs.c b/libavfilter/avf_showfreqs.c index

Re: [FFmpeg-devel] Merging the major bump from Libav

2015-09-03 Thread wm4
On Thu, 3 Sep 2015 13:57:13 +0200 Hendrik Leppkes wrote: > Hey, > > I plan to finally merge the major bump from Libav this weekend. > I just wanted to give everyone a heads-up in case they didn't actually > notice that its coming, or are waiting for it to happen. +1 > If

Re: [FFmpeg-devel] [PATCH]Remove experimental flag from j2k encoder

2015-09-03 Thread Carl Eugen Hoyos
Carl Eugen Hoyos ag.or.at> writes: > The encoder produces conforming and - if requested - > lossless files, it should not be marked experimental imo. Ping? > Why is memory consumption a specific issue for this encoder? I still consider this a weak argument. Carl Eugen

[FFmpeg-devel] [PATCH]lavc/dnxhdenc: Fix setting intra_quant_bias for library users

2015-09-03 Thread Carl Eugen Hoyos
Hi! I suspect I forgot attached in 7f9656f10df564a3d9b913420c7aca0e24f17f82 Please review, Carl Eugen diff --git a/libavcodec/dnxhdenc.c b/libavcodec/dnxhdenc.c index af84399..3ede1ed 100644 --- a/libavcodec/dnxhdenc.c +++ b/libavcodec/dnxhdenc.c @@ -360,7 +360,7 @@ static av_cold int

Re: [FFmpeg-devel] [PATCH] swscale: re enable gamma conversion

2015-09-03 Thread Pedro Arthur
did you run any benchmarks? :) > No, but it is only enabled explicitly by flag, so it can only penalize performance if the user explicitly enables it. ___ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org http://ffmpeg.org/mailman/listinfo/ffmpeg-devel