Re: [libav-devel] [PATCH 1/2] dv: use AVStream.index instead of abusing AVStream.id

2012-10-29 Thread Anton Khirnov
ping -- Anton Khirnov ___ libav-devel mailing list libav-devel@libav.org https://lists.libav.org/mailman/listinfo/libav-devel

Re: [libav-devel] [PATCH] nut: support textual data

2012-10-29 Thread Luca Barbato
On 10/29/12 1:09 AM, Janne Grunau wrote: On 2012-10-25 14:05:40 +0200, Luca Barbato wrote: Plain text (utf8 encoded) data can be muxed and demuxed in nut. --- libavformat/nut.c| 7 ++- libavformat/nut.h| 1 + libavformat/nutdec.c | 1 + 3 files changed, 8 insertions(+), 1

Re: [libav-devel] [PATCH 1/2] lavr: add general API usage doxy

2012-10-29 Thread Luca Barbato
On 10/28/12 10:52 PM, Anton Khirnov wrote: --- libavresample/avresample.h | 70 libavutil/avutil.h |1 + 2 files changed, 71 insertions(+), 0 deletions(-) Ok. ___ libav-devel mailing

Re: [libav-devel] [PATCH 2/2] lavr: document upper bound on number of output samples.

2012-10-29 Thread Luca Barbato
On 10/28/12 10:52 PM, Anton Khirnov wrote: --- libavresample/avresample.h |4 1 files changed, 4 insertions(+), 0 deletions(-) Seems fine. ___ libav-devel mailing list libav-devel@libav.org

Re: [libav-devel] [PATCH 1/2] dv: use AVStream.index instead of abusing AVStream.id

2012-10-29 Thread Luca Barbato
On 10/24/12 6:17 PM, Anton Khirnov wrote: --- libavformat/dv.c |2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/libavformat/dv.c b/libavformat/dv.c index 70786b1..17c545c 100644 --- a/libavformat/dv.c +++ b/libavformat/dv.c @@ -372,7 +372,7 @@ int

Re: [libav-devel] [PATCH 1/2] Remove usage of INIT_AVX in h264_intrapred_10bit.asm.

2012-10-29 Thread Luca Barbato
On 10/29/12 2:39 AM, Ronald S. Bultje wrote: From: Ronald S. Bultje rsbul...@gmail.com --- libavcodec/x86/h264_intrapred_10bit.asm | 306 libavcodec/x86/h264_intrapred_init.c| 40 ++--- 2 files changed, 177 insertions(+), 169 deletions(-) I'd name it

Re: [libav-devel] [libav-commits] lavc: add CODEC_CAP_DR1 to all video decoders missing them

2012-10-29 Thread Måns Rullgård
Anton Khirnov g...@libav.org writes: diff --git a/libavcodec/indeo3.c b/libavcodec/indeo3.c index fc0d2bc..322d360 100644 --- a/libavcodec/indeo3.c +++ b/libavcodec/indeo3.c @@ -1114,4 +1114,5 @@ AVCodec ff_indeo3_decoder = { .decode = decode_frame, .capabilities =

Re: [libav-devel] [PATCH 1/2] dv: use AVStream.index instead of abusing AVStream.id

2012-10-29 Thread Janne Grunau
On 2012-10-24 18:17:52 +0200, Anton Khirnov wrote: --- libavformat/dv.c |2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/libavformat/dv.c b/libavformat/dv.c index 70786b1..17c545c 100644 --- a/libavformat/dv.c +++ b/libavformat/dv.c @@ -372,7 +372,7 @@ int

Re: [libav-devel] [PATCH 1/2] dv: use AVStream.index instead of abusing AVStream.id

2012-10-29 Thread Anton Khirnov
On Mon, 29 Oct 2012 10:19:14 +0100, Luca Barbato lu_z...@gentoo.org wrote: On 10/24/12 6:17 PM, Anton Khirnov wrote: --- libavformat/dv.c |2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/libavformat/dv.c b/libavformat/dv.c index 70786b1..17c545c 100644 ---

Re: [libav-devel] [PATCH] nut: support textual data

2012-10-29 Thread Janne Grunau
On 2012-10-29 10:17:14 +0100, Luca Barbato wrote: On 10/29/12 1:09 AM, Janne Grunau wrote: On 2012-10-25 14:05:40 +0200, Luca Barbato wrote: Plain text (utf8 encoded) data can be muxed and demuxed in nut. --- libavformat/nut.c| 7 ++- libavformat/nut.h| 1 +

Re: [libav-devel] [libav-commits] lavc: add CODEC_CAP_DR1 to all video decoders missing them

2012-10-29 Thread Anton Khirnov
On Mon, 29 Oct 2012 10:37:31 +, Måns Rullgård m...@mansr.com wrote: Anton Khirnov g...@libav.org writes: diff --git a/libavcodec/indeo3.c b/libavcodec/indeo3.c index fc0d2bc..322d360 100644 --- a/libavcodec/indeo3.c +++ b/libavcodec/indeo3.c @@ -1114,4 +1114,5 @@ AVCodec

[libav-devel] [PATCH] indeo3: remove duplicate capabilities line.

2012-10-29 Thread Anton Khirnov
--- libavcodec/indeo3.c |1 - 1 file changed, 1 deletion(-) diff --git a/libavcodec/indeo3.c b/libavcodec/indeo3.c index 322d360..fc0d2bc 100644 --- a/libavcodec/indeo3.c +++ b/libavcodec/indeo3.c @@ -1114,5 +1114,4 @@ AVCodec ff_indeo3_decoder = { .decode = decode_frame,

Re: [libav-devel] [PATCH] indeo3: remove duplicate capabilities line.

2012-10-29 Thread Janne Grunau
On 2012-10-29 12:17:23 +0100, Anton Khirnov wrote: --- libavcodec/indeo3.c |1 - 1 file changed, 1 deletion(-) diff --git a/libavcodec/indeo3.c b/libavcodec/indeo3.c index 322d360..fc0d2bc 100644 --- a/libavcodec/indeo3.c +++ b/libavcodec/indeo3.c @@ -1114,5 +1114,4 @@ AVCodec

[libav-devel] [PATCH] nut: support textual data

2012-10-29 Thread Luca Barbato
Plain text (utf8 encoded) data can be muxed and demuxed in nut. --- Now with a section for data in the documentation as well. doc/nut.texi | 6 ++ libavformat/nut.c| 7 ++- libavformat/nut.h| 1 + libavformat/nutdec.c | 1 + 4 files changed, 14 insertions(+), 1

Re: [libav-devel] [PATCH] nut: support textual data

2012-10-29 Thread Janne Grunau
On 2012-10-29 13:16:20 +0100, Luca Barbato wrote: Plain text (utf8 encoded) data can be muxed and demuxed in nut. --- Now with a section for data in the documentation as well. doc/nut.texi | 6 ++ libavformat/nut.c| 7 ++- libavformat/nut.h| 1 +

Re: [libav-devel] [PATCH 06/11] network: use HAVE_THREADS instead of local hack

2012-10-29 Thread Diego Biurrun
On Sat, Oct 27, 2012 at 01:50:54AM +0100, Mans Rullgard wrote: HAVE_THREADS is set in config.h if pthreads or w32threads is available, which presumably the proper condition here. Also fixes undefined behaviour in preprocessor directives. Signed-off-by: Mans Rullgard m...@mansr.com ---

Re: [libav-devel] [PATCH 05/11] mov: make enum MovChannelLayoutTag definition standards compliant

2012-10-29 Thread Diego Biurrun
On Fri, Oct 26, 2012 at 06:29:47PM -0700, Diego Elio Pettenò wrote: On 26/10/2012 18:18, Måns Rullgård wrote: Only to follow the pattern. Should yours do the same then, or should one Diego at random send a patch to remove the rest? I nominate you :) Diego

Re: [libav-devel] [PATCH 1/2] lavr: add general API usage doxy

2012-10-29 Thread Diego Biurrun
On Sun, Oct 28, 2012 at 10:52:54PM +0100, Anton Khirnov wrote: --- a/libavresample/avresample.h +++ b/libavresample/avresample.h @@ -23,9 +23,75 @@ + * E.g. the following code will setup conversion from planar float sample format Starting a sentence with e.g. sounds weird to me. I'd

Re: [libav-devel] [PATCH 1/2] lavr: add general API usage doxy

2012-10-29 Thread Diego Biurrun
On Mon, Oct 29, 2012 at 02:58:02PM +0100, Diego Biurrun wrote: On Sun, Oct 28, 2012 at 10:52:54PM +0100, Anton Khirnov wrote: --- a/libavresample/avresample.h +++ b/libavresample/avresample.h @@ -23,9 +23,75 @@ + * E.g. the following code will setup conversion from planar float

Re: [libav-devel] [PATCH 1/1] doc: add apidoc target for doxygen API documentation

2012-10-29 Thread Diego Biurrun
On Sun, Oct 28, 2012 at 01:34:55AM +0200, Janne Grunau wrote: Documentation includes only the externally visible API of the installed headers. What about the rest? --- a/doc/Makefile +++ b/doc/Makefile @@ -14,6 +14,7 @@ DOCS = $(HTMLPAGES) $(MANPAGES) $(PODPAGES) all-$(CONFIG_DOC):

Re: [libav-devel] [PATCH] lavfi: add ashowinfo filter

2012-10-29 Thread Diego Biurrun
On Sun, Oct 28, 2012 at 11:02:26PM +0100, Anton Khirnov wrote: On Tue, 23 Oct 2012 21:55:13 +0200, Diego Biurrun di...@biurrun.de wrote: On Tue, Oct 23, 2012 at 09:40:08PM +0200, Anton Khirnov wrote: --- a/doc/filters.texi +++ b/doc/filters.texi @@ -175,6 +175,47 @@ stream ends. The

[libav-devel] [PATCH 1/1] doc: add apidoc target for doxygen API documentation

2012-10-29 Thread Janne Grunau
On 2012-10-29 15:03:45 +0100, Diego Biurrun wrote: On Sun, Oct 28, 2012 at 01:34:55AM +0200, Janne Grunau wrote: Documentation includes only the externally visible API of the installed headers. What about the rest? doxygen doc/Doxyfile If you prefer make support for propose a output

Re: [libav-devel] [PATCH 1/1] doc: add apidoc target for doxygen API documentation

2012-10-29 Thread Måns Rullgård
Janne Grunau janne-li...@jannau.net writes: Documentation includes only the externally visible API of the installed headers. Based on a patch by Anton Khirnov an...@khirnov.net. --- Makefile| 2 ++ doc/Makefile| 7 ++- doc/doxy-wrapper.sh | 14 ++

[libav-devel] [PATCH 2/8] lavu: add av_ctz() for trailing zero bit count

2012-10-29 Thread Justin Ruggles
--- doc/APIchanges |3 ++ libavutil/Makefile |2 +- libavutil/{log2.c = intmath.c} |5 +++ libavutil/intmath.h | 55 +++ libavutil/version.h |2 +- 5 files changed, 65 insertions(+), 2

Re: [libav-devel] [PATCH 30/45] qcelpdec: set channel layout

2012-10-29 Thread Justin Ruggles
On 10/23/2012 05:43 PM, Justin Ruggles wrote: --- libavcodec/qcelpdec.c |5 - 1 files changed, 4 insertions(+), 1 deletions(-) diff --git a/libavcodec/qcelpdec.c b/libavcodec/qcelpdec.c index edb1d24..9c2b354 100644 --- a/libavcodec/qcelpdec.c +++ b/libavcodec/qcelpdec.c @@

Re: [libav-devel] [PATCH 02/45] lavc: check channel count after decoder init

2012-10-29 Thread Justin Ruggles
On 10/23/2012 05:43 PM, Justin Ruggles wrote: Ensures the decoder did not set channel count to an insanely high value during initialization, which could cause large memory usage when it tries to get a buffer during decoding. --- libavcodec/utils.c |5 + 1 files changed, 5

Re: [libav-devel] [PATCH 19/45] flacdec: allow mid-stream channel layout change

2012-10-29 Thread Justin Ruggles
On 10/23/2012 05:43 PM, Justin Ruggles wrote: Although the libFLAC decoder cannot handle such a change, it is allowed by the spec and could potentially occur with live streams. --- libavcodec/flac.c| 19 +++ libavcodec/flac.h|3 +++

Re: [libav-devel] [PATCH 20/45] flacdec: do not warn on sample rate change

2012-10-29 Thread Justin Ruggles
On 10/23/2012 05:43 PM, Justin Ruggles wrote: --- libavcodec/flacdec.c |6 +- 1 files changed, 1 insertions(+), 5 deletions(-) diff --git a/libavcodec/flacdec.c b/libavcodec/flacdec.c index 20af820..10e33f2 100644 --- a/libavcodec/flacdec.c +++ b/libavcodec/flacdec.c @@ -460,12

Re: [libav-devel] [PATCH 30/45] qcelpdec: set channel layout

2012-10-29 Thread Kostya Shishkov
On Mon, Oct 29, 2012 at 10:31:01AM -0400, Justin Ruggles wrote: On 10/23/2012 05:43 PM, Justin Ruggles wrote: --- libavcodec/qcelpdec.c |5 - 1 files changed, 4 insertions(+), 1 deletions(-) diff --git a/libavcodec/qcelpdec.c b/libavcodec/qcelpdec.c index edb1d24..9c2b354

Re: [libav-devel] [PATCH 20/45] flacdec: do not warn on sample rate change

2012-10-29 Thread Kostya Shishkov
On Mon, Oct 29, 2012 at 10:34:36AM -0400, Justin Ruggles wrote: On 10/23/2012 05:43 PM, Justin Ruggles wrote: --- libavcodec/flacdec.c |6 +- 1 files changed, 1 insertions(+), 5 deletions(-) diff --git a/libavcodec/flacdec.c b/libavcodec/flacdec.c index 20af820..10e33f2

Re: [libav-devel] [PATCH 2/8] lavu: add av_ctz() for trailing zero bit count

2012-10-29 Thread Måns Rullgård
Justin Ruggles justin.rugg...@gmail.com writes: +#if HAVE_FAST_CLZ AV_GCC_VERSION_AT_LEAST(3,4) +#ifndef ff_ctz +#define ff_ctz(v) (__builtin_ctz(v)) Note that __builtin_ctz is undefined for an input of zero. Also, no need for the parens. +#endif +#endif + +#ifndef ff_ctz +#define

Re: [libav-devel] [PATCH 1/1] doc: add apidoc target for doxygen API documentation

2012-10-29 Thread Diego Biurrun
On Mon, Oct 29, 2012 at 03:19:27PM +0100, Janne Grunau wrote: On 2012-10-29 15:03:45 +0100, Diego Biurrun wrote: On Sun, Oct 28, 2012 at 01:34:55AM +0200, Janne Grunau wrote: Documentation includes only the externally visible API of the installed headers. What about the rest?

Re: [libav-devel] [PATCH 1/2] Remove usage of INIT_AVX in h264_intrapred_10bit.asm.

2012-10-29 Thread Diego Biurrun
On Sun, Oct 28, 2012 at 06:39:22PM -0700, Ronald S. Bultje wrote: From: Ronald S. Bultje rsbul...@gmail.com --- libavcodec/x86/h264_intrapred_10bit.asm | 306 libavcodec/x86/h264_intrapred_init.c| 40 ++--- 2 files changed, 177 insertions(+), 169

Re: [libav-devel] [PATCH 2/2] Remove INIT_AVX from x86inc.asm.

2012-10-29 Thread Diego Biurrun
On Sun, Oct 28, 2012 at 06:39:23PM -0700, Ronald S. Bultje wrote: From: Ronald S. Bultje rsbul...@gmail.com --- libavutil/x86/x86inc.asm | 8 1 file changed, 8 deletions(-) https://patches.libav.org/patch/25255/ Diego ___ libav-devel

Re: [libav-devel] [PATCH 2/2] Use ptrdiff_t instead of int for intra pred stride function parameter.

2012-10-29 Thread Diego Biurrun
On Sun, Oct 28, 2012 at 08:44:54PM -0700, Ronald S. Bultje wrote: From: Ronald S. Bultje rsbul...@gmail.com --- libavcodec/arm/h264pred_init_arm.c | 36 +++--- libavcodec/h264pred.c| 79 libavcodec/h264pred.h| 18 +--

Re: [libav-devel] [PATCH 2/2] Use ptrdiff_t instead of int for intra pred stride function parameter.

2012-10-29 Thread Ronald S. Bultje
Hi, On Mon, Oct 29, 2012 at 7:53 AM, Diego Biurrun di...@biurrun.de wrote: On Sun, Oct 28, 2012 at 08:44:54PM -0700, Ronald S. Bultje wrote: From: Ronald S. Bultje rsbul...@gmail.com --- libavcodec/arm/h264pred_init_arm.c | 36 +++--- libavcodec/h264pred.c| 79

Re: [libav-devel] [PATCH 2/8] lavu: add av_ctz() for trailing zero bit count

2012-10-29 Thread Justin Ruggles
On 10/29/2012 10:44 AM, Måns Rullgård wrote: Justin Ruggles justin.rugg...@gmail.com writes: +#if HAVE_FAST_CLZ AV_GCC_VERSION_AT_LEAST(3,4) +#ifndef ff_ctz +#define ff_ctz(v) (__builtin_ctz(v)) Note that __builtin_ctz is undefined for an input of zero. Also, no need for the parens.

Re: [libav-devel] [PATCH 1/1] doc: add apidoc target for doxygen API documentation

2012-10-29 Thread Janne Grunau
On 2012-10-29 15:45:43 +0100, Diego Biurrun wrote: On Mon, Oct 29, 2012 at 03:19:27PM +0100, Janne Grunau wrote: On 2012-10-29 15:03:45 +0100, Diego Biurrun wrote: On Sun, Oct 28, 2012 at 01:34:55AM +0200, Janne Grunau wrote: Documentation includes only the externally visible API of the

[libav-devel] [PATCH 1/1] doc: add apidoc target for doxygen API documentation

2012-10-29 Thread Janne Grunau
--- a/doc/Makefile +++ b/doc/Makefile @@ -41,6 +42,9 @@ doc/%.1: doc/%.pod $(GENTEXI) $(DOCS): | doc/ +doc/doxy/html: $(SRC_PATH)/doc/Doxyfile $(INSTHEADERS) | doc/ + $(M)$(SRC_PATH)/doc/doxy-wrapper.sh $(SRC_PATH) $^ Moving the order-only prerequisite to the $(DOCS) line

Re: [libav-devel] [PATCH 2/2] Use ptrdiff_t instead of int for intra pred stride function parameter.

2012-10-29 Thread Diego Biurrun
On Mon, Oct 29, 2012 at 08:06:31AM -0700, Ronald S. Bultje wrote: On Mon, Oct 29, 2012 at 7:53 AM, Diego Biurrun di...@biurrun.de wrote: On Sun, Oct 28, 2012 at 08:44:54PM -0700, Ronald S. Bultje wrote: From: Ronald S. Bultje rsbul...@gmail.com --- libavcodec/arm/h264pred_init_arm.c

Re: [libav-devel] [PATCH 1/1] doc: add apidoc target for doxygen API documentation

2012-10-29 Thread Diego Biurrun
On Mon, Oct 29, 2012 at 04:23:37PM +0100, Janne Grunau wrote: On 2012-10-29 15:45:43 +0100, Diego Biurrun wrote: On Mon, Oct 29, 2012 at 03:19:27PM +0100, Janne Grunau wrote: On 2012-10-29 15:03:45 +0100, Diego Biurrun wrote: On Sun, Oct 28, 2012 at 01:34:55AM +0200, Janne Grunau wrote:

Re: [libav-devel] [PATCH 05/11] mov: make enum MovChannelLayoutTag definition standards compliant

2012-10-29 Thread Alex Converse
On Fri, Oct 26, 2012 at 5:50 PM, Mans Rullgard m...@mansr.com wrote: The C99 standard requires that the expression that defines the value of an enumeration constant shall be an integer constant expression that has a value representable as an int. Signed-off-by: Mans Rullgard m...@mansr.com

Re: [libav-devel] [PATCH 05/11] mov: make enum MovChannelLayoutTag definition standards compliant

2012-10-29 Thread Måns Rullgård
Alex Converse alex.conve...@gmail.com writes: On Fri, Oct 26, 2012 at 5:50 PM, Mans Rullgard m...@mansr.com wrote: The C99 standard requires that the expression that defines the value of an enumeration constant shall be an integer constant expression that has a value representable as an int.

Re: [libav-devel] [PATCH 02/45] lavc: check channel count after decoder init

2012-10-29 Thread Luca Barbato
On 10/29/2012 03:32 PM, Justin Ruggles wrote: On 10/23/2012 05:43 PM, Justin Ruggles wrote: Ensures the decoder did not set channel count to an insanely high value during initialization, which could cause large memory usage when it tries to get a buffer during decoding. ---

Re: [libav-devel] [PATCH 1/2] Use PRED4x4/8x8/8x8L/16x16 macros to declare x86 intrapred prototypes.

2012-10-29 Thread Luca Barbato
On 10/29/2012 04:44 AM, Ronald S. Bultje wrote: From: Ronald S. Bultje rsbul...@gmail.com If you don't mind x86: use PRED4x4/8x8/8x8L/16x16 macros to declare intrapred prototypes might fit better. lu ___ libav-devel mailing list

[libav-devel] [PATCH] configure: Simplify check_64bit() invocation through a default test

2012-10-29 Thread Diego Biurrun
--- configure | 10 +- 1 files changed, 5 insertions(+), 5 deletions(-) diff --git a/configure b/configure index 1a006be..1aae7d6 100755 --- a/configure +++ b/configure @@ -2664,7 +2664,7 @@ check_host_cflags -Wall check_64bit(){ arch32=$1 arch64=$2 -expr=$3 +

Re: [libav-devel] [PATCH 1/2] Use PRED4x4/8x8/8x8L/16x16 macros to declare x86 intrapred prototypes.

2012-10-29 Thread Diego Biurrun
On Sun, Oct 28, 2012 at 08:44:53PM -0700, Ronald S. Bultje wrote: From: Ronald S. Bultje rsbul...@gmail.com --- libavcodec/x86/h264_intrapred.asm| 80 +- libavcodec/x86/h264_intrapred_init.c | 296 ++- 2 files changed, 190 insertions(+), 186

[libav-devel] [PATCH] cmdutils: Conditionally compile libswscale-related bits

2012-10-29 Thread Diego Biurrun
This fixes compilation with libswscale disabled. --- cmdutils.c |7 ++- 1 files changed, 6 insertions(+), 1 deletions(-) diff --git a/cmdutils.c b/cmdutils.c index 6570968..630301f 100644 --- a/cmdutils.c +++ b/cmdutils.c @@ -371,7 +371,10 @@ int opt_default(void *optctx, const char

Re: [libav-devel] [PATCH] cmdutils: Conditionally compile libswscale-related bits

2012-10-29 Thread Anton Khirnov
On Mon, 29 Oct 2012 18:38:23 +0100, Diego Biurrun di...@biurrun.de wrote: This fixes compilation with libswscale disabled. --- cmdutils.c |7 ++- 1 files changed, 6 insertions(+), 1 deletions(-) diff --git a/cmdutils.c b/cmdutils.c index 6570968..630301f 100644 --- a/cmdutils.c

Re: [libav-devel] [PATCH 06/12] fate: ac3: Place E-AC-3 tests and AC-3 tests in different groups

2012-10-29 Thread Justin Ruggles
On 10/20/2012 12:53 PM, Diego Biurrun wrote: --- fate-ac3 target not split anymore. tests/fate/ac3.mak | 21 - 1 files changed, 12 insertions(+), 9 deletions(-) diff --git a/tests/fate/ac3.mak b/tests/fate/ac3.mak index b2bfb8e..235414d 100644 ---

Re: [libav-devel] [PATCH 1/2] fate: Add shorthands for acodec PCM and ADPCM tests

2012-10-29 Thread Justin Ruggles
On 10/20/2012 09:46 AM, Diego Biurrun wrote: --- tests/fate/acodec.mak |8 ++-- 1 files changed, 6 insertions(+), 2 deletions(-) diff --git a/tests/fate/acodec.mak b/tests/fate/acodec.mak index 609559c..531f2f3 100644 --- a/tests/fate/acodec.mak +++ b/tests/fate/acodec.mak @@

Re: [libav-devel] [PATCH 09/10] avconv: Drop unused function argument from do_video_stats()

2012-10-29 Thread Diego Biurrun
On Fri, Oct 26, 2012 at 01:15:59PM +0200, Diego Biurrun wrote: --- avconv.c |5 ++--- 1 files changed, 2 insertions(+), 3 deletions(-) OKed by Anton on IRC. Diego ___ libav-devel mailing list libav-devel@libav.org

[libav-devel] [PATCH 1/4] aacdec: use float planar sample format for output

2012-10-29 Thread Justin Ruggles
--- libavcodec/aac.h|7 ++- libavcodec/aacdec.c | 102 ++ libavcodec/aacsbr.c |6 +-- 3 files changed, 67 insertions(+), 48 deletions(-) diff --git a/libavcodec/aac.h b/libavcodec/aac.h index 9c6ac27..6c5d962 100644 ---

Re: [libav-devel] [PATCH 2/3] lpc: Add a function for calculating reflection coefficients from samples

2012-10-29 Thread Martin Storsjö
On Sat, 27 Oct 2012, Martin Storsjö wrote: --- Is there any point in having this function, or should the caller just call these three functions manually? --- libavcodec/lpc.c | 12 libavcodec/lpc.h |3 +++ 2 files changed, 15 insertions(+) This (and 1/3) ok'd by Justin on

Re: [libav-devel] [PATCH 3/3] avcodec: Add a RFC 3389 comfort noise codec

2012-10-29 Thread Anton Khirnov
On Sat, 27 Oct 2012 23:34:17 +0300, Martin Storsjö mar...@martin.st wrote: This isn't too useful as a normal codec, but can be used in voip style applications. The decoder updates the noise generator parameters when a packet is given to it for decoding, but if called with an empty packet, it

Re: [libav-devel] [PATCH] lavfi: add ashowinfo filter

2012-10-29 Thread Justin Ruggles
On 10/23/2012 03:40 PM, Anton Khirnov wrote: It can be useful for debugging. Based on a patch by Stefano Sabatini stefano.sabatini-l...@poste.it --- Changelog |1 + doc/filters.texi | 41 ++ libavfilter/Makefile |1 +

Re: [libav-devel] [PATCH 1/2] Remove usage of INIT_AVX in h264_intrapred_10bit.asm.

2012-10-29 Thread Justin Ruggles
On 10/28/2012 09:39 PM, Ronald S. Bultje wrote: From: Ronald S. Bultje rsbul...@gmail.com --- libavcodec/x86/h264_intrapred_10bit.asm | 306 libavcodec/x86/h264_intrapred_init.c| 40 ++--- 2 files changed, 177 insertions(+), 169 deletions(-) The

Re: [libav-devel] [PATCH 1/6] float_dsp: add vector_dmul_scalar() to multiply a vector of doubles

2012-10-29 Thread Justin Ruggles
On 09/29/2012 01:17 AM, Justin Ruggles wrote: Include x86-optimized versions for SSE2 and AVX. --- libavutil/float_dsp.c |9 + libavutil/float_dsp.h | 15 +++ libavutil/x86/float_dsp.asm| 40

Re: [libav-devel] [PATCH 3/3] avcodec: Add a RFC 3389 comfort noise codec

2012-10-29 Thread Martin Storsjö
On Mon, 29 Oct 2012, Anton Khirnov wrote: On Sat, 27 Oct 2012 23:34:17 +0300, Martin Storsjö mar...@martin.st wrote: This isn't too useful as a normal codec, but can be used in voip style applications. The decoder updates the noise generator parameters when a packet is given to it for

[libav-devel] [PATCH] FATE: fix (AD)PCM test dependencies broken in e519990

2012-10-29 Thread Anton Khirnov
--- Not sure this is the right way, but it fixes FATE after a distclean for me --- tests/fate/acodec.mak |4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/tests/fate/acodec.mak b/tests/fate/acodec.mak index 531f2f3..71521ab 100644 --- a/tests/fate/acodec.mak +++

[libav-devel] [PATCH] x86: lavr: fix stack allocation for 7 and 8 channel downmixing on x86-32

2012-10-29 Thread Justin Ruggles
From: Ronald S. Bultje rsbul...@gmail.com Fixes crashes on Win32 and stack overruns on x86-32 in general. --- libavresample/x86/audio_mix.asm |7 +++ 1 files changed, 7 insertions(+), 0 deletions(-) diff --git a/libavresample/x86/audio_mix.asm b/libavresample/x86/audio_mix.asm index

[libav-devel] [PATCH] Remove usage of INIT_AVX in h264_intrapred_10bit.asm.

2012-10-29 Thread Ronald S. Bultje
From: Ronald S. Bultje rsbul...@gmail.com Replace INIT_AVX by INIT_XMM avx. Port the whole file to use cpuflag based function declarations. Remove (now unused) cputype argument in function declaration macros. Change function prototypes to have mmx2 instead of mmxext as suffix, since that's

Re: [libav-devel] [PATCH] Remove usage of INIT_AVX in h264_intrapred_10bit.asm.

2012-10-29 Thread Justin Ruggles
On 10/29/2012 05:13 PM, Ronald S. Bultje wrote: From: Ronald S. Bultje rsbul...@gmail.com Replace INIT_AVX by INIT_XMM avx. Port the whole file to use cpuflag based function declarations. Remove (now unused) cputype argument in function declaration macros. Change function prototypes to have

Re: [libav-devel] [PATCH 2/2] Remove INIT_AVX from x86inc.asm.

2012-10-29 Thread Justin Ruggles
On 10/29/2012 10:51 AM, Diego Biurrun wrote: On Sun, Oct 28, 2012 at 06:39:23PM -0700, Ronald S. Bultje wrote: From: Ronald S. Bultje rsbul...@gmail.com --- libavutil/x86/x86inc.asm | 8 1 file changed, 8 deletions(-) https://patches.libav.org/patch/25255/ both patches LGTM :)

Re: [libav-devel] [PATCH] FATE: fix (AD)PCM test dependencies broken in e519990

2012-10-29 Thread Diego Biurrun
On Mon, Oct 29, 2012 at 09:26:46PM +0100, Anton Khirnov wrote: --- Not sure this is the right way, but it fixes FATE after a distclean for me I wonder what FATE failure you are seeing - while after your patch the PCM tests are part of the fate-acodec target, which is probably better than not

Re: [libav-devel] [PATCH] FATE: fix (AD)PCM test dependencies broken in e519990

2012-10-29 Thread Martin Storsjö
On Mon, 29 Oct 2012, Diego Biurrun wrote: On Mon, Oct 29, 2012 at 09:26:46PM +0100, Anton Khirnov wrote: --- Not sure this is the right way, but it fixes FATE after a distclean for me I wonder what FATE failure you are seeing Most probably the issue that makes half of the instances at

Re: [libav-devel] [PATCH] FATE: fix (AD)PCM test dependencies broken in e519990

2012-10-29 Thread Diego Biurrun
On Tue, Oct 30, 2012 at 12:30:41AM +0200, Martin Storsjö wrote: On Mon, 29 Oct 2012, Diego Biurrun wrote: On Mon, Oct 29, 2012 at 09:26:46PM +0100, Anton Khirnov wrote: --- Not sure this is the right way, but it fixes FATE after a distclean for me I wonder what FATE failure you are seeing

Re: [libav-devel] [PATCH] FATE: fix (AD)PCM test dependencies broken in e519990

2012-10-29 Thread Diego Biurrun
On Mon, Oct 29, 2012 at 09:26:46PM +0100, Anton Khirnov wrote: --- Not sure this is the right way, but it fixes FATE after a distclean for me --- tests/fate/acodec.mak |4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) OK Diego ___

Re: [libav-devel] [PATCH 1/2] Use PRED4x4/8x8/8x8L/16x16 macros to declare x86 intrapred prototypes.

2012-10-29 Thread Ronald S. Bultje
Hi, On Mon, Oct 29, 2012 at 10:25 AM, Luca Barbato lu_z...@gentoo.org wrote: On 10/29/2012 04:44 AM, Ronald S. Bultje wrote: From: Ronald S. Bultje rsbul...@gmail.com If you don't mind x86: use PRED4x4/8x8/8x8L/16x16 macros to declare intrapred prototypes might fit better. Will change

[libav-devel] [PATCH] x86inc: Add cpuflags_mmxext alias for cpuflags_mmx2

2012-10-29 Thread Diego Biurrun
This allows using mmxext as name in Libav while staying compatible with changes to the YASM macro infrastructure imported from x264. --- libavutil/x86/x86inc.asm |1 + 1 files changed, 1 insertions(+), 0 deletions(-) diff --git a/libavutil/x86/x86inc.asm b/libavutil/x86/x86inc.asm index

Re: [libav-devel] [PATCH] x86inc: Add cpuflags_mmxext alias for cpuflags_mmx2

2012-10-29 Thread Ronald S. Bultje
Hi, On Mon, Oct 29, 2012 at 5:15 PM, Diego Biurrun di...@biurrun.de wrote: This allows using mmxext as name in Libav while staying compatible with changes to the YASM macro infrastructure imported from x264. --- libavutil/x86/x86inc.asm |1 + 1 files changed, 1 insertions(+), 0

Re: [libav-devel] [PATCH] x86inc: Add cpuflags_mmxext alias for cpuflags_mmx2

2012-10-29 Thread Diego Biurrun
On Mon, Oct 29, 2012 at 05:25:08PM -0700, Ronald S. Bultje wrote: On Mon, Oct 29, 2012 at 5:15 PM, Diego Biurrun di...@biurrun.de wrote: This allows using mmxext as name in Libav while staying compatible with changes to the YASM macro infrastructure imported from x264. ---

[libav-devel] [RFC] rtp: generate payload type from stream id

2012-10-29 Thread Luca Barbato
Makes possible to support multiple video/audio streams with different fmtp. --- It isn't working yet and I'm too tired to find the bug, the concept is there anyway. libavformat/internal.h | 4 ++-- libavformat/movenc.c | 2 +- libavformat/rtp.c | 5 +++-- libavformat/rtp.h | 9

[libav-devel] [PATCH 1/2] swscale: support gray to 9bit and 10bit formats

2012-10-29 Thread Luca Barbato
With the input of Kostya and Ronald. --- libswscale/swscale.c | 38 -- libswscale/swscale_unscaled.c | 32 ++-- 2 files changed, 66 insertions(+), 4 deletions(-) diff --git a/libswscale/swscale.c b/libswscale/swscale.c

[libav-devel] [PATCH 2/2] pixfmt: support more yuva formats

2012-10-29 Thread Luca Barbato
Signed-off-by: Luca Barbato lu_z...@gentoo.org --- libavcodec/raw.c | 19 libavformat/nut.c | 28 + libavutil/pixdesc.c | 234 ++ libavutil/pixfmt.h| 28 + libswscale/utils.c| 18

[libav-devel] [PATCH 2/4] x86inc: Only define program_name if the macro is unset

2012-10-29 Thread Diego Biurrun
This allows overriding the value from outside of the file. --- I consider this change fit for upstream. Loren? Jason? libavutil/x86/x86inc.asm |4 +++- 1 files changed, 3 insertions(+), 1 deletions(-) diff --git a/libavutil/x86/x86inc.asm b/libavutil/x86/x86inc.asm index 1fe9f55..b0df2b2

[libav-devel] [PATCH 3/4] x86inc: Set program_name outside of x86inc.asm

2012-10-29 Thread Diego Biurrun
This reduces the local difference to the x264 upstream version. --- libavutil/x86/x86inc.asm |2 +- libavutil/x86/x86util.asm |2 ++ 2 files changed, 3 insertions(+), 1 deletions(-) diff --git a/libavutil/x86/x86inc.asm b/libavutil/x86/x86inc.asm index b0df2b2..52ee46a 100644 ---

[libav-devel] [PATCH 1/4] x86: include x86inc.asm in x86util.asm

2012-10-29 Thread Diego Biurrun
This is necessary to allow refactoring some x86util macros with cpuflags. It also allows having custom macro definitions outside of x86inc.asm. --- libavcodec/x86/ac3dsp.asm |1 - libavcodec/x86/dct32.asm|1 - libavcodec/x86/deinterlace.asm |1 -

[libav-devel] [PATCH 4/4] x86util: Add cpuflags_mmxext alias for cpuflags_mmx2

2012-10-29 Thread Diego Biurrun
mmxext is a more sensible name and more common in outside projects. --- libavutil/x86/x86util.asm |1 + 1 files changed, 1 insertions(+), 0 deletions(-) diff --git a/libavutil/x86/x86util.asm b/libavutil/x86/x86util.asm index 5a11171..3e0443a 100644 --- a/libavutil/x86/x86util.asm +++

Re: [libav-devel] [PATCH] x86inc: Add cpuflags_mmxext alias for cpuflags_mmx2

2012-10-29 Thread Ronald S. Bultje
Hi, On Mon, Oct 29, 2012 at 5:56 PM, Diego Biurrun di...@biurrun.de wrote: On Mon, Oct 29, 2012 at 05:25:08PM -0700, Ronald S. Bultje wrote: On Mon, Oct 29, 2012 at 5:15 PM, Diego Biurrun di...@biurrun.de wrote: This allows using mmxext as name in Libav while staying compatible with changes

Re: [libav-devel] [PATCH 1/2] swscale: support gray to 9bit and 10bit formats

2012-10-29 Thread Ronald S. Bultje
Hi, On Mon, Oct 29, 2012 at 7:14 PM, Luca Barbato lu_z...@gentoo.org wrote: With the input of Kostya and Ronald. --- libswscale/swscale.c | 38 -- libswscale/swscale_unscaled.c | 32 ++-- 2 files changed, 66

Re: [libav-devel] [PATCH 2/2] pixfmt: support more yuva formats

2012-10-29 Thread Ronald S. Bultje
Hi, On Mon, Oct 29, 2012 at 7:14 PM, Luca Barbato lu_z...@gentoo.org wrote: Signed-off-by: Luca Barbato lu_z...@gentoo.org --- libavcodec/raw.c | 19 libavformat/nut.c | 28 + libavutil/pixdesc.c | 234

Re: [libav-devel] [PATCH 1/4] x86: include x86inc.asm in x86util.asm

2012-10-29 Thread Ronald S. Bultje
Hi, On Mon, Oct 29, 2012 at 7:28 PM, Diego Biurrun di...@biurrun.de wrote: -%include x86inc.asm %include x86util.asm Didn't you come up with the rule that each file should contain all headers it directly takes symbols from? How do these changes fit with that scheme? Ronald

Re: [libav-devel] [PATCH 4/4] x86util: Add cpuflags_mmxext alias for cpuflags_mmx2

2012-10-29 Thread Ronald S. Bultje
Hi, On Mon, Oct 29, 2012 at 7:28 PM, Diego Biurrun di...@biurrun.de wrote: mmxext is a more sensible name and more common in outside projects. --- libavutil/x86/x86util.asm |1 + 1 files changed, 1 insertions(+), 0 deletions(-) No. Ronald ___