Re: [FFmpeg-devel] [PATCH 2/2] x86/vf_w3fdif: simplify w3fdif_simple_high

2015-10-11 Thread Paul B Mahol
On 10/11/15, James Almer wrote: > Signed-off-by: James Almer > --- > libavfilter/x86/vf_w3fdif.asm | 16 +++- > 1 file changed, 7 insertions(+), 9 deletions(-) > > diff --git a/libavfilter/x86/vf_w3fdif.asm b/libavfilter/x86/vf_w3fdif.asm >

Re: [FFmpeg-devel] [FFmpeg-cvslog] avconv: Do not try to configure filter outputs without streams

2015-10-11 Thread Michael Niedermayer
On Sat, Oct 10, 2015 at 09:31:05AM -0400, Ganesh Ajjanagadde wrote: > On Sat, Oct 10, 2015 at 9:18 AM, Michael Niedermayer > wrote: > > Hi > > > > On Sat, Oct 10, 2015 at 09:43:55AM +0200, Luca Barbato wrote: > >> ffmpeg | branch: master | Luca Barbato

Re: [FFmpeg-devel] [PATCH] avfilter/WIP: add selectivecolor filter

2015-10-11 Thread Clément Bœsch
On Sat, Aug 15, 2015 at 07:02:11PM +0200, Clément Bœsch wrote: > --- > Only K adjustment remains to fix before it's ready for a first upstream > version. Help figuring out is still welcome. Otherwise, since this is very > close to the final version, review is welcome as well. > --- >

Re: [FFmpeg-devel] [PATCH] os2threads: Add pthread_once emulation

2015-10-11 Thread Hendrik Leppkes
On Sun, Oct 11, 2015 at 4:32 AM, Dave Yeo wrote: > Based on code by Yuri Dario, http://svn.netlabs.org/repos/ports/pthread/trunk > > Signed-off-by: Dave Yeo > --- > compat/os2threads.h | 45 + > 1 file changed,

Re: [FFmpeg-devel] [RFC] avx2 and scalarproduct

2015-10-11 Thread Kieran Kunhya
On 9 Oct 2015 10:13 pm, "Paul B Mahol" wrote: > > diff --git a/libavcodec/x86/audiodsp.asm b/libavcodec/x86/audiodsp.asm > index 3ffb27f..246e945 100644 > --- a/libavcodec/x86/audiodsp.asm > +++ b/libavcodec/x86/audiodsp.asm > @@ -41,7 +41,14 @@ cglobal scalarproduct_int16,

Re: [FFmpeg-devel] [RFC] avx2 and scalarproduct

2015-10-11 Thread Ronald S. Bultje
Hi Kieran, On Sun, Oct 11, 2015 at 5:02 AM, Kieran Kunhya wrote: > On 9 Oct 2015 10:13 pm, "Paul B Mahol" wrote: > > > > diff --git a/libavcodec/x86/audiodsp.asm b/libavcodec/x86/audiodsp.asm > > index 3ffb27f..246e945 100644 > > ---

Re: [FFmpeg-devel] [PATCH] os2threads: Add pthread_once emulation

2015-10-11 Thread Ronald S. Bultje
Hi, On Sat, Oct 10, 2015 at 10:32 PM, Dave Yeo wrote: > Based on code by Yuri Dario, > http://svn.netlabs.org/repos/ports/pthread/trunk > > Signed-off-by: Dave Yeo > --- > compat/os2threads.h | 45 + > 1 file

Re: [FFmpeg-devel] av_ctz optimization

2015-10-11 Thread Ronald S. Bultje
Hi, On Sat, Oct 10, 2015 at 10:50 PM, James Almer wrote: > On 10/10/2015 11:44 PM, Ganesh Ajjanagadde wrote: > > Hi all, > > > > Turns out that the De-Bruijn method can be used to speed up av_ctz as > > well. But before going about it, I was curious as to why such an > >

Re: [FFmpeg-devel] [PATCH] vp9: add 10/12bpp sse2 SIMD version for idct_idct_16x16.

2015-10-11 Thread Ronald S. Bultje
Hi, On Thu, Oct 8, 2015 at 10:19 AM, Ronald S. Bultje wrote: > Hi, > > On Thu, Oct 8, 2015 at 10:16 AM, Ronald S. Bultje > wrote: > >> --- >> libavcodec/x86/vp9dsp_init_16bpp_template.c | 2 + >> libavcodec/x86/vp9itxfm_16bpp.asm | 237 >>

[FFmpeg-devel] Patch for Ticket 4922

2015-10-11 Thread Ganesh Ajjanagadde
Hi all, Bug report from user: https://trac.ffmpeg.org/ticket/4922, and associated patch has been attached. The patch LGTM, except for the commit message, which should be changed to "configure: fix configure when using gcc". This should be backported. Apologies for the breakage. Regards, Ganesh

Re: [FFmpeg-devel] [PATCHv2] avutil/mathematics: speed up av_gcd by using Stein's binary GCD algorithm

2015-10-11 Thread wm4
On Sat, 10 Oct 2015 21:58:47 -0400 Ganesh Ajjanagadde wrote: > This uses Stein's binary GCD algorithm: > https://en.wikipedia.org/wiki/Binary_GCD_algorithm > to get a roughly 4x speedup over Euclidean GCD on standard architectures > with a compiler intrinsic for ctzll,

Re: [FFmpeg-devel] [PATCH] avcodec/webvttdec: Deal with WebVTT escapes

2015-10-11 Thread Clément Bœsch
On Fri, Oct 09, 2015 at 04:29:03PM +0100, Ricardo Constantino wrote: > Bare ampersand characters are still accepted, even though out-of-spec. > Also fixes adjacent tags not being parsed. > > Fixes trac #4915 > > Signed-off-by: Ricardo Constantino > --- >

Re: [FFmpeg-devel] Patch for Ticket 4922

2015-10-11 Thread Ganesh Ajjanagadde
On Sun, Oct 11, 2015 at 9:26 AM, Ganesh Ajjanagadde wrote: > Hi all, > > Bug report from user: https://trac.ffmpeg.org/ticket/4922, and > associated patch has been attached. The patch LGTM, except for the > commit message, which should be changed to "configure: fix configure >

Re: [FFmpeg-devel] [PATCH] avcodec/webvttdec: Deal with WebVTT escapes

2015-10-11 Thread Ricardo Constantino
On 11 October 2015 at 15:37, Clément Bœsch wrote: > nbsp → \h in ASS nevcairiel suggested not relying on \h being parsed as nbsp in .srt with VSFilter and Libass, but maybe using the unicode char itself. Unless FFmpeg converts \h to the char code in srtenc or something? Aegisub

Re: [FFmpeg-devel] [PATCH 3/5] lavu/jni: add helpers to manage android application contexts

2015-10-11 Thread Ronald S. Bultje
Hi, On Fri, Oct 9, 2015 at 12:26 PM, Matthieu Bouron wrote: > From: Matthieu Bouron > > --- > libavutil/jni.c | 104 > +++ > libavutil/jni.h | 28 + >

[FFmpeg-devel] [PATCH 9/9] simple_idct: align with x86 version

2015-10-11 Thread Christophe Gisquet
This is slightly beneficial to omse (0.1691 to 0.1688). The x86 idct is thus marked as a valid simple implementation. --- libavcodec/simple_idct_template.c | 15 +++ libavcodec/x86/idctdsp_init.c | 8 +++- 2 files changed, 6 insertions(+), 17 deletions(-) diff --git

[FFmpeg-devel] [PATCH 8/9] x86: dct-test: add more idcts

2015-10-11 Thread Christophe Gisquet
In particular for 10 and 12 bits. --- libavcodec/dct-test.c | 2 ++ libavcodec/x86/dct-test.c | 10 ++ 2 files changed, 12 insertions(+) diff --git a/libavcodec/dct-test.c b/libavcodec/dct-test.c index 56e1a62..72fe353 100644 --- a/libavcodec/dct-test.c +++ b/libavcodec/dct-test.c

[FFmpeg-devel] [PATCH 5/9] x86: proresdsp: simple_idct: free or use 1 xmm reg

2015-10-11 Thread Christophe Gisquet
m15 is zeroed but never used. If it's not needed, decrease by 1 the number of xmm regs used (prores), otherwise, make use of it, for the rounder in the row pass of simple_idct. --- libavcodec/x86/proresdsp.asm | 8 libavcodec/x86/simple_idct10.asm | 9 +

[FFmpeg-devel] [PATCH 2/9] x86: prores: templatize 10 bits simple_idct

2015-10-11 Thread Christophe Gisquet
This should be reused for a generic simple_idct10 function. Requires a bit of trickery to declare common constants in C. --- libavcodec/x86/constants.c| 28 +++ libavcodec/x86/constants.h| 16 ++ libavcodec/x86/proresdsp.asm | 263

[FFmpeg-devel] [PATCH 6/9] x86: simple_idct: add pure idct functions

2015-10-11 Thread Christophe Gisquet
This is done by not passing clamping regs to the macro, and there, deducing that this is a pure idct. There are not many users (vf_spp is one), and the _add version doesn't look like it will ever be used. --- libavcodec/x86/idctdsp_init.c | 4 +- libavcodec/x86/proresdsp.asm

[FFmpeg-devel] [PATCH 1/9] dnxhdenc: fix scan used for bitstream generation

2015-10-11 Thread Christophe Gisquet
The functions related to bitstream reading must use the natural zigzag order, and not the one permuted for use in the iDCT. This currently results in bitstreams with 2 issues: - AC coefficients are encoded in an unexpected order; - Incorrect weights are applied in the (de)quantization. This

[FFmpeg-devel] [PATCH 3/9] simple_idct10: improve precision

2015-10-11 Thread Christophe Gisquet
omse goes from 0.03060703 (which fails for dct-test) to 0.01663750. --- libavcodec/simple_idct.c | 9 -- libavcodec/simple_idct_template.c | 45 ++- tests/ref/fate/dnxhr-444 | 2 +-

[FFmpeg-devel] [PATCH 4/9] x86: simple_idct_put: 10bits versions

2015-10-11 Thread Christophe Gisquet
Modeled from the prores version. Clips to [0;1023] and is bitexact. Bitexactness requires to add an offset in a different place compared to prores or C, and makes the function approximately 2% slower. For 16 frames of a DNxHD 4:2:2 10bits test sequence: C:60861 decicycles in idct, 1048205

[FFmpeg-devel] [PATCH 7/9] x86: simple_idct: 12bits versions

2015-10-11 Thread Christophe Gisquet
On 12 frames of a 444p 12 bits DNxHR sequence, _put function: C: 78902 decicycles in idct, 262071 runs, 73 skips avx: 32478 decicycles in idct, 262045 runs, 99 skips Difference between the 2: stddev:0.39 PSNR:104.47 MAXDIFF:2 This is unavoidable and due to the

[FFmpeg-devel] [PATCH] configure: fix configure when using gcc

2015-10-11 Thread Ganesh Ajjanagadde
Fixes Ticket4922. Commit 060102389e572abb2beaed3b9f5e1036aeea43f1 broke configure. From: Jean-Yves Avenard Signed-off-by: Ganesh Ajjanagadde --- configure | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/configure b/configure

Re: [FFmpeg-devel] [PATCH] avcodec/webvttdec: Deal with WebVTT escapes

2015-10-11 Thread Ricardo Constantino
Ok, nevermind. I see the srt and webvttenc samples use \h already, so that's what you'll want. On 11 October 2015 at 16:53, Ricardo Constantino wrote: > > On 11 October 2015 at 15:37, Clément Bœsch wrote: > >> nbsp → \h in ASS > > > nevcairiel suggested not

Re: [FFmpeg-devel] av_ctz optimization

2015-10-11 Thread James Almer
On 10/11/2015 8:01 AM, Ronald S. Bultje wrote: > Hi, > > On Sat, Oct 10, 2015 at 10:50 PM, James Almer wrote: > >> On 10/10/2015 11:44 PM, Ganesh Ajjanagadde wrote: >>> Hi all, >>> >>> Turns out that the De-Bruijn method can be used to speed up av_ctz as >>> well. But before

Re: [FFmpeg-devel] Patch for Ticket 4922

2015-10-11 Thread Ganesh Ajjanagadde
On Sun, Oct 11, 2015 at 11:22 AM, Ganesh Ajjanagadde wrote: > On Sun, Oct 11, 2015 at 9:26 AM, Ganesh Ajjanagadde wrote: >> Hi all, >> >> Bug report from user: https://trac.ffmpeg.org/ticket/4922, and >> associated patch has been attached. The patch LGTM,

Re: [FFmpeg-devel] [PATCH 3/3] fate/subtitles: Add a new test for WebVTT

2015-10-11 Thread Ricardo Constantino
Attached is a new version of the sample with the missing non-breaking spaces tests. WebVTT_extended_tester.vtt Description: Binary data ___ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org http://ffmpeg.org/mailman/listinfo/ffmpeg-devel

Re: [FFmpeg-devel] [PATCH] configure: fix configure when using gcc

2015-10-11 Thread Nicolas George
Le decadi 20 vendémiaire, an CCXXIV, Ganesh Ajjanagadde a écrit : > I am not so sure, there was a reason for it documented in the > 060102389e572abb2beaed3b9f5e1036aeea43f1 commit message. Indeed. We have until gcc version 20 to find a solution. Regards, -- Nicolas George

Re: [FFmpeg-devel] av_ctz optimization

2015-10-11 Thread Timothy Gu
On Sun, Oct 11, 2015 at 9:10 AM Ganesh Ajjanagadde wrote: > Isn't that an API change and not an ABI change (it was in doc/APIchanges)? > No, since intmath.h is not public. Timothy ___ ffmpeg-devel mailing list

[FFmpeg-devel] [PATCH] avcodec/h264: remove redundant and bogus get_format call

2015-10-11 Thread wm4
The AVCodecContext.get_format callback is not only used for pixel format negotiation with the API user, but also for hwaccel init. For the latter, it's required that some codec parameters, in particular the codec profile, are set when the callback is invoked. This patch removes a get_format

Re: [FFmpeg-devel] [PATCH 2/2] x86/vf_w3fdif: simplify w3fdif_simple_high

2015-10-11 Thread James Almer
On 10/11/2015 4:31 AM, Paul B Mahol wrote: > On 10/11/15, James Almer wrote: >> Signed-off-by: James Almer >> --- >> libavfilter/x86/vf_w3fdif.asm | 16 +++- >> 1 file changed, 7 insertions(+), 9 deletions(-) >> >> diff --git

Re: [FFmpeg-devel] setting up a proper Github mirror

2015-10-11 Thread Timothy Gu
On Sun, Oct 11, 2015 at 11:44 AM Ganesh Ajjanagadde wrote: > I noticed that the Github mirror: https://github.com/FFmpeg/FFmpeg is > over 3 hours out of sync with the main repos, making it unusable as a > fetch url for development. Anyone knows why this is the case? > It is

[FFmpeg-devel] [PATCH 2/2] avcodec/h264_slice: Never call get_format() for just checking if the format matches

2015-10-11 Thread Michael Niedermayer
From: Michael Niedermayer Signed-off-by: Michael Niedermayer --- libavcodec/h264_slice.c |4 1 file changed, 4 insertions(+) diff --git a/libavcodec/h264_slice.c b/libavcodec/h264_slice.c index cce97d9..daa3737 100644 ---

Re: [FFmpeg-devel] av_ctz optimization

2015-10-11 Thread Ganesh Ajjanagadde
On Sun, Oct 11, 2015 at 12:06 PM, James Almer wrote: > On 10/11/2015 8:01 AM, Ronald S. Bultje wrote: >> Hi, >> >> On Sat, Oct 10, 2015 at 10:50 PM, James Almer wrote: >> >>> On 10/10/2015 11:44 PM, Ganesh Ajjanagadde wrote: Hi all, Turns out

Re: [FFmpeg-devel] [PATCH] configure: fix configure when using gcc

2015-10-11 Thread Ganesh Ajjanagadde
On Sun, Oct 11, 2015 at 12:15 PM, Nicolas George wrote: > Le decadi 20 vendémiaire, an CCXXIV, Ganesh Ajjanagadde a écrit : >> Fixes Ticket4922. >> >> Commit 060102389e572abb2beaed3b9f5e1036aeea43f1 broke configure. > > The commit message could be a little more explicit: it

Re: [FFmpeg-devel] [PATCHv2] avutil/mathematics: speed up av_gcd by using Stein's binary GCD algorithm

2015-10-11 Thread Ganesh Ajjanagadde
On Sun, Oct 11, 2015 at 12:33 PM, wm4 wrote: > On Sun, 11 Oct 2015 09:59:39 -0400 > Ganesh Ajjanagadde wrote: > >> On Sun, Oct 11, 2015 at 9:34 AM, wm4 wrote: >> > On Sat, 10 Oct 2015 21:58:47 -0400 >> > Ganesh Ajjanagadde

Re: [FFmpeg-devel] [PATCH] configure: fix configure when using gcc

2015-10-11 Thread Timothy Gu
On Sun, Oct 11, 2015 at 8:21 AM Ganesh Ajjanagadde wrote: > Fixes Ticket4922. > > Commit 060102389e572abb2beaed3b9f5e1036aeea43f1 broke configure. > > From: Jean-Yves Avenard > You might want to use `git commit --author` instead of manually adding

Re: [FFmpeg-devel] [PATCH] configure: fix configure when using gcc

2015-10-11 Thread Ganesh Ajjanagadde
On Sun, Oct 11, 2015 at 12:53 PM, Ganesh Ajjanagadde wrote: > On Sun, Oct 11, 2015 at 12:50 PM, Timothy Gu wrote: >> On Sun, Oct 11, 2015 at 8:21 AM Ganesh Ajjanagadde >> wrote: >>> >>> Fixes Ticket4922. >>> >>> Commit

Re: [FFmpeg-devel] av_ctz optimization

2015-10-11 Thread Ronald S. Bultje
Hi, On Sun, Oct 11, 2015 at 12:53 PM, Ganesh Ajjanagadde wrote: > On Sun, Oct 11, 2015 at 12:33 PM, Timothy Gu > wrote: > > On Sun, Oct 11, 2015 at 9:10 AM Ganesh Ajjanagadde > wrote: > > > >> Isn't that an API change and not an ABI

Re: [FFmpeg-devel] [PATCH 2/2] x86/vf_w3fdif: simplify w3fdif_simple_high

2015-10-11 Thread Ronald S. Bultje
Hi, On Sun, Oct 11, 2015 at 1:17 PM, James Almer wrote: > On 10/11/2015 4:31 AM, Paul B Mahol wrote: > > On 10/11/15, James Almer wrote: > >> Signed-off-by: James Almer > >> --- > >> libavfilter/x86/vf_w3fdif.asm | 16 +++-

[FFmpeg-devel] [PATCH] intmath: remove av_ctz.

2015-10-11 Thread Ronald S. Bultje
It's an a non-installed header and only used in one place (flacenc). Since ff_ctz is static inline, it's fine to use that instead. --- libavcodec/flacenc.c | 2 +- libavutil/intmath.c | 5 - libavutil/intmath.h | 8 3 files changed, 1 insertion(+), 14 deletions(-) diff --git

Re: [FFmpeg-devel] Patch for Ticket 4922

2015-10-11 Thread Ganesh Ajjanagadde
On Sun, Oct 11, 2015 at 12:12 PM, Ganesh Ajjanagadde wrote: > On Sun, Oct 11, 2015 at 11:22 AM, Ganesh Ajjanagadde wrote: >> On Sun, Oct 11, 2015 at 9:26 AM, Ganesh Ajjanagadde wrote: >>> Hi all, >>> >>> Bug report from user:

Re: [FFmpeg-devel] [PATCH] avformat/rtmpcrypt: fix discarded const warning

2015-10-11 Thread Ronald S. Bultje
Hi, On Sun, Oct 11, 2015 at 2:15 PM, Ganesh Ajjanagadde wrote: > On Fri, Sep 18, 2015 at 5:02 PM, Michael Niedermayer > wrote: > > On Fri, Sep 18, 2015 at 09:50:29PM +0200, Nicolas George wrote: > >> Le jour du Génie, an CCXXIII, Ganesh Ajjanagadde a écrit :

[FFmpeg-devel] setting up a proper Github mirror

2015-10-11 Thread Ganesh Ajjanagadde
Hi all, I noticed that the Github mirror: https://github.com/FFmpeg/FFmpeg is over 3 hours out of sync with the main repos, making it unusable as a fetch url for development. Anyone knows why this is the case? Also, it looks like one can set some fancy UI illustrating that it is a mirror, see

Re: [FFmpeg-devel] policy on "necro-bumping" patches

2015-10-11 Thread Ganesh Ajjanagadde
On Sat, Sep 26, 2015 at 12:00 PM, Henrik Gramner wrote: > On Sat, Sep 26, 2015 at 5:18 PM, Ganesh Ajjanagadde wrote: >> On Sat, Sep 26, 2015 at 11:09 AM, Hendrik Leppkes >> wrote: >>> Please ping the actual thread of the patch, and not

Re: [FFmpeg-devel] setting up a proper Github mirror

2015-10-11 Thread Clément Bœsch
On Sun, Oct 11, 2015 at 02:43:53PM -0400, Ganesh Ajjanagadde wrote: > Hi all, > > I noticed that the Github mirror: https://github.com/FFmpeg/FFmpeg is > over 3 hours out of sync with the main repos, making it unusable as a > fetch url for development. Anyone knows why this is the case? > Isn't

Re: [FFmpeg-devel] setting up a proper Github mirror

2015-10-11 Thread Timothy Gu
On Sun, Oct 11, 2015 at 12:16 PM Ganesh Ajjanagadde wrote: > Alternatively (for now, more sustainable is a proper hook), could you > post here your alias to do this? > git remote set-url origin --push --add g...@github.com:FFmpeg/FFmpeg.git Timothy

Re: [FFmpeg-devel] [PATCH 1/2] avcodec/h264_slice: Use non_j_pixfmt() in the format match code

2015-10-11 Thread wm4
On Sun, 11 Oct 2015 21:16:26 +0200 Michael Niedermayer wrote: > From: Michael Niedermayer > > Signed-off-by: Michael Niedermayer > --- > libavcodec/h264_slice.c | 24 > 1 file changed, 12

Re: [FFmpeg-devel] [PATCH] avcodec/h264: remove redundant and bogus get_format call

2015-10-11 Thread wm4
On Sun, 11 Oct 2015 21:22:40 +0200 Michael Niedermayer wrote: > On Sun, Oct 11, 2015 at 07:05:27PM +0200, wm4 wrote: > > The AVCodecContext.get_format callback is not only used for pixel format > > negotiation with the API user, but also for hwaccel init. For the > >

[FFmpeg-devel] [PATCH 3/3] fate/subtitles: Add a new test for WebVTT

2015-10-11 Thread Ricardo Constantino
Includes escapes that should now be supported and a few features not yet fully supported, like comments, regions, classes, ruby, and lang. All were tested with https://quuz.org/webvtt/ for validation, except regions because the validator doesn't support them yet, and I couldn't find any other way

[FFmpeg-devel] [PATCH 1/3] avcodec/webvttdec: Deal with WebVTT escapes

2015-10-11 Thread Ricardo Constantino
Bare ampersand characters are still accepted, even though out-of-spec. Also fixes adjacent tags not being parsed. Fixes trac #4915 Signed-off-by: Ricardo Constantino --- libavcodec/webvttdec.c | 11 ++- 1 file changed, 10 insertions(+), 1 deletion(-) diff --git

[FFmpeg-devel] [PATCH 2/3] avformat/webvttdec: Don't stop parsing on comments

2015-10-11 Thread Ricardo Constantino
Signed-off-by: Ricardo Constantino --- libavformat/webvttdec.c | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/libavformat/webvttdec.c b/libavformat/webvttdec.c index 43c2a63..47a3255 100644 --- a/libavformat/webvttdec.c +++ b/libavformat/webvttdec.c @@

Re: [FFmpeg-devel] [PATCH 1/2] x86/vf_w3fdif: move pxor outside the loop in w3fdif_complex_low

2015-10-11 Thread Paul B Mahol
On 10/11/15, James Almer wrote: > Signed-off-by: James Almer > --- > libavfilter/x86/vf_w3fdif.asm | 8 > 1 file changed, 4 insertions(+), 4 deletions(-) > > diff --git a/libavfilter/x86/vf_w3fdif.asm b/libavfilter/x86/vf_w3fdif.asm > index

Re: [FFmpeg-devel] [PATCH] configure: fix configure when using gcc

2015-10-11 Thread Ganesh Ajjanagadde
On Sun, Oct 11, 2015 at 12:50 PM, Timothy Gu wrote: > On Sun, Oct 11, 2015 at 8:21 AM Ganesh Ajjanagadde > wrote: >> >> Fixes Ticket4922. >> >> Commit 060102389e572abb2beaed3b9f5e1036aeea43f1 broke configure. >> >> From: Jean-Yves Avenard

Re: [FFmpeg-devel] av_ctz optimization

2015-10-11 Thread Ganesh Ajjanagadde
On Sun, Oct 11, 2015 at 12:33 PM, Timothy Gu wrote: > On Sun, Oct 11, 2015 at 9:10 AM Ganesh Ajjanagadde wrote: > >> Isn't that an API change and not an ABI change (it was in doc/APIchanges)? >> > > No, since intmath.h is not public. So basically av_ctz

Re: [FFmpeg-devel] [PATCH 1/2] x86/vf_w3fdif: move pxor outside the loop in w3fdif_complex_low

2015-10-11 Thread James Almer
On 10/11/2015 1:28 PM, Paul B Mahol wrote: > On 10/11/15, James Almer wrote: >> Signed-off-by: James Almer >> --- >> libavfilter/x86/vf_w3fdif.asm | 8 >> 1 file changed, 4 insertions(+), 4 deletions(-) >> >> diff --git

Re: [FFmpeg-devel] [PATCH] avformat/rtmpcrypt: fix discarded const warning

2015-10-11 Thread Ganesh Ajjanagadde
On Fri, Sep 18, 2015 at 5:02 PM, Michael Niedermayer wrote: > On Fri, Sep 18, 2015 at 09:50:29PM +0200, Nicolas George wrote: >> Le jour du Génie, an CCXXIII, Ganesh Ajjanagadde a écrit : >> > This patch silences a -Wdiscarded-qualifiers observed with GCC 5.2. >> > >> >

Re: [FFmpeg-devel] [PATCH] ffplay: log possible error of SDL_EnableKeyRepeat

2015-10-11 Thread Ganesh Ajjanagadde
On Fri, Oct 9, 2015 at 9:03 AM, Ganesh Ajjanagadde wrote: > On Fri, Oct 9, 2015 at 4:21 AM, wm4 wrote: >> On Thu, 8 Oct 2015 18:46:44 -0400 >> Ganesh Ajjanagadde wrote: >> >>> Note that for the current SDL source code, 0 is

Re: [FFmpeg-devel] [PATCH] avformat/rtmpcrypt: fix discarded const warning

2015-10-11 Thread Ronald S. Bultje
Hi, On Sun, Oct 11, 2015 at 2:35 PM, Ganesh Ajjanagadde wrote: > On Sun, Oct 11, 2015 at 2:33 PM, Ronald S. Bultje > wrote: > > Hi, > > > > On Sun, Oct 11, 2015 at 2:15 PM, Ganesh Ajjanagadde > > wrote: > > > >> On Fri, Sep 18, 2015 at

Re: [FFmpeg-devel] setting up a proper Github mirror

2015-10-11 Thread Ganesh Ajjanagadde
On Sun, Oct 11, 2015 at 3:09 PM, Timothy Gu wrote: > On Sun, Oct 11, 2015 at 11:44 AM Ganesh Ajjanagadde > wrote: > >> I noticed that the Github mirror: https://github.com/FFmpeg/FFmpeg is >> over 3 hours out of sync with the main repos, making it

Re: [FFmpeg-devel] [PATCH] intmath: remove av_ctz.

2015-10-11 Thread Timothy Gu
On Sun, Oct 11, 2015 at 11:29 AM Ronald S. Bultje wrote: > It's an a non-installed header and only used in one place (flacenc). > Since ff_ctz is static inline, it's fine to use that instead. > --- > libavcodec/flacenc.c | 2 +- > libavutil/intmath.c | 5 - >

Re: [FFmpeg-devel] [PATCH] avcodec/h264: remove redundant and bogus get_format call

2015-10-11 Thread Michael Niedermayer
On Sun, Oct 11, 2015 at 07:05:27PM +0200, wm4 wrote: > The AVCodecContext.get_format callback is not only used for pixel format > negotiation with the API user, but also for hwaccel init. For the > latter, it's required that some codec parameters, in particular the > codec profile, are set when

Re: [FFmpeg-devel] [PATCH] configure: fix configure when using gcc

2015-10-11 Thread Nicolas George
Le decadi 20 vendémiaire, an CCXXIV, Ganesh Ajjanagadde a écrit : > Fixes Ticket4922. > > Commit 060102389e572abb2beaed3b9f5e1036aeea43f1 broke configure. The commit message could be a little more explicit: it forgot the ! that inverted the result. Also, it seems that the test went from '2.' to

Re: [FFmpeg-devel] [PATCH 1/9] dnxhdenc: fix scan used for bitstream generation

2015-10-11 Thread Michael Niedermayer
On Sun, Oct 11, 2015 at 04:06:05PM +0200, Christophe Gisquet wrote: > The functions related to bitstream reading must use the natural zigzag > order, and not the one permuted for use in the iDCT. This currently > results in bitstreams with 2 issues: > - AC coefficients are encoded in an unexpected

Re: [FFmpeg-devel] [PATCHv2] avutil/mathematics: speed up av_gcd by using Stein's binary GCD algorithm

2015-10-11 Thread wm4
On Sun, 11 Oct 2015 09:59:39 -0400 Ganesh Ajjanagadde wrote: > On Sun, Oct 11, 2015 at 9:34 AM, wm4 wrote: > > On Sat, 10 Oct 2015 21:58:47 -0400 > > Ganesh Ajjanagadde wrote: > > > >> This uses Stein's binary GCD algorithm: > >>

Re: [FFmpeg-devel] [PATCH] configure: fix configure when using gcc

2015-10-11 Thread Ganesh Ajjanagadde
On Sun, Oct 11, 2015 at 12:23 PM, Nicolas George wrote: > Le decadi 20 vendémiaire, an CCXXIV, Ganesh Ajjanagadde a écrit : >> I am not so sure, there was a reason for it documented in the >> 060102389e572abb2beaed3b9f5e1036aeea43f1 commit message. > > Indeed. We have until gcc

Re: [FFmpeg-devel] [PATCHv2] avfilter/buffersrc: add av_warn_unused_result attributes

2015-10-11 Thread Ganesh Ajjanagadde
On Wed, Oct 7, 2015 at 10:18 AM, Ganesh Ajjanagadde wrote: > On Wed, Oct 7, 2015 at 10:01 AM, Clément Bœsch wrote: >> On Wed, Oct 07, 2015 at 09:31:49AM -0400, Ganesh Ajjanagadde wrote: >>> On Wed, Oct 7, 2015 at 8:59 AM, Clément Bœsch wrote: >>> > On

Re: [FFmpeg-devel] [PATCHv2] doc/developer: add note on patches that fix warnings

2015-10-11 Thread Ganesh Ajjanagadde
On Tue, Oct 6, 2015 at 6:55 PM, Ganesh Ajjanagadde wrote: > This adds some recommendations while submitting patches that fix warnings. > > Signed-off-by: Ganesh Ajjanagadde > --- > doc/developer.texi | 6 ++ > 1 file changed, 6 insertions(+) >

Re: [FFmpeg-devel] [PATCHv2] avfilter/buffersrc: add av_warn_unused_result attributes

2015-10-11 Thread Ronald S. Bultje
Hi, On Sun, Oct 11, 2015 at 2:20 PM, Ganesh Ajjanagadde wrote: > On Wed, Oct 7, 2015 at 10:18 AM, Ganesh Ajjanagadde > wrote: > > On Wed, Oct 7, 2015 at 10:01 AM, Clément Bœsch wrote: > >> On Wed, Oct 07, 2015 at 09:31:49AM -0400, Ganesh

Re: [FFmpeg-devel] [PATCH] avformat/rtmpcrypt: fix discarded const warning

2015-10-11 Thread Ganesh Ajjanagadde
On Sun, Oct 11, 2015 at 2:33 PM, Ronald S. Bultje wrote: > Hi, > > On Sun, Oct 11, 2015 at 2:15 PM, Ganesh Ajjanagadde > wrote: > >> On Fri, Sep 18, 2015 at 5:02 PM, Michael Niedermayer >> wrote: >> > On Fri, Sep 18, 2015 at 09:50:29PM

[FFmpeg-devel] [PATCH] .gitignore: some font temporaries

2015-10-11 Thread Ganesh Ajjanagadde
--- .gitignore | 1 + 1 file changed, 1 insertion(+) diff --git a/.gitignore b/.gitignore index 0b26f68..b215828 100644 --- a/.gitignore +++ b/.gitignore @@ -3,3 +3,4 @@ /htdocs/main.rss /htdocs/components /htdocs/style.less +/htdocs/fonts/*.woff2 -- 2.6.1

Re: [FFmpeg-devel] [PATCH 2/2] avcodec/h264_slice: Never call get_format() for just checking if the format matches

2015-10-11 Thread Hendrik Leppkes
On Sun, Oct 11, 2015 at 10:43 PM, wm4 wrote: > On Sun, 11 Oct 2015 21:55:12 +0200 > Michael Niedermayer wrote: > >> On Sun, Oct 11, 2015 at 09:39:32PM +0200, wm4 wrote: >> > On Sun, 11 Oct 2015 21:16:27 +0200 >> > Michael Niedermayer

Re: [FFmpeg-devel] [PATCH 1/9] dnxhdenc: fix scan used for bitstream generation

2015-10-11 Thread Christophe Gisquet
Hi, 2015-10-11 18:20 GMT+02:00 Michael Niedermayer : > On Sun, Oct 11, 2015 at 04:06:05PM +0200, Christophe Gisquet wrote: >> The functions related to bitstream reading must use the natural zigzag >> order, and not the one permuted for use in the iDCT. This currently >>

Re: [FFmpeg-devel] [PATCH 2/2] avcodec/h264_slice: Never call get_format() for just checking if the format matches

2015-10-11 Thread Michael Niedermayer
On Sun, Oct 11, 2015 at 10:43:04PM +0200, wm4 wrote: > On Sun, 11 Oct 2015 21:55:12 +0200 > Michael Niedermayer wrote: > > > On Sun, Oct 11, 2015 at 09:39:32PM +0200, wm4 wrote: > > > On Sun, 11 Oct 2015 21:16:27 +0200 > > > Michael Niedermayer wrote: >

Re: [FFmpeg-devel] [PATCH 2/2] avcodec/h264_slice: Never call get_format() for just checking if the format matches

2015-10-11 Thread wm4
On Sun, 11 Oct 2015 23:00:07 +0200 Hendrik Leppkes wrote: > On Sun, Oct 11, 2015 at 10:43 PM, wm4 wrote: > > On Sun, 11 Oct 2015 21:55:12 +0200 > > Michael Niedermayer wrote: > > > >> On Sun, Oct 11, 2015 at 09:39:32PM +0200,

[FFmpeg-devel] [PATCH] avcodec: remove leftover iff_byterun1 decoder

2015-10-11 Thread Andreas Cadhalpun
It was merged with the iff_ilbm decoder in commit 929a24efff9a208a52748605eb412ffb915c1403. Define AV_CODEC_ID_IFF_BYTERUN1 as AV_CODEC_ID_IFF_ILBM for API compatibility. Signed-off-by: Andreas Cadhalpun --- This changes ABI, but that should be still OK. ---

Re: [FFmpeg-devel] [PATCH 2/2] avcodec/h264_slice: Never call get_format() for just checking if the format matches

2015-10-11 Thread Hendrik Leppkes
On Sun, Oct 11, 2015 at 11:14 PM, wm4 wrote: > On Sun, 11 Oct 2015 23:00:07 +0200 > Hendrik Leppkes wrote: > >> On Sun, Oct 11, 2015 at 10:43 PM, wm4 wrote: >> > On Sun, 11 Oct 2015 21:55:12 +0200 >> > Michael Niedermayer

Re: [FFmpeg-devel] [PATCH 2/2] avcodec/h264_slice: Never call get_format() for just checking if the format matches

2015-10-11 Thread wm4
On Sun, 11 Oct 2015 23:09:19 +0200 Michael Niedermayer wrote: > On Sun, Oct 11, 2015 at 10:43:04PM +0200, wm4 wrote: > > On Sun, 11 Oct 2015 21:55:12 +0200 > > Michael Niedermayer wrote: > > > > > On Sun, Oct 11, 2015 at 09:39:32PM +0200, wm4

Re: [FFmpeg-devel] [PATCH 2/2] avcodec/h264_slice: Never call get_format() for just checking if the format matches

2015-10-11 Thread wm4
On Sun, 11 Oct 2015 23:20:02 +0200 Hendrik Leppkes wrote: > On Sun, Oct 11, 2015 at 11:14 PM, wm4 wrote: > > On Sun, 11 Oct 2015 23:00:07 +0200 > > Hendrik Leppkes wrote: > > > >> On Sun, Oct 11, 2015 at 10:43 PM, wm4

Re: [FFmpeg-devel] [PATCH] intmath: remove av_ctz.

2015-10-11 Thread Andreas Cadhalpun
On 11.10.2015 20:28, Ronald S. Bultje wrote: > It's an a non-installed header and only used in one place (flacenc). ^ This 'an' should not be there. > Since ff_ctz is static inline, it's fine to use that instead. > --- > libavcodec/flacenc.c | 2 +- > libavutil/intmath.c | 5 - >

Re: [FFmpeg-devel] [PATCH] intmath: remove av_ctz.

2015-10-11 Thread James Almer
On 10/11/2015 6:26 PM, Andreas Cadhalpun wrote: > Anyway, please also remove the entry for av_ctz from APIchanges. > It's not public API, so it never belonged there. Huh, even an APIChanges line? Wonder if intmath.h was public at some point, or if whoever added this wasn't aware it was an

Re: [FFmpeg-devel] [PATCH] avcodec: remove leftover iff_byterun1 decoder

2015-10-11 Thread Ronald S. Bultje
Hi, On Sun, Oct 11, 2015 at 5:17 PM, Andreas Cadhalpun < andreas.cadhal...@googlemail.com> wrote: > It was merged with the iff_ilbm decoder in commit > 929a24efff9a208a52748605eb412ffb915c1403. > > Define AV_CODEC_ID_IFF_BYTERUN1 as AV_CODEC_ID_IFF_ILBM for API > compatibility. > >

[FFmpeg-devel] [PATCH] intmath: remove av_ctz.

2015-10-11 Thread Ronald S. Bultje
It's a non-installed header and only used in one place (flacenc). Since ff_ctz is static inline, it's fine to use that instead. --- doc/APIchanges | 3 --- libavcodec/flacenc.c | 2 +- libavutil/intmath.c | 5 - libavutil/intmath.h | 14 ++ 4 files changed, 7

Re: [FFmpeg-devel] [PATCH 1/3] ffmpeg: modify tty state when stderr is redirected

2015-10-11 Thread Ronald S. Bultje
Hi, On Fri, Jul 31, 2015 at 8:36 AM, Ganesh Ajjanagadde wrote: > Removes unnecessary isatty(), fixes Ticket2964 > > Signed-off-by: Ganesh Ajjanagadde > --- > ffmpeg.c | 8 +--- > 1 file changed, 1 insertion(+), 7 deletions(-) > > diff --git

Re: [FFmpeg-devel] [PATCH] avcodec: remove leftover iff_byterun1 decoder

2015-10-11 Thread Andreas Cadhalpun
Hi Ronald, On 11.10.2015 23:39, Ronald S. Bultje wrote: > On Sun, Oct 11, 2015 at 5:17 PM, Andreas Cadhalpun < > andreas.cadhal...@googlemail.com> wrote: >> --- a/libavcodec/avcodec.h >> +++ b/libavcodec/avcodec.h >> @@ -241,7 +241,7 @@ enum AVCodecID { >> AV_CODEC_ID_ANM, >>

Re: [FFmpeg-devel] [PATCH] intmath: remove av_ctz.

2015-10-11 Thread Andreas Cadhalpun
On 11.10.2015 23:44, Ronald S. Bultje wrote: > It's a non-installed header and only used in one place (flacenc). > Since ff_ctz is static inline, it's fine to use that instead. > --- > doc/APIchanges | 3 --- > libavcodec/flacenc.c | 2 +- > libavutil/intmath.c | 5 - >

Re: [FFmpeg-devel] [PATCH 1/3] ffmpeg: modify tty state when stderr is redirected

2015-10-11 Thread Ganesh Ajjanagadde
On Sun, Oct 11, 2015 at 5:45 PM, Ronald S. Bultje wrote: > Hi, > > On Fri, Jul 31, 2015 at 8:36 AM, Ganesh Ajjanagadde > wrote: >> >> Removes unnecessary isatty(), fixes Ticket2964 >> >> Signed-off-by: Ganesh Ajjanagadde >> >>

Re: [FFmpeg-devel] [PATCH] avcodec/h264: remove redundant and bogus get_format call

2015-10-11 Thread Michael Niedermayer
On Sun, Oct 11, 2015 at 09:27:13PM +0200, wm4 wrote: > On Sun, 11 Oct 2015 21:22:40 +0200 > Michael Niedermayer wrote: > > > On Sun, Oct 11, 2015 at 07:05:27PM +0200, wm4 wrote: > > > The AVCodecContext.get_format callback is not only used for pixel format > > >

Re: [FFmpeg-devel] [PATCH] avcodec: remove leftover iff_byterun1 decoder

2015-10-11 Thread Ronald S. Bultje
Howdy, On Sun, Oct 11, 2015 at 5:50 PM, Andreas Cadhalpun < andreas.cadhal...@googlemail.com> wrote: > Hi Ronald, > > On 11.10.2015 23:39, Ronald S. Bultje wrote: > > On Sun, Oct 11, 2015 at 5:17 PM, Andreas Cadhalpun < > > andreas.cadhal...@googlemail.com> wrote: > >> --- a/libavcodec/avcodec.h

Re: [FFmpeg-devel] [PATCH] web/src/about: add note on portability

2015-10-11 Thread Moritz Barsnick
On Sun, Oct 11, 2015 at 16:29:16 -0400, Ganesh Ajjanagadde wrote: > + Microsoft Windows, the BSD's, Solaris, etc. under a wide variety of build ^ Even abbreviations and acronyms don't get an apostrophe with their plural 's'.

Re: [FFmpeg-devel] [PATCH] avcodec: remove leftover iff_byterun1 decoder

2015-10-11 Thread Andreas Cadhalpun
On 11.10.2015 23:56, Ronald S. Bultje wrote: > On Sun, Oct 11, 2015 at 5:50 PM, Andreas Cadhalpun < > andreas.cadhal...@googlemail.com> wrote: >> On 11.10.2015 23:39, Ronald S. Bultje wrote: >>> On Sun, Oct 11, 2015 at 5:17 PM, Andreas Cadhalpun < >>> andreas.cadhal...@googlemail.com> wrote:

Re: [FFmpeg-devel] [PATCHv2] doc/developer: add note on patches that fix warnings

2015-10-11 Thread Michael Niedermayer
On Sun, Oct 11, 2015 at 02:27:11PM -0400, Ganesh Ajjanagadde wrote: > On Tue, Oct 6, 2015 at 6:55 PM, Ganesh Ajjanagadde > wrote: > > This adds some recommendations while submitting patches that fix warnings. > > > > Signed-off-by: Ganesh Ajjanagadde

Re: [FFmpeg-devel] [PATCHv2] doc/developer: add note on patches that fix warnings

2015-10-11 Thread Ganesh Ajjanagadde
On Sun, Oct 11, 2015 at 6:33 PM, Michael Niedermayer wrote: > On Sun, Oct 11, 2015 at 02:27:11PM -0400, Ganesh Ajjanagadde wrote: >> On Tue, Oct 6, 2015 at 6:55 PM, Ganesh Ajjanagadde >> wrote: >> > This adds some recommendations while submitting

Re: [FFmpeg-devel] [PATCH] avcodec: remove leftover iff_byterun1 decoder

2015-10-11 Thread Ronald S. Bultje
Hi, On Sun, Oct 11, 2015 at 6:26 PM, Andreas Cadhalpun < andreas.cadhal...@googlemail.com> wrote: > On 11.10.2015 23:56, Ronald S. Bultje wrote: > > On Sun, Oct 11, 2015 at 5:50 PM, Andreas Cadhalpun < > > andreas.cadhal...@googlemail.com> wrote: > >> On 11.10.2015 23:39, Ronald S. Bultje wrote:

Re: [FFmpeg-devel] GCC 5.1 warning: -Warray-bounds

2015-10-11 Thread Ganesh Ajjanagadde
On Wed, Jul 22, 2015 at 10:56 PM, Ganesh Ajjanagadde wrote: > On Tue, Jul 21, 2015 at 10:46 PM, James Almer wrote: >> On 21/07/15 11:43 PM, Ganesh Ajjanagadde wrote: >>> or try to work upstream with GCC to remove these spurious warnings. >> >> If it can be

Re: [FFmpeg-devel] [PATCH 2/2] x86/vf_w3fdif: simplify w3fdif_simple_high

2015-10-11 Thread James Almer
On 10/11/2015 3:11 PM, Ronald S. Bultje wrote: > Hi, > > On Sun, Oct 11, 2015 at 1:17 PM, James Almer wrote: > >> On 10/11/2015 4:31 AM, Paul B Mahol wrote: >>> On 10/11/15, James Almer wrote: Signed-off-by: James Almer ---

Re: [FFmpeg-devel] [PATCH 2/2] avcodec/h264_slice: Never call get_format() for just checking if the format matches

2015-10-11 Thread wm4
On Sun, 11 Oct 2015 21:16:27 +0200 Michael Niedermayer wrote: > From: Michael Niedermayer > > Signed-off-by: Michael Niedermayer > --- > libavcodec/h264_slice.c |4 > 1 file changed, 4 insertions(+) > > diff --git

Re: [FFmpeg-devel] [PATCH 2/2] avcodec/h264_slice: Never call get_format() for just checking if the format matches

2015-10-11 Thread Michael Niedermayer
On Sun, Oct 11, 2015 at 09:39:32PM +0200, wm4 wrote: > On Sun, 11 Oct 2015 21:16:27 +0200 > Michael Niedermayer wrote: > > > From: Michael Niedermayer > > > > Signed-off-by: Michael Niedermayer > > --- > >

  1   2   >