Re: [FFmpeg-devel] [PATCH 6/7] lavu/aes: add x86 AESNI optimizations

2015-10-11 Thread Nicolas George
Le primidi 21 vendémiaire, an CCXXIV, Rodger Combs a écrit : > --- > libavutil/aes.c | 2 ++ > libavutil/aes_internal.h | 2 ++ > libavutil/x86/Makefile | 4 ++- > libavutil/x86/aes.asm| 93 > > libavutil/x86/aes_init.c | 37 +

[FFmpeg-devel] [PATCH] lavu/aes: add x86 AESNI optimizations

2015-10-11 Thread Rodger Combs
--- libavutil/aes.c | 2 ++ libavutil/aes_internal.h | 2 ++ libavutil/x86/Makefile | 4 ++- libavutil/x86/aes.asm| 92 libavutil/x86/aes_init.c | 37 +++ 5 files changed, 136 insertions(+), 1 deletion(-) create mo

[FFmpeg-devel] [PATCH] lavu: add AESNI CPU flag

2015-10-11 Thread Rodger Combs
--- configure | 4 doc/APIchanges| 3 +++ libavutil/cpu.c | 4 libavutil/cpu.h | 1 + libavutil/version.h | 2 +- libavutil/x86/cpu.c | 2 ++ libavutil/x86/cpu.h | 3 +++ libavutil/x86/x86inc.asm | 13 +++--

Re: [FFmpeg-devel] [PATCH 6/7] lavu/aes: add x86 AESNI optimizations

2015-10-11 Thread James Almer
On 10/12/2015 2:38 AM, Rodger Combs wrote: > +%macro AES_CRYPT 1 > +%if %1 == 1 > +%define CRYPT aesdec > +%define LAST aesdeclast > +cglobal aes_decrypt, 6,6,2 > +%else > +%define CRYPT aesenc > +%define LAST aesenclast > +cglobal aes_encrypt, 6,6,2 > +%endif > +pxor xm1, xm1 > +mov r5d,

[FFmpeg-devel] [PATCH 5/7] lavu/aes: test CBC functionality

2015-10-11 Thread Rodger Combs
--- libavutil/aes.c | 14 +- 1 file changed, 9 insertions(+), 5 deletions(-) diff --git a/libavutil/aes.c b/libavutil/aes.c index c1d860f..1fac4e8 100644 --- a/libavutil/aes.c +++ b/libavutil/aes.c @@ -280,7 +280,7 @@ int main(int argc, char **argv) { 0x10, 0xa5, 0x88, 0x69,

[FFmpeg-devel] [PATCH 7/7] checkasm: add tests for AES

2015-10-11 Thread Rodger Combs
--- tests/checkasm/Makefile | 2 +- tests/checkasm/aes.c | 86 +++ tests/checkasm/checkasm.c | 1 + tests/checkasm/checkasm.h | 1 + 4 files changed, 89 insertions(+), 1 deletion(-) create mode 100644 tests/checkasm/aes.c diff --git a/tests/c

[FFmpeg-devel] [PATCH 4/7] lavu/aes: align AVAES struct members

2015-10-11 Thread Rodger Combs
--- libavutil/aes_internal.h | 5 +++-- 1 file changed, 3 insertions(+), 2 deletions(-) diff --git a/libavutil/aes_internal.h b/libavutil/aes_internal.h index e5bf4bd..4944258 100644 --- a/libavutil/aes_internal.h +++ b/libavutil/aes_internal.h @@ -21,6 +21,7 @@ #ifndef AVUTIL_AES_INTERNAL_H #d

[FFmpeg-devel] [PATCH 6/7] lavu/aes: add x86 AESNI optimizations

2015-10-11 Thread Rodger Combs
--- libavutil/aes.c | 2 ++ libavutil/aes_internal.h | 2 ++ libavutil/x86/Makefile | 4 ++- libavutil/x86/aes.asm| 93 libavutil/x86/aes_init.c | 37 +++ 5 files changed, 137 insertions(+), 1 deletion(-) create mo

[FFmpeg-devel] [PATCH 3/7] lavu/aes: add runtime dispatch for crypt function

2015-10-11 Thread Rodger Combs
--- libavutil/aes.c | 47 +++ libavutil/aes_internal.h | 1 + 2 files changed, 32 insertions(+), 16 deletions(-) diff --git a/libavutil/aes.c b/libavutil/aes.c index 280e5fa..c1d860f 100644 --- a/libavutil/aes.c +++ b/libavutil/aes.c @@ -126,3

[FFmpeg-devel] [PATCH 2/7] lavu/aes: move AVAES to separate internal header

2015-10-11 Thread Rodger Combs
--- libavutil/aes.c | 16 +--- libavutil/aes_internal.h | 41 + 2 files changed, 42 insertions(+), 15 deletions(-) create mode 100644 libavutil/aes_internal.h diff --git a/libavutil/aes.c b/libavutil/aes.c index 8d4..280e5fa 100644

[FFmpeg-devel] [PATCH 1/7] lavu: add AESNI CPU flag

2015-10-11 Thread Rodger Combs
--- configure | 6 +- doc/APIchanges| 3 +++ libavutil/cpu.c | 6 +- libavutil/cpu.h | 1 + libavutil/version.h | 2 +- libavutil/x86/cpu.c | 2 ++ libavutil/x86/cpu.h | 3 +++ libavutil/x86/x86inc.asm | 41 +

Re: [FFmpeg-devel] [PATCH 3/3][RFC] avfilter/vf_chromakey: Add OpenCL acceleration

2015-10-11 Thread highgod0401
From: Timo Rothenpieler Date: 2015-10-10 17:47 To: FFmpeg development discussions and patches CC: highgod0401 Subject: Re: [FFmpeg-devel] [PATCH 3/3][RFC] avfilter/vf_chromakey: Add OpenCL acceleration > Could you please fix it? > > Thanks > > Best regards curl https://github.com/BtbN/FFmpeg/c

[FFmpeg-devel] [PATCH] vf_lut: Add support for RGB48 and RGBA64

2015-10-11 Thread Steven Robertson
Thanks for taking a look! Steve 0001-vf_lut-Add-support-for-RGB48-and-RGBA64.patch Description: Binary data ___ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org http://ffmpeg.org/mailman/listinfo/ffmpeg-devel

Re: [FFmpeg-devel] [PATCH] avutil/intmath: use de Bruijn based ff_ctz

2015-10-11 Thread James Almer
On 10/11/2015 10:55 PM, Ganesh Ajjanagadde wrote: > It has already been demonstrated that the de Bruijn method has benefits > over the current implementation: commit > 971d12b7f9d7be3ca8eb98e6c04ed521f83cbd3c. > That commit implemented it for long long, this extends it to the 32 bit > version. >

Re: [FFmpeg-devel] [PATCH 1/7] Add AESNI CPU flag

2015-10-11 Thread James Almer
On 10/12/2015 12:20 AM, Rodger Combs wrote: > --- > configure | 4 > libavutil/cpu.c | 3 +++ > libavutil/cpu.h | 1 + > libavutil/x86/cpu.c | 2 ++ > libavutil/x86/cpu.h | 3 +++ > libavutil/x86/x86inc.asm | 1 + > tests/checkasm/checkasm.c |

Re: [FFmpeg-devel] [PATCH 6/7] lavu/aes: add x86 AESNI optimizations

2015-10-11 Thread James Almer
On 10/12/2015 12:20 AM, Rodger Combs wrote: > --- > libavutil/aes.c | 2 ++ > libavutil/aes_internal.h | 2 ++ > libavutil/x86/Makefile | 4 ++- > libavutil/x86/aes.asm| 85 > > libavutil/x86/aes_init.c | 42 ++

Re: [FFmpeg-devel] [PATCH 4/7] lavu/aes: align AVAES struct

2015-10-11 Thread James Almer
On 10/12/2015 12:20 AM, Rodger Combs wrote: > --- > libavutil/aes.c | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/libavutil/aes.c b/libavutil/aes.c > index 06b1872..ab2d602 100644 > --- a/libavutil/aes.c > +++ b/libavutil/aes.c > @@ -274,7 +274,7 @@ int av_aes_init(AVAE

Re: [FFmpeg-devel] [PATCH 3/7] lavu/aes: add runtime dispatch for crypt function

2015-10-11 Thread James Almer
On 10/12/2015 12:20 AM, Rodger Combs wrote: > diff --git a/libavutil/aes_internal.h b/libavutil/aes_internal.h > index e2841ef..37b9568 100644 > --- a/libavutil/aes_internal.h > +++ b/libavutil/aes_internal.h > @@ -36,6 +36,7 @@ typedef struct AVAES { > av_aes_block round_key[15]; > av_ae

[FFmpeg-devel] [PATCH 5/7] lavu/aes: test CBC functionality

2015-10-11 Thread Rodger Combs
--- libavutil/aes.c | 14 +- 1 file changed, 9 insertions(+), 5 deletions(-) diff --git a/libavutil/aes.c b/libavutil/aes.c index ab2d602..c917706 100644 --- a/libavutil/aes.c +++ b/libavutil/aes.c @@ -280,7 +280,7 @@ int main(int argc, char **argv) { 0x10, 0xa5, 0x88, 0x69,

[FFmpeg-devel] [PATCH 7/7] checkasm: add tests for AES

2015-10-11 Thread Rodger Combs
--- tests/checkasm/Makefile | 2 +- tests/checkasm/aes.c | 85 +++ tests/checkasm/checkasm.c | 1 + tests/checkasm/checkasm.h | 1 + 4 files changed, 88 insertions(+), 1 deletion(-) create mode 100644 tests/checkasm/aes.c diff --git a/tests/c

[FFmpeg-devel] [PATCH 6/7] lavu/aes: add x86 AESNI optimizations

2015-10-11 Thread Rodger Combs
--- libavutil/aes.c | 2 ++ libavutil/aes_internal.h | 2 ++ libavutil/x86/Makefile | 4 ++- libavutil/x86/aes.asm| 85 libavutil/x86/aes_init.c | 42 5 files changed, 134 insertions(+), 1 deletion(-) crea

[FFmpeg-devel] [PATCH 3/7] lavu/aes: add runtime dispatch for crypt function

2015-10-11 Thread Rodger Combs
--- libavutil/aes.c | 47 +++ libavutil/aes_internal.h | 1 + 2 files changed, 32 insertions(+), 16 deletions(-) diff --git a/libavutil/aes.c b/libavutil/aes.c index 280e5fa..06b1872 100644 --- a/libavutil/aes.c +++ b/libavutil/aes.c @@ -126,3

[FFmpeg-devel] [PATCH 4/7] lavu/aes: align AVAES struct

2015-10-11 Thread Rodger Combs
--- libavutil/aes.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/libavutil/aes.c b/libavutil/aes.c index 06b1872..ab2d602 100644 --- a/libavutil/aes.c +++ b/libavutil/aes.c @@ -274,7 +274,7 @@ int av_aes_init(AVAES *a, const uint8_t *key, int key_bits, int decrypt) int mai

[FFmpeg-devel] [PATCH 1/7] Add AESNI CPU flag

2015-10-11 Thread Rodger Combs
--- configure | 4 libavutil/cpu.c | 3 +++ libavutil/cpu.h | 1 + libavutil/x86/cpu.c | 2 ++ libavutil/x86/cpu.h | 3 +++ libavutil/x86/x86inc.asm | 1 + tests/checkasm/checkasm.c | 1 + 7 files changed, 15 insertions(+) diff --git a/configu

[FFmpeg-devel] [PATCH 2/7] lavu/aes: move AVAES to separate internal header

2015-10-11 Thread Rodger Combs
--- libavutil/aes.c | 16 +--- libavutil/aes_internal.h | 41 + 2 files changed, 42 insertions(+), 15 deletions(-) create mode 100644 libavutil/aes_internal.h diff --git a/libavutil/aes.c b/libavutil/aes.c index 8d4..280e5fa 100644

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

2015-10-11 Thread Ronald S. Bultje
Hi, On Sun, Oct 11, 2015 at 9:17 PM, Ganesh Ajjanagadde wrote: > On Sun, Oct 11, 2015 at 9:12 PM, Ganesh Ajjanagadde > wrote: > > On Sun, Oct 11, 2015 at 6:04 PM, Ronald S. Bultje > wrote: > >> Hi, > >> > >> On Sun, Oct 11, 2015 at 5:52 PM, Andreas Cadhalpun < > >> andreas.cadhal...@googlemail

[FFmpeg-devel] [PATCH] avutil/intmath: use de Bruijn based ff_ctz

2015-10-11 Thread Ganesh Ajjanagadde
It has already been demonstrated that the de Bruijn method has benefits over the current implementation: commit 971d12b7f9d7be3ca8eb98e6c04ed521f83cbd3c. That commit implemented it for long long, this extends it to the 32 bit version. The function is renamed from ff_ctz to ff_ctz32 since it cruci

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

2015-10-11 Thread Ganesh Ajjanagadde
On Sun, Oct 11, 2015 at 9:38 PM, Timothy Gu wrote: > On Sun, Oct 11, 2015 at 6:31 PM Ganesh Ajjanagadde wrote: > >> Just to make this clear: suppose (hypothetically) av_ctz was public. >> If my application never used av_ctz, but I (or my distro) upgrades >> libav*, then I don't need to recompile/

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

2015-10-11 Thread Timothy Gu
On Sun, Oct 11, 2015 at 6:31 PM Ganesh Ajjanagadde wrote: > Just to make this clear: suppose (hypothetically) av_ctz was public. > If my application never used av_ctz, but I (or my distro) upgrades > libav*, then I don't need to recompile/relink my code? > If we are only talking about av_ctz her

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

2015-10-11 Thread Ganesh Ajjanagadde
On Sun, Oct 11, 2015 at 9:24 PM, James Almer wrote: > On 10/11/2015 10:12 PM, Ganesh Ajjanagadde wrote: >> On Sun, Oct 11, 2015 at 6:04 PM, Ronald S. Bultje wrote: >>> Hi, >>> >>> On Sun, Oct 11, 2015 at 5:52 PM, Andreas Cadhalpun < >>> andreas.cadhal...@googlemail.com> wrote: >>> On 11.10.2

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

2015-10-11 Thread Ganesh Ajjanagadde
On Sun, Oct 11, 2015 at 9:24 PM, James Almer wrote: > On 10/11/2015 10:12 PM, Ganesh Ajjanagadde wrote: >> On Sun, Oct 11, 2015 at 6:04 PM, Ronald S. Bultje wrote: >>> Hi, >>> >>> On Sun, Oct 11, 2015 at 5:52 PM, Andreas Cadhalpun < >>> andreas.cadhal...@googlemail.com> wrote: >>> On 11.10.2

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

2015-10-11 Thread James Almer
On 10/11/2015 10:12 PM, Ganesh Ajjanagadde wrote: > On Sun, Oct 11, 2015 at 6:04 PM, Ronald S. Bultje wrote: >> Hi, >> >> On Sun, Oct 11, 2015 at 5:52 PM, Andreas Cadhalpun < >> andreas.cadhal...@googlemail.com> wrote: >> >>> On 11.10.2015 23:44, Ronald S. Bultje wrote: It's a non-installed h

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

2015-10-11 Thread Ganesh Ajjanagadde
On Sun, Oct 11, 2015 at 9:12 PM, Ganesh Ajjanagadde wrote: > On Sun, Oct 11, 2015 at 6:04 PM, Ronald S. Bultje wrote: >> Hi, >> >> On Sun, Oct 11, 2015 at 5:52 PM, Andreas Cadhalpun < >> andreas.cadhal...@googlemail.com> wrote: >> >>> On 11.10.2015 23:44, Ronald S. Bultje wrote: >>> > It's a non-

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

2015-10-11 Thread Ganesh Ajjanagadde
On Sun, Oct 11, 2015 at 6:04 PM, Ronald S. Bultje wrote: > Hi, > > On Sun, Oct 11, 2015 at 5:52 PM, Andreas Cadhalpun < > andreas.cadhal...@googlemail.com> wrote: > >> 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_

Re: [FFmpeg-devel] [PATCHv2 1/2] avfilter/all: propagate errors of functions from avfilter/formats

2015-10-11 Thread Ganesh Ajjanagadde
On Sat, Oct 10, 2015 at 9:47 AM, Nicolas George wrote: > Le nonidi 19 vendémiaire, an CCXXIV, Clement Boesch a écrit : >> You could change that last function to return AVERROR(ENOMEM) in case a >> parameter is NULL, but that's weird semantic. > > Since it is an internal function, it is not really

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

2015-10-11 Thread Ganesh Ajjanagadde
On Sun, Oct 11, 2015 at 6:00 PM, Moritz Barsnick wrote: > 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

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 --- libavfilter/x86/vf_w3fdif.asm | 16 +++- >>

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 fixed upstream then that's certainly the

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] [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 patches that fix warnings. >> > >> > Signed-off-by

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 > > --- > > doc/developer.texi | 6 ++ > >

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] [PATCH 1/2] ffplay: close streams and AVFormatContext in the main thread

2015-10-11 Thread Marton Balint
On Wed, 7 Oct 2015, Marton Balint wrote: To avoid race conditions. Signed-off-by: Marton Balint --- ffplay.c | 28 ++-- 1 file changed, 14 insertions(+), 14 deletions(-) Pushed both patches in the series. Regards, Marton __

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 11:07:25PM +0200, Christophe Gisquet wrote: > 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

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

2015-10-11 Thread Ronald S. Bultje
Hi, On Sun, Oct 11, 2015 at 5:52 PM, Andreas Cadhalpun < andreas.cadhal...@googlemail.com> wrote: > 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

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'. http://www.oxforddictionaries.com/words/

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] 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 > > > negotiation with the API user, but

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 - > libavutil/int

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 >> >> --- >> ffmpeg.c | 8 +--- >> 1 file changed, 1 insertion(+), 7 d

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, >> AV_CODEC_ID

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 a/ffmpeg.c b/ffmpeg.c > index 5575e2f..fe250e5 10

[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 insertions(

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

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 internal

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

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

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

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

[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

[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] 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:

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

[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

[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

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

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

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

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: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] [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,

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

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

  1   2   >