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 > index f02319b..f2001a4 100644 > --- a/libav

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, 45 insertions(+) > > diff --git a/compat/

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, 3,3,3, v1, v2, order >

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 | Mon Oct 5 > >> 20:49:55 2015 +0200| [8b8

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 > > --- a/libavcodec/x86/audiodsp.asm > > +++ b/libavcodec/x86/audiod

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 changed, 45 insertions(+) > > diff --git

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 > > interface is actually pu

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. > --- > doc/filters.t

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 >> ++-- >> 2 files ch

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 + > libavutil/jni_internal.c | 30 ++ > libavutil/jni_inter

[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 F

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, and a roughly 2x speedup ot

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 9:34 AM, wm4 wrote: > 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 com

[FFmpeg-devel] [PATCH 0/9] v3: simple_idct for 10/12 bits and DNxHD

2015-10-11 Thread Christophe Gisquet
Summary: - fixed dct-test additions - changed coefficients for 10- and 12- C simple_idct, to match the x86 ones and gain a bit on dct-test omse I'm not really happy about the 10bits coeff change: to me, this can cause overflows, but I couldn't manage to produce that. Christophe Gisquet (9): d

[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 curren

[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 +- tests/ref/vsynth/vsynth1-dnxhd-7

[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 run

[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 + li

[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 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 scal

[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 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 a/libavco

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 > --- > libavcodec/webvttdec.c | 13 +++-

[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 index 3d9803a..7e55e92 100755 --- a/configure +

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 > when using gcc". > >

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 converts \h to r

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 relying on \h being parsed as nbsp in

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 going about it, I wa

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 that the De-Bruijn method can be used t

[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 a/libavcodec/webvttdec.c

[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 @@ -92,7 +92,8 @@ static

[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

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, except for the >> commit message, which s

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 : > 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] 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 forgot the ! that > inv

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] [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] [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 988b847..f02319b 100644 > --- a/libavfilter/x8

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@ffmpeg.org http://ffmpeg.org

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: > >> https://en.wikipedia.org/wiki/Binary_GCD_algorithm > >> to get a

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 version 20 to find

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 wrote: >> > >> >> This uses Stein's binary GCD algorithm: >> >> https://

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 a "From" line in the commit message body. Timo

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 was a bungled operation. Someone needed it

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 > > > You might want to use `git commit --author` instead

[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 invoca

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 a/libavfilter/x86/vf_w3fdif.asm b/libavfilter/x86/vf_w3fdi

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 a/libavfilter/x86/vf_w3fdif.asm b/libavfilter/x86/vf_w3fdif.asm >>

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 060102389e572abb2beaed3b9f5e1036aeea43f1 broke configure. >>> >>> From: Jean-

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 change (it was in > doc/APIchanges)? > >> > > > > No, since i

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 +++- > >> 1 file changed, 7 insertions(+), 9 deletions(-) > >>

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. >> > >> > Signed-off-by: Ganesh Ajjan

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 Tue, Oct 06, 2015 at 06:53:47PM -0400, Ganes

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 always returned. >>> Nevertheless, this makes the code more robust. >>>

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(+) > > diff --git a/doc/developer.texi b/doc/develope

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: https://trac.ffmpeg.org/ticket/4922, and >>> associated patch has been

[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 a/liba

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 Ajjanagadde wrote: > >>> On Wed, Oct 7, 2015 at 8:59 AM,

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 : > >> > This patch silences a -Wdisca

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 +0200, Nicolas George wrote: >> >> Le jour du Génie, an CCXX

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 5:02 PM, Michael Niedermayer > >> wrote: > >> > On Fri, S

[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 for

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 some seemingly >>> unrelated thread, so people are actually

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 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 not an automatic mirr

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 unusable as a >> fetch url for development. Anyon

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 - > libavutil/intmath.h | 8 --

[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 --- a/libavcodec/h264_slice.c +++ b/libavcodec/h264_slice.c @@ -985

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

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

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

2015-10-11 Thread James Almer
On 10/11/2015 4:16 PM, Timothy Gu wrote: > 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/int

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

2015-10-11 Thread Ganesh Ajjanagadde
On Sun, Oct 11, 2015 at 3:04 PM, Clément Bœsch wrote: > 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 f

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] 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 the

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 insertions(+), 12 deletions(-) > > diff --git a/libavcodec/h264_slice.c b/lib

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 > > latter, it's required that so

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

2015-10-11 Thread Michael Niedermayer
On Sun, Oct 11, 2015 at 09:04:32PM +0200, Clément Bœsch wrote: > 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 >

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

2015-10-11 Thread Ganesh Ajjanagadde
On Sun, Oct 11, 2015 at 3:20 PM, Timothy Gu wrote: > 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/FFm

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 a/libavcodec/h264_slice.c b/libavcodec/h264_slice.c > index cce97d9..

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

2015-10-11 Thread Ganesh Ajjanagadde
On Sun, Oct 11, 2015 at 3:33 PM, Michael Niedermayer wrote: > On Sun, Oct 11, 2015 at 09:04:32PM +0200, Clément Bœsch wrote: >> 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

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

2015-10-11 Thread Ben Boeckel
On Sun, 11 Oct, 2015 at 16:34:51 GMT, Ganesh Ajjanagadde wrote: > That is a long time hopefully (estimated 15+ years), unless gcc starts > pulling a "fast release" cycle like Firefox or Chrome. Sort of. Old point releases will be major version number bumps (GCC 6 is under development already). --

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

2015-10-11 Thread Ganesh Ajjanagadde
On Sun, Oct 11, 2015 at 3:45 PM, Ben Boeckel wrote: > On Sun, 11 Oct, 2015 at 16:34:51 GMT, Ganesh Ajjanagadde wrote: >> That is a long time hopefully (estimated 15+ years), unless gcc starts >> pulling a "fast release" cycle like Firefox or Chrome. > > Sort of. Old point releases will be major ve

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 > > --- > > libavcodec/h264_slice.c |4 > > 1 file changed, 4 insertions(+) > > > > diff

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

2015-10-11 Thread Hendrik Leppkes
On Sun, Oct 11, 2015 at 9:55 PM, Ganesh Ajjanagadde wrote: > On Sun, Oct 11, 2015 at 3:45 PM, Ben Boeckel wrote: >> On Sun, 11 Oct, 2015 at 16:34:51 GMT, Ganesh Ajjanagadde wrote: >>> That is a long time hopefully (estimated 15+ years), unless gcc starts >>> pulling a "fast release" cycle like Fi

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

2015-10-11 Thread Michael Niedermayer
On Sun, Oct 11, 2015 at 03:43:19PM -0400, Ganesh Ajjanagadde wrote: > On Sun, Oct 11, 2015 at 3:33 PM, Michael Niedermayer > wrote: > > On Sun, Oct 11, 2015 at 09:04:32PM +0200, Clément Bœsch wrote: > >> On Sun, Oct 11, 2015 at 02:43:53PM -0400, Ganesh Ajjanagadde wrote: > >> > Hi all, > >> > > >>

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

2015-10-11 Thread Ganesh Ajjanagadde
On Sun, Oct 11, 2015 at 4:05 PM, Hendrik Leppkes wrote: > On Sun, Oct 11, 2015 at 9:55 PM, Ganesh Ajjanagadde wrote: >> On Sun, Oct 11, 2015 at 3:45 PM, Ben Boeckel wrote: >>> On Sun, 11 Oct, 2015 at 16:34:51 GMT, Ganesh Ajjanagadde wrote: That is a long time hopefully (estimated 15+ years)

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

2015-10-11 Thread Ganesh Ajjanagadde
On Sun, Oct 11, 2015 at 4:11 PM, Michael Niedermayer wrote: > On Sun, Oct 11, 2015 at 03:43:19PM -0400, Ganesh Ajjanagadde wrote: >> On Sun, Oct 11, 2015 at 3:33 PM, Michael Niedermayer >> wrote: >> > On Sun, Oct 11, 2015 at 09:04:32PM +0200, Clément Bœsch wrote: >> >> On Sun, Oct 11, 2015 at 02:

[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 ___

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

2015-10-11 Thread Ganesh Ajjanagadde
--- src/about | 6 +- 1 file changed, 5 insertions(+), 1 deletion(-) diff --git a/src/about b/src/about index 8d8f606..b9a0db3 100644 --- a/src/about +++ b/src/about @@ -4,7 +4,11 @@ transcode, mux, demux, stream, filter and play pretty much anything that humans and machines have cr

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: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: > > > > > From: Michael Niedermayer > > > > > > Signed-off-by: Michael Niedermayer > > > --- > > > libav

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 wrote: >> > >> > > From: Michael Niedermayer >> > > >> > > Si

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 >> results in bitstreams with

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: > > > > > > > From: 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: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, wm4 wrote: > >> > On Sun, 11 Oct 2015 21:16:27 +0200 > >> > Michael N

[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. --- libavcodec/Makefile | 1 - li

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 wrote: >> > >> >> On Sun, Oct 11, 2015 at 09:39:32PM +0200, wm4 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: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 wrote: > > > > On Sun, 11 Oct 2015 21:16:27 +0200 >

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 wrote: > >> > On Sun, 11 Oct 2015 21:55:12 +0200 > >> > Michael Niedermayer

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 - > liba

  1   2   >