Re: [libav-devel] [PATCH] eval: fix printing of NaN in eval fate test.

2012-07-26 Thread Alex Converse
On Wed, Jul 25, 2012 at 8:42 PM, Ronald S. Bultje rsbul...@gmail.com wrote: From: Ronald S. Bultje rsbul...@gmail.com This fixes make fate-eval on MSVC builds. Without this, the test outputs -1.#NaN instead of nan on MSVS 2010. --- libavutil/eval.c |5 + 1 file changed, 5

Re: [libav-devel] [PATCH 1/1] Added urldecode support to http authentication.

2012-07-26 Thread Antti Seppälä
On 26 July 2012 00:19, Martin Storsjö mar...@martin.st wrote: Pushed with the style nitpicks fixed and the wording of the commit message slightly adjusted. // Martin Thanks! And thank you to Diego as well from review comments. Best regards -- Antti

Re: [libav-devel] [PATCH] libopenjpeg: support YUV and deep RGB pixel formats

2012-07-26 Thread Kostya Shishkov
On Tue, Jul 24, 2012 at 07:43:06PM +0200, Luca Barbato wrote: From: Michael Bradshaw mbrads...@sorensonmedia.com Based on FFmpeg version from commit 3275981207e30e140cffaea334ac390f1a04266a --- Rebased since Diego pushed the wrong version. libavcodec/libopenjpegdec.c | 299

Re: [libav-devel] [PATCH] swscale: bury one more piece of inline asm under HAVE_INLINE_ASM.

2012-07-26 Thread Diego Biurrun
On Thu, Jul 26, 2012 at 05:10:10AM +0200, Luca Barbato wrote: On 07/26/2012 04:27 AM, Ronald S. Bultje wrote: From: Ronald S. Bultje rsbul...@gmail.com --- libswscale/swscale.c |2 +- 1 file changed, 1 insertion(+), 1 deletion(-) Ok. No, not OK. This is just a repackaged

Re: [libav-devel] [PATCH] x86/dsputilenc: bury inline asm under HAVE_INLINE_ASM.

2012-07-26 Thread Diego Biurrun
On Wed, Jul 25, 2012 at 08:30:19PM -0700, Ronald S. Bultje wrote: From: Ronald S. Bultje rsbul...@gmail.com --- libavcodec/dct-test.c |2 +- libavcodec/x86/dsputilenc_mmx.c | 80 +++ libavcodec/x86/fdct_mmx.c |4 ++

Re: [libav-devel] [PATCH v4] build: do full flag handling for all compiler-type tools

2012-07-26 Thread Diego Biurrun
On Wed, Jul 25, 2012 at 11:33:43PM +0100, Måns Rullgård wrote: Diego Biurrun di...@biurrun.de writes: On Wed, Jul 25, 2012 at 02:10:05PM +0100, Mans Rullgard wrote: This adds a full identification probe of CC, AS, LD and HOSTCC, and sets up correct flags and dependency tracking for each.

Re: [libav-devel] [PATCH] x86/dsputilenc: bury inline asm under HAVE_INLINE_ASM.

2012-07-26 Thread Måns Rullgård
Ronald S. Bultje rsbul...@gmail.com writes: From: Ronald S. Bultje rsbul...@gmail.com --- libavcodec/dct-test.c |2 +- libavcodec/x86/dsputilenc_mmx.c | 80 +++ libavcodec/x86/fdct_mmx.c |4 ++ libavcodec/x86/motion_est_mmx.c

Re: [libav-devel] [PATCH 1/4] build: Rename aandct component to more descriptive aandcttables

2012-07-26 Thread Diego Biurrun
On Wed, Jul 25, 2012 at 12:08:54AM +0200, Diego Biurrun wrote: --- Yes, Mans, it does strike me as peculiar and confusing. configure | 26 +- libavcodec/Makefile |2 +- 2 files changed, 14 insertions(+), 14 deletions(-) .. ping .. - this is really

Re: [libav-devel] [PATCH] Drop support for completely obsolete Gopher protocol

2012-07-26 Thread Diego Biurrun
On Mon, Jul 23, 2012 at 05:47:51PM +0200, Diego Biurrun wrote: --- configure|1 - doc/general.texi |1 - doc/protocols.texi |4 -- libavformat/Makefile |1 - libavformat/allformats.c |1 - libavformat/gopher.c | 125

Re: [libav-devel] [PATCH 3/3] doc: Clarify licensing issues arising from external libraries

2012-07-26 Thread Diego Biurrun
On Sun, Jul 22, 2012 at 12:17:30AM +0200, Diego Biurrun wrote: --- LICENSE | 37 - 1 files changed, 24 insertions(+), 13 deletions(-) .. ping .. Diego ___ libav-devel mailing list libav-devel@libav.org

Re: [libav-devel] [PATCH 1/3] configure: Drop special-case license handling of x11grab input device

2012-07-26 Thread Diego Biurrun
On Mon, Jul 23, 2012 at 11:35:30AM +0200, Diego Biurrun wrote: On Sun, Jul 22, 2012 at 02:04:10AM +0200, Luca Barbato wrote: On 07/22/2012 12:17 AM, Diego Biurrun wrote: All other GPLed internal components that are not glue code for external libraries are not special-cased either. ---

Re: [libav-devel] [PATCH 2/3] x11grab: cosmetics: consistent naming for x11grab-related things

2012-07-26 Thread Diego Biurrun
On Sun, Jul 22, 2012 at 12:17:29AM +0200, Diego Biurrun wrote: --- configure|2 +- libavdevice/Makefile |2 +- libavdevice/alldevices.c |2 +- libavdevice/x11grab.c| 25 - 4 files changed, 15 insertions(+), 16 deletions(-) ..

[libav-devel] [PATCH 1/3] Improve descriptiveness of a number of codec and container long names

2012-07-26 Thread Diego Biurrun
--- libavcodec/aacenc.c |2 +- libavcodec/amrnbdec.c|2 +- libavcodec/amrwbdec.c|2 +- libavcodec/bmp.c |2 +- libavcodec/bmpenc.c |2 +- libavcodec/cavsdec.c |2 +- libavcodec/cook.c|2 +-

[libav-devel] [PATCH 3/3] Drop redundant specifiers from class_name strings

2012-07-26 Thread Diego Biurrun
--- libavcodec/ac3dec.c |4 ++-- libavcodec/apedec.c |2 +- libavcodec/g722dec.c |2 +- libavcodec/g723_1.c |2 +- libavcodec/libmp3lame.c |2 +- libavcodec/libvpxenc.c|2 +- libavcodec/mjpegdec.c |2 +-

[libav-devel] [PATCH 1/2] rtmp: Prevent reading outside of an allocate buffer when receiving server bandwidth packets

2012-07-26 Thread Samuel Pitoiset
When the server sends a server bandwidth packet less than 4 bytes long we return a proper error code. --- libavformat/rtmpproto.c | 7 +++ 1 file changed, 7 insertions(+) diff --git a/libavformat/rtmpproto.c b/libavformat/rtmpproto.c index a2efe38..8d388ab 100644 ---

[libav-devel] [PATCH 2/2] rtmp: Allow having more unknown data at the end of a chunk size packet without failing

2012-07-26 Thread Samuel Pitoiset
When the server sends a chunk size packet less than 4 bytes long we return a proper error code. --- libavformat/rtmpproto.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/libavformat/rtmpproto.c b/libavformat/rtmpproto.c index 8d388ab..bfdd535 100644 ---

Re: [libav-devel] round of stray commits

2012-07-26 Thread Diego Biurrun
On Tue, Jul 24, 2012 at 11:46:07AM +0200, Janne Grunau wrote: On 2012-07-24 03:23:09 +0200, Diego Biurrun wrote: On Tue, Jul 24, 2012 at 02:03:00AM +0200, Diego Biurrun wrote: I took the Git server offline in roundabout a minute, so hopefully nothing should have picked up the stray

Re: [libav-devel] [PATCH 1/2] rtmp: Prevent reading outside of an allocate buffer when receiving server bandwidth packets

2012-07-26 Thread Martin Storsjö
On Thu, 26 Jul 2012, Samuel Pitoiset wrote: When the server sends a server bandwidth packet less than 4 bytes long we return a proper error code. This is better, but you're still excessively wordy. rtmp: Check the buffer length in handle_server_bw That would be enough, right? You don't need

[libav-devel] [PATCH 1/2] rtmp: Prevent reading outside of an allocate buffer when receiving server bandwidth packets

2012-07-26 Thread Samuel Pitoiset
--- libavformat/rtmpproto.c | 7 +++ 1 file changed, 7 insertions(+) diff --git a/libavformat/rtmpproto.c b/libavformat/rtmpproto.c index a2efe38..183afae 100644 --- a/libavformat/rtmpproto.c +++ b/libavformat/rtmpproto.c @@ -950,6 +950,13 @@ static int handle_server_bw(URLContext *s,

[libav-devel] [PATCH 2/2] rtmp: Allow having more unknown data at the end of a chunk size packet without failing

2012-07-26 Thread Samuel Pitoiset
--- libavformat/rtmpproto.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/libavformat/rtmpproto.c b/libavformat/rtmpproto.c index 183afae..4e5eddb 100644 --- a/libavformat/rtmpproto.c +++ b/libavformat/rtmpproto.c @@ -885,9 +885,9 @@ static int

Re: [libav-devel] [PATCH 2/3] x11grab: cosmetics: consistent naming for x11grab-related things

2012-07-26 Thread Luca Barbato
On 07/22/2012 12:17 AM, Diego Biurrun wrote: --- configure|2 +- libavdevice/Makefile |2 +- libavdevice/alldevices.c |2 +- libavdevice/x11grab.c| 25 - 4 files changed, 15 insertions(+), 16 deletions(-) Fine for me. --

Re: [libav-devel] [PATCH 3/3] doc: Clarify licensing issues arising from external libraries

2012-07-26 Thread Luca Barbato
On 07/22/2012 12:17 AM, Diego Biurrun wrote: --- LICENSE | 37 - 1 files changed, 24 insertions(+), 13 deletions(-) So far ok but +The Fraunhofer AAC library, FAAC and OpenSSL are under licenses incompatible +with all (L)GPL versions. Thus,

Re: [libav-devel] [PATCH v4] build: do full flag handling for all compiler-type tools

2012-07-26 Thread Måns Rullgård
Diego Biurrun di...@biurrun.de writes: On Wed, Jul 25, 2012 at 11:33:43PM +0100, Måns Rullgård wrote: Diego Biurrun di...@biurrun.de writes: On Wed, Jul 25, 2012 at 02:10:05PM +0100, Mans Rullgard wrote: This adds a full identification probe of CC, AS, LD and HOSTCC, and sets up correct

Re: [libav-devel] [PATCH 1/2] rtmp: Prevent reading outside of an allocate buffer when receiving server bandwidth packets

2012-07-26 Thread Martin Storsjö
On Thu, 26 Jul 2012, Samuel Pitoiset wrote: --- libavformat/rtmpproto.c | 7 +++ 1 file changed, 7 insertions(+) diff --git a/libavformat/rtmpproto.c b/libavformat/rtmpproto.c index a2efe38..183afae 100644 --- a/libavformat/rtmpproto.c +++ b/libavformat/rtmpproto.c @@ -950,6 +950,13 @@

Re: [libav-devel] [PATCH 2/2] rtmp: Allow having more unknown data at the end of a chunk size packet without failing

2012-07-26 Thread Martin Storsjö
On Thu, 26 Jul 2012, Samuel Pitoiset wrote: --- libavformat/rtmpproto.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/libavformat/rtmpproto.c b/libavformat/rtmpproto.c index 183afae..4e5eddb 100644 --- a/libavformat/rtmpproto.c +++ b/libavformat/rtmpproto.c @@ -885,9

Re: [libav-devel] [PATCH 1/3] configure: Drop special-case license handling of x11grab input device

2012-07-26 Thread Luca Barbato
On 07/26/2012 11:54 AM, Diego Biurrun wrote: On Mon, Jul 23, 2012 at 11:35:30AM +0200, Diego Biurrun wrote: On Sun, Jul 22, 2012 at 02:04:10AM +0200, Luca Barbato wrote: On 07/22/2012 12:17 AM, Diego Biurrun wrote: All other GPLed internal components that are not glue code for external

Re: [libav-devel] [PATCH 1/4] build: Rename aandct component to more descriptive aandcttables

2012-07-26 Thread Luca Barbato
On 07/26/2012 11:25 AM, Diego Biurrun wrote: On Wed, Jul 25, 2012 at 12:08:54AM +0200, Diego Biurrun wrote: --- Yes, Mans, it does strike me as peculiar and confusing. configure | 26 +- libavcodec/Makefile |2 +- 2 files changed, 14 insertions(+),

Re: [libav-devel] [PATCH] msvc: check for snprintf

2012-07-26 Thread Ronald S. Bultje
Hi, On Wed, Jul 25, 2012 at 10:32 PM, Luca Barbato lu_z...@gentoo.org wrote: From: Ronald S. Bultje rsbul...@gmail.com --- Here my initial twist about it, ideally I'd consider moving os_support in libavu and include it automagically from config.h I'm not sure why, we do similar hacks for

Re: [libav-devel] [PATCH] eval: fix printing of NaN in eval fate test.

2012-07-26 Thread Ronald S. Bultje
Hi, On Wed, Jul 25, 2012 at 11:05 PM, Alex Converse alex.conve...@gmail.com wrote: On Wed, Jul 25, 2012 at 8:42 PM, Ronald S. Bultje rsbul...@gmail.com wrote: From: Ronald S. Bultje rsbul...@gmail.com This fixes make fate-eval on MSVC builds. Without this, the test outputs -1.#NaN instead

Re: [libav-devel] [PATCH] swscale: bury one more piece of inline asm under HAVE_INLINE_ASM.

2012-07-26 Thread Ronald S. Bultje
Hi, On Thu, Jul 26, 2012 at 2:06 AM, Diego Biurrun di...@biurrun.de wrote: On Thu, Jul 26, 2012 at 05:10:10AM +0200, Luca Barbato wrote: On 07/26/2012 04:27 AM, Ronald S. Bultje wrote: From: Ronald S. Bultje rsbul...@gmail.com --- libswscale/swscale.c |2 +- 1 file changed, 1

Re: [libav-devel] [PATCH] x86/dsputilenc: bury inline asm under HAVE_INLINE_ASM.

2012-07-26 Thread Ronald S. Bultje
Hi, On Thu, Jul 26, 2012 at 2:23 AM, Måns Rullgård m...@mansr.com wrote: Ronald S. Bultje rsbul...@gmail.com writes: From: Ronald S. Bultje rsbul...@gmail.com --- libavcodec/dct-test.c |2 +- libavcodec/x86/dsputilenc_mmx.c | 80 +++

Re: [libav-devel] [PATCH] swscale: bury one more piece of inline asm under HAVE_INLINE_ASM.

2012-07-26 Thread Måns Rullgård
Ronald S. Bultje rsbul...@gmail.com writes: Hi, On Thu, Jul 26, 2012 at 2:06 AM, Diego Biurrun di...@biurrun.de wrote: On Thu, Jul 26, 2012 at 05:10:10AM +0200, Luca Barbato wrote: On 07/26/2012 04:27 AM, Ronald S. Bultje wrote: From: Ronald S. Bultje rsbul...@gmail.com ---

Re: [libav-devel] [PATCH] x86/dsputilenc: bury inline asm under HAVE_INLINE_ASM.

2012-07-26 Thread Måns Rullgård
Ronald S. Bultje rsbul...@gmail.com writes: Hi, On Thu, Jul 26, 2012 at 2:23 AM, Måns Rullgård m...@mansr.com wrote: Ronald S. Bultje rsbul...@gmail.com writes: From: Ronald S. Bultje rsbul...@gmail.com --- libavcodec/dct-test.c |2 +- libavcodec/x86/dsputilenc_mmx.c |

[libav-devel] [PATCH] flvdec: remove spurious use of stream id

2012-07-26 Thread Luca Barbato
We match streams by codec id now. --- libavformat/flvdec.c | 15 +++ 1 files changed, 7 insertions(+), 8 deletions(-) diff --git a/libavformat/flvdec.c b/libavformat/flvdec.c index 12c2504..3382454 100644 --- a/libavformat/flvdec.c +++ b/libavformat/flvdec.c @@ -69,12 +69,11 @@

Re: [libav-devel] [PATCH] swscale: bury one more piece of inline asm under HAVE_INLINE_ASM.

2012-07-26 Thread Ronald S. Bultje
Hi, On Thu, Jul 26, 2012 at 2:06 AM, Diego Biurrun di...@biurrun.de wrote: On Thu, Jul 26, 2012 at 05:10:10AM +0200, Luca Barbato wrote: On 07/26/2012 04:27 AM, Ronald S. Bultje wrote: From: Ronald S. Bultje rsbul...@gmail.com --- libswscale/swscale.c |2 +- 1 file changed, 1

Re: [libav-devel] [PATCH] swscale: bury one more piece of inline asm under HAVE_INLINE_ASM.

2012-07-26 Thread Martin Storsjö
On Thu, 26 Jul 2012, Ronald S. Bultje wrote: Hi, On Thu, Jul 26, 2012 at 2:06 AM, Diego Biurrun di...@biurrun.de wrote: On Thu, Jul 26, 2012 at 05:10:10AM +0200, Luca Barbato wrote: On 07/26/2012 04:27 AM, Ronald S. Bultje wrote: From: Ronald S. Bultje rsbul...@gmail.com ---

Re: [libav-devel] [PATCH] swscale: bury one more piece of inline asm under HAVE_INLINE_ASM.

2012-07-26 Thread Ronald S. Bultje
Hi, On Thu, Jul 26, 2012 at 7:30 AM, Martin Storsjö mar...@martin.st wrote: On Thu, 26 Jul 2012, Ronald S. Bultje wrote: Hi, On Thu, Jul 26, 2012 at 2:06 AM, Diego Biurrun di...@biurrun.de wrote: On Thu, Jul 26, 2012 at 05:10:10AM +0200, Luca Barbato wrote: On 07/26/2012 04:27 AM, Ronald

Re: [libav-devel] [PATCH v4] build: do full flag handling for all compiler-type tools

2012-07-26 Thread Derek Buitenhuis
On 26/07/2012 1:27 AM, Måns Rullgård wrote: You actually can use its own assembler with -integrated-as. That's something else. With -integrated-as, clang/llvm outputs machine code directly without going through a separate assembler. It can't be used to assemble code in a .s file. Seems

Re: [libav-devel] [libav-commits] x86: fft: convert sse inline asm to yasm

2012-07-26 Thread John Stebbins
On 07/25/2012 07:38 PM, John Stebbins wrote: On 07/25/2012 12:05 AM, Jason Garrett-Glaser wrote: On Tue, Jul 24, 2012 at 9:02 AM, John Stebbins stebb...@jetheaddev.com wrote: On 07/24/2012 05:53 PM, Jason Garrett-Glaser wrote: On Tue, Jul 24, 2012 at 8:34 AM, Måns Rullgård m...@mansr.com

[libav-devel] [PATCH] rtmp: Check the buffer length in handle_ping

2012-07-26 Thread Samuel Pitoiset
--- libavformat/rtmpproto.c | 6 ++ 1 file changed, 6 insertions(+) diff --git a/libavformat/rtmpproto.c b/libavformat/rtmpproto.c index a2efe38..a32c4a9 100644 --- a/libavformat/rtmpproto.c +++ b/libavformat/rtmpproto.c @@ -915,6 +915,12 @@ static int handle_ping(URLContext *s, RTMPPacket

Re: [libav-devel] [PATCH] swscale: bury one more piece of inline asm under HAVE_INLINE_ASM.

2012-07-26 Thread Måns Rullgård
Ronald S. Bultje rsbul...@gmail.com writes: Hi, On Thu, Jul 26, 2012 at 7:30 AM, Martin Storsjö mar...@martin.st wrote: On Thu, 26 Jul 2012, Ronald S. Bultje wrote: Hi, On Thu, Jul 26, 2012 at 2:06 AM, Diego Biurrun di...@biurrun.de wrote: On Thu, Jul 26, 2012 at 05:10:10AM +0200, Luca

Re: [libav-devel] [PATCH v4] build: do full flag handling for all compiler-type tools

2012-07-26 Thread Måns Rullgård
Derek Buitenhuis derek.buitenh...@gmail.com writes: On 26/07/2012 1:27 AM, Måns Rullgård wrote: You actually can use its own assembler with -integrated-as. That's something else. With -integrated-as, clang/llvm outputs machine code directly without going through a separate assembler. It

Re: [libav-devel] [PATCH] flvdec: remove spurious use of stream id

2012-07-26 Thread Justin Ruggles
On 07/26/2012 10:18 AM, Luca Barbato wrote: We match streams by codec id now. --- libavformat/flvdec.c | 15 +++ 1 files changed, 7 insertions(+), 8 deletions(-) diff --git a/libavformat/flvdec.c b/libavformat/flvdec.c index 12c2504..3382454 100644 ---

Re: [libav-devel] [PATCH] swscale: bury one more piece of inline asm under HAVE_INLINE_ASM.

2012-07-26 Thread Ronald S. Bultje
Hi, On Thu, Jul 26, 2012 at 9:05 AM, Måns Rullgård m...@mansr.com wrote: Ronald S. Bultje rsbul...@gmail.com writes: On Thu, Jul 26, 2012 at 7:30 AM, Martin Storsjö mar...@martin.st wrote: On Thu, 26 Jul 2012, Ronald S. Bultje wrote: On Thu, Jul 26, 2012 at 2:06 AM, Diego Biurrun

Re: [libav-devel] [PATCH] avf: introduce zerolatency option

2012-07-26 Thread Josh Allmann
From: Luca Barbato lu_z...@gentoo.org Date: Thu, 26 Jul 2012 09:39:55 -0700 Subject: [PATCH] avf: introduce nobuffer option Useful in cases where a significant analyzeduration is still needed, while minimizing buffering before output. An example is processing low-latency streams where all media

Re: [libav-devel] [PATCH] rtmp: Check the buffer length in handle_ping

2012-07-26 Thread Martin Storsjö
On Thu, 26 Jul 2012, Samuel Pitoiset wrote: --- libavformat/rtmpproto.c | 6 ++ 1 file changed, 6 insertions(+) diff --git a/libavformat/rtmpproto.c b/libavformat/rtmpproto.c index a2efe38..a32c4a9 100644 --- a/libavformat/rtmpproto.c +++ b/libavformat/rtmpproto.c @@ -915,6 +915,12 @@

Re: [libav-devel] [PATCH] rtmp: Check the buffer length in handle_ping

2012-07-26 Thread Samuel Pitoiset
On Thu, Jul 26, 2012 at 7:27 PM, Martin Storsjö mar...@martin.st wrote: On Thu, 26 Jul 2012, Samuel Pitoiset wrote: --- libavformat/rtmpproto.c | 6 ++ 1 file changed, 6 insertions(+) diff --git a/libavformat/rtmpproto.c b/libavformat/rtmpproto.c index a2efe38..a32c4a9 100644 ---

Re: [libav-devel] [PATCH] rtmp: Check the buffer length in handle_ping

2012-07-26 Thread Samuel Pitoiset
On Thu, Jul 26, 2012 at 8:17 PM, Martin Storsjö mar...@martin.st wrote: On Thu, 26 Jul 2012, Samuel Pitoiset wrote: On Thu, Jul 26, 2012 at 7:27 PM, Martin Storsjö mar...@martin.st wrote: On Thu, 26 Jul 2012, Samuel Pitoiset wrote: --- libavformat/rtmpproto.c | 6 ++ 1 file changed, 6

[libav-devel] [PATCH] tscc2: fix typo in array index

2012-07-26 Thread Kostya Shishkov
--- libavcodec/tscc2.c |4 ++-- 1 files changed, 2 insertions(+), 2 deletions(-) diff --git a/libavcodec/tscc2.c b/libavcodec/tscc2.c index 5248c9f..42f7837 100644 --- a/libavcodec/tscc2.c +++ b/libavcodec/tscc2.c @@ -298,8 +298,8 @@ static int tscc2_decode_frame(AVCodecContext *avctx, void

Re: [libav-devel] [PATCH] rtmp: Check the buffer length in handle_ping

2012-07-26 Thread Samuel Pitoiset
On Thu, Jul 26, 2012 at 8:31 PM, Samuel Pitoiset samuel.pitoi...@gmail.com wrote: On Thu, Jul 26, 2012 at 8:25 PM, Martin Storsjö mar...@martin.st wrote: On Thu, 26 Jul 2012, Samuel Pitoiset wrote: On Thu, Jul 26, 2012 at 8:17 PM, Martin Storsjö mar...@martin.st wrote: On Thu, 26 Jul 2012,

Re: [libav-devel] [PATCH] rtmp: Check the buffer length in handle_ping

2012-07-26 Thread Martin Storsjö
On Thu, 26 Jul 2012, Samuel Pitoiset wrote: On Thu, Jul 26, 2012 at 8:25 PM, Martin Storsjö mar...@martin.st wrote: On Thu, 26 Jul 2012, Samuel Pitoiset wrote: On Thu, Jul 26, 2012 at 8:17 PM, Martin Storsjö mar...@martin.st wrote: On Thu, 26 Jul 2012, Samuel Pitoiset wrote: On Thu, Jul

Re: [libav-devel] [PATCH] rtmp: Check the buffer length in handle_ping

2012-07-26 Thread Martin Storsjö
On Thu, 26 Jul 2012, Samuel Pitoiset wrote: On Thu, Jul 26, 2012 at 8:31 PM, Samuel Pitoiset samuel.pitoi...@gmail.com wrote: On Thu, Jul 26, 2012 at 8:25 PM, Martin Storsjö mar...@martin.st wrote: On Thu, 26 Jul 2012, Samuel Pitoiset wrote: On Thu, Jul 26, 2012 at 8:17 PM, Martin Storsjö

Re: [libav-devel] [PATCH] rtmp: Check the buffer length in handle_ping

2012-07-26 Thread Samuel Pitoiset
On Thu, Jul 26, 2012 at 8:37 PM, Martin Storsjö mar...@martin.st wrote: On Thu, 26 Jul 2012, Samuel Pitoiset wrote: On Thu, Jul 26, 2012 at 8:31 PM, Samuel Pitoiset samuel.pitoi...@gmail.com wrote: On Thu, Jul 26, 2012 at 8:25 PM, Martin Storsjö mar...@martin.st wrote: On Thu, 26 Jul 2012,

[libav-devel] [PATCH] rtmp: Check the buffer length of ping packets

2012-07-26 Thread Samuel Pitoiset
--- libavformat/rtmpproto.c | 12 1 file changed, 12 insertions(+) diff --git a/libavformat/rtmpproto.c b/libavformat/rtmpproto.c index a2efe38..03accfd 100644 --- a/libavformat/rtmpproto.c +++ b/libavformat/rtmpproto.c @@ -515,6 +515,12 @@ static int gen_pong(URLContext *s,

[libav-devel] [PATCH] rtmp: Check the buffer length of ping packets

2012-07-26 Thread Samuel Pitoiset
--- libavformat/rtmpproto.c | 12 1 file changed, 12 insertions(+) diff --git a/libavformat/rtmpproto.c b/libavformat/rtmpproto.c index a2efe38..ce85892 100644 --- a/libavformat/rtmpproto.c +++ b/libavformat/rtmpproto.c @@ -515,6 +515,12 @@ static int gen_pong(URLContext *s,

Re: [libav-devel] [PATCH] rtmp: Check the buffer length of ping packets

2012-07-26 Thread Samuel Pitoiset
On Thu, Jul 26, 2012 at 8:45 PM, Samuel Pitoiset samuel.pitoi...@gmail.com wrote: --- libavformat/rtmpproto.c | 12 1 file changed, 12 insertions(+) Sorry for the double post, I just fixed the coding style. -- Best regards, Samuel Pitoiset.

Re: [libav-devel] [PATCH] rtmp: Check the buffer length of ping packets

2012-07-26 Thread Martin Storsjö
On Thu, 26 Jul 2012, Samuel Pitoiset wrote: --- Patch ok, and use git send-email --annotate and write a comment in this area of the patch explaining the difference if you repost several versions of a patch, to explain what changed and why. All text in this area is left out from the actual

[libav-devel] [PATCH] rtmp: Add case for metadata packets to avoid undesired debug output in rtmp_parse_result

2012-07-26 Thread Samuel Pitoiset
--- libavformat/rtmpproto.c | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/libavformat/rtmpproto.c b/libavformat/rtmpproto.c index a2efe38..7ff3117 100644 --- a/libavformat/rtmpproto.c +++ b/libavformat/rtmpproto.c @@ -1095,7 +1095,8 @@ static int

Re: [libav-devel] [PATCH] x86inc: automatically insert vzeroupper for YMM functions.

2012-07-26 Thread Loren Merritt
On Wed, 25 Jul 2012, Ronald S. Bultje wrote: diff --git a/libavcodec/x86/dct32_sse.asm b/libavcodec/x86/dct32_sse.asm index e3c8a45..351c88d 100644 --- a/libavcodec/x86/dct32_sse.asm +++ b/libavcodec/x86/dct32_sse.asm @@ -278,8 +278,6 @@ cglobal dct32_float_avx, 2,3,8, out, in, tmp

Re: [libav-devel] [PATCH] x86inc: automatically insert vzeroupper for YMM functions.

2012-07-26 Thread Justin Ruggles
On 07/25/2012 11:52 PM, Ronald S. Bultje wrote: From: Ronald S. Bultje rsbul...@gmail.com --- libavcodec/x86/dct32_sse.asm|2 -- libavcodec/x86/dsputil_yasm.asm | 14 -- libavcodec/x86/fft_mmx.asm |6 -- libavresample/x86/audio_convert.asm |

[libav-devel] [RFC] split HAVE_MMX/MMX2/SSE/SSE2/SSSE3/AVX for inline asm vs. yasm

2012-07-26 Thread Ronald S. Bultje
Hi guys, discussion thread. We currently use HAVE_SSSE3 and related macros to indicate that we want to compile these and that our compiler tools are good enough to know what to do with it. As a result, we currently use HAVE_AVX around all avx code (yasm only - we don't have any avx inline asm),

Re: [libav-devel] [PATCH 1/3] Improve descriptiveness of a number of codec and container long names

2012-07-26 Thread Diego Biurrun
On Thu, Jul 26, 2012 at 02:42:33PM +0200, Luca Barbato wrote: On 07/26/2012 11:57 AM, Diego Biurrun wrote: .flush = cavs_flush, -.long_name = NULL_IF_CONFIG_SMALL(Chinese AVS video (AVS1-P2, JiZhun profile)), +.long_name = NULL_IF_CONFIG_SMALL(Chinese

[libav-devel] [PATCH] Improve descriptiveness of a number of codec and container long names

2012-07-26 Thread Diego Biurrun
--- libavcodec/aacdec.c |2 +- libavcodec/aacenc.c |2 +- libavcodec/amrnbdec.c|2 +- libavcodec/amrwbdec.c|2 +- libavcodec/assdec.c |2 +- libavcodec/assenc.c |2 +- libavcodec/bmp.c |2 +-

Re: [libav-devel] [RFC] split HAVE_MMX/MMX2/SSE/SSE2/SSSE3/AVX for inline asm vs. yasm

2012-07-26 Thread Diego Biurrun
On Thu, Jul 26, 2012 at 01:50:17PM -0700, Ronald S. Bultje wrote: discussion thread. We currently use HAVE_SSSE3 and related macros to indicate that we want to compile these and that our compiler tools are good enough to know what to do with it. As a result, we currently use HAVE_AVX around

Re: [libav-devel] [PATCH 3/3] doc: Clarify licensing issues arising from external libraries

2012-07-26 Thread Diego Biurrun
On Thu, Jul 26, 2012 at 02:28:29PM +0200, Luca Barbato wrote: On 07/22/2012 12:17 AM, Diego Biurrun wrote: --- LICENSE | 37 - 1 files changed, 24 insertions(+), 13 deletions(-) So far ok but +The Fraunhofer AAC library, FAAC and OpenSSL are

Re: [libav-devel] [PATCH] tscc2: fix typo in array index

2012-07-26 Thread Derek Buitenhuis
On 26/07/2012 2:25 PM, Kostya Shishkov wrote: --- libavcodec/tscc2.c |4 ++-- 1 files changed, 2 insertions(+), 2 deletions(-) OK. How did this work at all before? - Derek ___ libav-devel mailing list libav-devel@libav.org

Re: [libav-devel] [PATCH] Improve descriptiveness of a number of codec and container long names

2012-07-26 Thread Luca Barbato
On 07/26/2012 11:12 PM, Diego Biurrun wrote: --- Fine for me. lu -- Luca Barbato Gentoo/linux http://dev.gentoo.org/~lu_zero ___ libav-devel mailing list libav-devel@libav.org https://lists.libav.org/mailman/listinfo/libav-devel

Re: [libav-devel] [PATCH] rtmp: Add case for metadata packets to avoid undesired debug output in rtmp_parse_result

2012-07-26 Thread Luca Barbato
On 07/26/2012 09:10 PM, Samuel Pitoiset wrote: --- libavformat/rtmpproto.c | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/libavformat/rtmpproto.c b/libavformat/rtmpproto.c index a2efe38..7ff3117 100644 --- a/libavformat/rtmpproto.c +++ b/libavformat/rtmpproto.c @@

Re: [libav-devel] [RFC] split HAVE_MMX/MMX2/SSE/SSE2/SSSE3/AVX for inline asm vs. yasm

2012-07-26 Thread Martin Storsjö
On Thu, 26 Jul 2012, Diego Biurrun wrote: On Thu, Jul 26, 2012 at 01:50:17PM -0700, Ronald S. Bultje wrote: discussion thread. We currently use HAVE_SSSE3 and related macros to indicate that we want to compile these and that our compiler tools are good enough to know what to do with it. As a

Re: [libav-devel] [RFC] split HAVE_MMX/MMX2/SSE/SSE2/SSSE3/AVX for inline asm vs. yasm

2012-07-26 Thread Luca Barbato
On 07/26/2012 11:53 PM, Martin Storsjö wrote: Currently avx has ssse3 as a dependency, where ssse3 is tested via inline asm while avx is tested via yasm. On MSVC the ssse3 inline asm test obviously fails, leading to HAVE_AVX=0, giving no AVX yasm either. Clearing it would be good. lu --

Re: [libav-devel] [PATCH 3/3] doc: Clarify licensing issues arising from external libraries

2012-07-26 Thread Luca Barbato
On 07/26/2012 11:40 PM, Diego Biurrun wrote: On Thu, Jul 26, 2012 at 02:28:29PM +0200, Luca Barbato wrote: On 07/22/2012 12:17 AM, Diego Biurrun wrote: --- LICENSE | 37 - 1 files changed, 24 insertions(+), 13 deletions(-) So far ok but +The

Re: [libav-devel] [PATCH] msvc: check for snprintf

2012-07-26 Thread Luca Barbato
On 07/26/2012 03:38 PM, Ronald S. Bultje wrote: I'm not sure why, we do similar hacks for pretty much all math functions and a few other string-related functions in lavu already. You refer to libm.h Why is snprintf() different? - it is not always_inline, so we could have clashes (fixed with

Re: [libav-devel] [PATCH] eval: fix printing of NaN in eval fate test.

2012-07-26 Thread Luca Barbato
On 07/26/2012 03:39 PM, Ronald S. Bultje wrote: Hi, On Wed, Jul 25, 2012 at 11:05 PM, Alex Converse alex.conve...@gmail.com wrote: On Wed, Jul 25, 2012 at 8:42 PM, Ronald S. Bultje rsbul...@gmail.com wrote: From: Ronald S. Bultje rsbul...@gmail.com This fixes make fate-eval on MSVC

Re: [libav-devel] [libav-commits] x86: fft: convert sse inline asm to yasm

2012-07-26 Thread Luca Barbato
On 07/26/2012 05:23 PM, John Stebbins wrote: Just confirming that the patch x86inc: automatically insert vzeroupper for YMM functions. solves this problem. Glad to know, thanks for reporting =) lu -- Luca Barbato Gentoo/linux http://dev.gentoo.org/~lu_zero

Re: [libav-devel] [RFC] split HAVE_MMX/MMX2/SSE/SSE2/SSSE3/AVX for inline asm vs. yasm

2012-07-26 Thread Ronald S. Bultje
Hi, On Thu, Jul 26, 2012 at 2:39 PM, Diego Biurrun di...@biurrun.de wrote: On Thu, Jul 26, 2012 at 01:50:17PM -0700, Ronald S. Bultje wrote: discussion thread. We currently use HAVE_SSSE3 and related macros to indicate that we want to compile these and that our compiler tools are good enough

[libav-devel] [PATCH 1/7] factor identical ff_inplace_start_frame out of two filters

2012-07-26 Thread Loren Merritt
--- libavfilter/vf_delogo.c | 38 +- libavfilter/vf_gradfun.c | 37 + libavfilter/video.c | 36 libavfilter/video.h |1 + 4 files changed, 39 insertions(+), 73

[libav-devel] [PATCH 2/7] vf_hqdn3d: cosmetics

2012-07-26 Thread Loren Merritt
Change code style to match the rest of libav. --- libavfilter/vf_hqdn3d.c | 308 +++ 1 files changed, 152 insertions(+), 156 deletions(-) diff --git a/libavfilter/vf_hqdn3d.c b/libavfilter/vf_hqdn3d.c index 9e01606..1fa70c6 100644 ---

[libav-devel] [PATCH 3/7] vf_hqdn3d: simplify and optimize

2012-07-26 Thread Loren Merritt
14% faster on penryn, 2% on sandybridge, 9% on bulldozer --- libavfilter/vf_hqdn3d.c | 157 +++--- 1 files changed, 51 insertions(+), 106 deletions(-) diff --git a/libavfilter/vf_hqdn3d.c b/libavfilter/vf_hqdn3d.c index 1fa70c6..bb81b79 100644 ---

[libav-devel] [PATCH 5/7] vf_hqdn3d: support 10bit colordepth

2012-07-26 Thread Loren Merritt
--- libavfilter/vf_hqdn3d.c | 68 +- 1 files changed, 49 insertions(+), 19 deletions(-) diff --git a/libavfilter/vf_hqdn3d.c b/libavfilter/vf_hqdn3d.c index 46902fa..02f0cd1 100644 --- a/libavfilter/vf_hqdn3d.c +++ b/libavfilter/vf_hqdn3d.c @@ -27,6

[libav-devel] [PATCH 6/7] vf_hqdn3d: support 16bit colordepth

2012-07-26 Thread Loren Merritt
--- libavfilter/vf_hqdn3d.c | 70 +++ 1 files changed, 46 insertions(+), 24 deletions(-) diff --git a/libavfilter/vf_hqdn3d.c b/libavfilter/vf_hqdn3d.c index 02f0cd1..521fef8 100644 --- a/libavfilter/vf_hqdn3d.c +++ b/libavfilter/vf_hqdn3d.c @@

[libav-devel] [PATCH 7/7] vf_hqdn3d: x86 asm

2012-07-26 Thread Loren Merritt
13% faster on penryn, 16% on sandybridge, 15% on bulldozer Not simd; a compiler should have generated this, but gcc didn't. --- libavfilter/vf_hqdn3d.c| 25 +-- libavfilter/x86/Makefile |1 + libavfilter/x86/hqdn3d.asm | 105

Re: [libav-devel] [RFC] split HAVE_MMX/MMX2/SSE/SSE2/SSSE3/AVX for inline asm vs. yasm

2012-07-26 Thread Diego Biurrun
On Thu, Jul 26, 2012 at 03:42:24PM -0700, Ronald S. Bultje wrote: On Thu, Jul 26, 2012 at 2:39 PM, Diego Biurrun di...@biurrun.de wrote: On Thu, Jul 26, 2012 at 01:50:17PM -0700, Ronald S. Bultje wrote: discussion thread. We currently use HAVE_SSSE3 and related macros to indicate that we

Re: [libav-devel] [PATCH v4] build: do full flag handling for all compiler-type tools

2012-07-26 Thread Diego Biurrun
On Thu, Jul 26, 2012 at 01:28:41PM +0100, Måns Rullgård wrote: Did you have any comments on the actual patch? The patch is OK, good work. Diego ___ libav-devel mailing list libav-devel@libav.org https://lists.libav.org/mailman/listinfo/libav-devel

[libav-devel] [PATCH 4/7] vf_hqdn3d: reduce intermediate precision

2012-07-26 Thread Loren Merritt
11% faster on penryn, 7% on sandybridge, 5% on bulldozer Negligible change to output. --- libavfilter/vf_hqdn3d.c | 62 -- 1 files changed, 32 insertions(+), 30 deletions(-) diff --git a/libavfilter/vf_hqdn3d.c b/libavfilter/vf_hqdn3d.c index

Re: [libav-devel] [PATCH 1/7] factor identical ff_inplace_start_frame out of two filters

2012-07-26 Thread Luca Barbato
On 07/27/2012 12:51 AM, Loren Merritt wrote: --- libavfilter/vf_delogo.c | 38 +- libavfilter/vf_gradfun.c | 37 + libavfilter/video.c | 36 libavfilter/video.h |

Re: [libav-devel] [PATCH 3/7] vf_hqdn3d: simplify and optimize

2012-07-26 Thread Ronald S. Bultje
Hi, On Thu, Jul 26, 2012 at 3:51 PM, Loren Merritt lor...@u.washington.edu wrote: 14% faster on penryn, 2% on sandybridge, 9% on bulldozer --- libavfilter/vf_hqdn3d.c | 157 +++--- 1 files changed, 51 insertions(+), 106 deletions(-) Looks good. I am

Re: [libav-devel] [PATCH 4/7] vf_hqdn3d: reduce intermediate precision

2012-07-26 Thread Ronald S. Bultje
Hi, On Thu, Jul 26, 2012 at 3:51 PM, Loren Merritt lor...@u.washington.edu wrote: 11% faster on penryn, 7% on sandybridge, 5% on bulldozer Negligible change to output. --- libavfilter/vf_hqdn3d.c | 62 -- 1 files changed, 32 insertions(+), 30

[libav-devel] [PATCH] avf: introduce nobuffer option

2012-07-26 Thread Josh Allmann
From: Luca Barbato lu_z...@gentoo.org Useful in cases where a significant analyzeduration is still needed, while minimizing buffering before output. An example is processing low-latency streams where all media types won't necessarily come in if the analyzeduration is small. --- Updated against

Re: [libav-devel] [PATCH 5/7] vf_hqdn3d: support 10bit colordepth

2012-07-26 Thread Ronald S. Bultje
Hi, On Thu, Jul 26, 2012 at 3:51 PM, Loren Merritt lor...@u.washington.edu wrote: --- libavfilter/vf_hqdn3d.c | 68 +- 1 files changed, 49 insertions(+), 19 deletions(-) Can you add 9bpp support also? Not that it's used much, but it'll use the

Re: [libav-devel] [RFC] split HAVE_MMX/MMX2/SSE/SSE2/SSSE3/AVX for inline asm vs. yasm

2012-07-26 Thread Ronald S. Bultje
Hi, On Thu, Jul 26, 2012 at 3:54 PM, Diego Biurrun di...@biurrun.de wrote: On Thu, Jul 26, 2012 at 03:42:24PM -0700, Ronald S. Bultje wrote: On Thu, Jul 26, 2012 at 2:39 PM, Diego Biurrun di...@biurrun.de wrote: On Thu, Jul 26, 2012 at 01:50:17PM -0700, Ronald S. Bultje wrote: discussion

[libav-devel] [PATCH 1/6] build: Only compile and run dct-test if AAN DCT tables are enabled

2012-07-26 Thread Diego Biurrun
--- libavcodec/Makefile |2 +- tests/fate/dct.mak |2 +- 2 files changed, 2 insertions(+), 2 deletions(-) diff --git a/libavcodec/Makefile b/libavcodec/Makefile index 3a74338..7f1cadb 100644 --- a/libavcodec/Makefile +++ b/libavcodec/Makefile @@ -723,13 +723,13 @@

[libav-devel] [PATCH 2/6] configure: Drop some special-casing for unsupported gcc 2.95

2012-07-26 Thread Diego Biurrun
--- configure |3 --- 1 files changed, 0 insertions(+), 3 deletions(-) diff --git a/configure b/configure index fd90369..4502fc0 100755 --- a/configure +++ b/configure @@ -2065,9 +2065,6 @@ probe_cc(){ gcc_pkg_ver=$(expr $gcc_version : '[^ ]* \(([^)]*)\)')

[libav-devel] [PATCH 3/6] build: Rename YASMDEP variable to DEPYASM for consistency

2012-07-26 Thread Diego Biurrun
--- Makefile|2 +- configure |2 +- library.mak |2 +- 3 files changed, 3 insertions(+), 3 deletions(-) diff --git a/Makefile b/Makefile index 3716e9e..37faa76 100644 --- a/Makefile +++ b/Makefile @@ -11,7 +11,7 @@ ifndef V Q = @ ECHO = printf $(1)\t%s\n $(2) BRIEF =

[libav-devel] [PATCH 4/6] build: Simplify CFLAGS handling in header compilation rule

2012-07-26 Thread Diego Biurrun
--- Makefile |2 +- 1 files changed, 1 insertions(+), 1 deletions(-) diff --git a/Makefile b/Makefile index 37faa76..93c6621 100644 --- a/Makefile +++ b/Makefile @@ -46,7 +46,7 @@ COMPILE_S = $(call COMPILE,AS) $(COMPILE_S) %.ho: %.h - $(CC) $(CPPFLAGS) $(CFLAGS) -c -o $@ -x

[libav-devel] [PATCH 5/6] configure: Place non-user-selectable components in CONFIG_EXTRA

2012-07-26 Thread Diego Biurrun
--- configure | 16 1 files changed, 8 insertions(+), 8 deletions(-) diff --git a/configure b/configure index b447897..515620c 100755 --- a/configure +++ b/configure @@ -935,7 +935,6 @@ PROGRAM_LIST= CONFIG_LIST= $COMPONENT_LIST $PROGRAM_LIST -ac3dsp

[libav-devel] [PATCH 6/6] configure: Add missing descriptions to help output

2012-07-26 Thread Diego Biurrun
--- configure |5 - 1 files changed, 4 insertions(+), 1 deletions(-) diff --git a/configure b/configure index 515620c..1287dc4 100755 --- a/configure +++ b/configure @@ -207,16 +207,18 @@ Advanced options (experts only): --ar=AR use archive tool AR [$ar_default]

Re: [libav-devel] [PATCH 1/6] build: Only compile and run dct-test if AAN DCT tables are enabled

2012-07-26 Thread Ronald S. Bultje
Hi, On Thu, Jul 26, 2012 at 5:15 PM, Diego Biurrun di...@biurrun.de wrote: --- libavcodec/Makefile |2 +- tests/fate/dct.mak |2 +- 2 files changed, 2 insertions(+), 2 deletions(-) This test tests a lot more than just aan dct? Ronald ___

Re: [libav-devel] [PATCH 3/7] vf_hqdn3d: simplify and optimize

2012-07-26 Thread Loren Merritt
On Thu, 26 Jul 2012, Ronald S. Bultje wrote: Hi, On Thu, Jul 26, 2012 at 3:51 PM, Loren Merritt lor...@u.washington.edu wrote: 14% faster on penryn, 2% on sandybridge, 9% on bulldozer --- libavfilter/vf_hqdn3d.c | 157 +++--- 1 files

[libav-devel] [PATCH 5/7] vf_hqdn3d: support 9 and 10bit colordepth

2012-07-26 Thread Loren Merritt
--- libavfilter/vf_hqdn3d.c | 72 ++ 1 files changed, 53 insertions(+), 19 deletions(-) diff --git a/libavfilter/vf_hqdn3d.c b/libavfilter/vf_hqdn3d.c index 46902fa..138b527 100644 --- a/libavfilter/vf_hqdn3d.c +++ b/libavfilter/vf_hqdn3d.c @@ -27,6

[libav-devel] [PATCH] x86inc: sync to latest version from x264.

2012-07-26 Thread Ronald S. Bultje
From: Ronald S. Bultje rsbul...@gmail.com --- libavutil/x86/x86inc.asm | 216 ++ 1 file changed, 124 insertions(+), 92 deletions(-) diff --git a/libavutil/x86/x86inc.asm b/libavutil/x86/x86inc.asm index b76a10c..23d9d57 100644 ---

  1   2   >