Re: [libav-devel] [PATCH 09/13] FATE: add a test for amix audio filter.

2012-07-08 Thread Anton Khirnov
On Sat, 07 Jul 2012 17:28:02 +0200, Diego Biurrun di...@biurrun.de wrote: On Thu, Jul 05, 2012 at 09:57:35PM +0200, Anton Khirnov wrote: --- /dev/null +++ b/tests/fate/filter.mak @@ -0,0 +1,23 @@ +FATE_AMIX += fate-filter-amix-simple +fate-filter-amix-simple: CMD = avconv

Re: [libav-devel] [PATCH 4/4] graph2dot: don't use getopt().

2012-07-08 Thread Ronald S. Bultje
Hi, On Sat, Jul 7, 2012 at 1:47 PM, Måns Rullgård m...@mansr.com wrote: Ronald S. Bultje rsbul...@gmail.com wrote: On Sat, Jul 7, 2012 at 11:03 AM, Anton Khirnov an...@khirnov.net wrote: -while ((c = getopt(argc, argv, hi:o:)) != -1) { -switch (c) { -case 'h': +while

Re: [libav-devel] [PATCH] x86/timer: implement an intrinsic-based version for rdtsc (AV_READ_TIME).

2012-07-08 Thread Ronald S. Bultje
Hi, On Sat, Jul 7, 2012 at 1:49 PM, Måns Rullgård m...@mansr.com wrote: Ronald S. Bultje rsbul...@gmail.com wrote: On Fri, Jul 6, 2012 at 10:30 AM, Luca Barbato lu_z...@gentoo.org wrote: On 07/06/2012 07:22 PM, Ronald S. Bultje wrote: From: Ronald S. Bultje rsbul...@gmail.com Seems fine.

[libav-devel] [PATCH] libx264: support aspect ratio switching

2012-07-08 Thread Jan Ekström
From: JULIAN GARDNER joo...@btinternet.com --- libavcodec/libx264.c |6 ++ 1 files changed, 6 insertions(+), 0 deletions(-) diff --git a/libavcodec/libx264.c b/libavcodec/libx264.c index 35c0030..adda881 100644 --- a/libavcodec/libx264.c +++ b/libavcodec/libx264.c @@ -148,6 +148,12 @@

Re: [libav-devel] [PATCH] configure: check for math functions without linking to -lm also.

2012-07-08 Thread Ronald S. Bultje
Hi, On Sat, Jul 7, 2012 at 3:57 PM, Derek Buitenhuis derek.buitenh...@gmail.com wrote: On 07/07/2012 6:09 PM, Diego Biurrun wrote: And this is required - why? Libm / math is not A Thing in MSVC and isn't required. It complains about -lm, but ignores it. I think this is mostly to shut up a

Re: [libav-devel] [PATCH] configure: check for math functions without linking to -lm also.

2012-07-08 Thread Diego Biurrun
On Sun, Jul 08, 2012 at 07:49:22AM -0700, Ronald S. Bultje wrote: On Sat, Jul 7, 2012 at 3:57 PM, Derek Buitenhuis derek.buitenh...@gmail.com wrote: On 07/07/2012 6:09 PM, Diego Biurrun wrote: And this is required - why? Libm / math is not A Thing in MSVC and isn't required. It

[libav-devel] [PATCH 1/2] Add av_calloc() helper.

2012-07-08 Thread Tomas Härdin
Hi The attached patch adds av_calloc() to libavutil. See the mxfdec thread ([PATCH] mxfdec: replace xav_log2(sizeof(..)) by x/sizeof(..)). This allows for simpler, smaller code and smaller binaries. /Tomas From f614808c28a58c14906c81962871d2c2dfc73afe Mon Sep 17 00:00:00 2001 From: Laurent Aimar

[libav-devel] [PATCH 2/2] lavu: Bump minor version for av_calloc()

2012-07-08 Thread Tomas Härdin
From f247a5c2c4c61dc5b3fecf348d90f1655ad1bcbe Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?Tomas=20H=C3=A4rdin?= tomas.har...@codemill.se Date: Sun, 8 Jul 2012 16:17:37 +0200 Subject: [PATCH 2/2] lavu: Bump minor version for av_calloc() --- doc/APIchanges |3 +++ libavutil/version.h |2

[libav-devel] [PATCH] configure: Try linking against math library without -lm first

2012-07-08 Thread Diego Biurrun
On some systems -lm is not required to link against libmath. --- configure |3 ++- 1 files changed, 2 insertions(+), 1 deletions(-) diff --git a/configure b/configure index ec68648..7e38c9c 100755 --- a/configure +++ b/configure @@ -2931,9 +2931,10 @@ for thread in $THREADS_LIST; do fi

Re: [libav-devel] [PATCH] configure: Try linking against math library without -lm first

2012-07-08 Thread Diego Elio Pettenò
Il 08/07/2012 17:19, Diego Biurrun ha scritto: On some systems -lm is not required to link against libmath. LGTM. As a side note, IIRC Haiku has (or had) the same need. -- Diego Elio Pettenò — Flameeyes flamee...@flameeyes.eu — http://blog.flameeyes.eu/

Re: [libav-devel] [PATCH 1/2] Add av_calloc() helper.

2012-07-08 Thread Tomas Härdin
On Sun, 2012-07-08 at 18:06 +0200, Diego Biurrun wrote: On Sun, Jul 08, 2012 at 05:18:23PM +0200, Tomas Härdin wrote: --- a/libavutil/mem.h +++ b/libavutil/mem.h @@ -113,6 +113,18 @@ void av_free(void *ptr); /** + * Allocate a block of nmemb * size bytes with alignment suitable for

Re: [libav-devel] [PATCH 1/2] Add av_calloc() helper.

2012-07-08 Thread Diego Biurrun
On Sun, Jul 08, 2012 at 06:30:05PM +0200, Tomas Härdin wrote: On Sun, 2012-07-08 at 18:06 +0200, Diego Biurrun wrote: On Sun, Jul 08, 2012 at 05:18:23PM +0200, Tomas Härdin wrote: --- a/libavutil/mem.h +++ b/libavutil/mem.h @@ -113,6 +113,18 @@ void av_free(void *ptr); /**

Re: [libav-devel] [PATCH 1/2] Add av_calloc() helper.

2012-07-08 Thread Luca Barbato
On 07/08/2012 05:18 PM, Tomas Härdin wrote: I thought you were making it an inline. lu -- Luca Barbato Gentoo/linux http://dev.gentoo.org/~lu_zero ___ libav-devel mailing list libav-devel@libav.org

Re: [libav-devel] [PATCH] configure: Try linking against math library without -lm first

2012-07-08 Thread Dave Yeo
On 07/08/12 08:26 am, Diego Elio Pettenò wrote: Il 08/07/2012 17:19, Diego Biurrun ha scritto: On some systems -lm is not required to link against libmath. LGTM. As a side note, IIRC Haiku has (or had) the same need. OS/2 solves the issue by installing an empty libm. Dave

Re: [libav-devel] [PATCH 1/3] lavu: use intrinsic for emms on systems lacking inline asm support.

2012-07-08 Thread Måns Rullgård
Ronald S. Bultje rsbul...@gmail.com writes: From: Ronald S. Bultje rsbul...@gmail.com --- configure|3 +++ libavutil/internal.h |7 +-- 2 files changed, 8 insertions(+), 2 deletions(-) diff --git a/configure b/configure index 4c1cdc3..3d02e75 100755 ---

Re: [libav-devel] [PATCH] dct/fft-test: use a replacement getopt() if the system has none present.

2012-07-08 Thread Måns Rullgård
Ronald S. Bultje rsbul...@gmail.com writes: Hi, On Wed, Jul 4, 2012 at 2:09 PM, Måns Rullgård m...@mansr.com wrote: Ronald S. Bultje rsbul...@gmail.com writes: On Tue, Jul 3, 2012 at 8:14 PM, Ronald S. Bultje rsbul...@gmail.com wrote: From: Ronald S. Bultje rsbul...@gmail.com This allows

Re: [libav-devel] [PATCH] dct/fft-test: use a replacement getopt() if the system has none present.

2012-07-08 Thread Måns Rullgård
Luca Barbato lu_z...@gentoo.org writes: On 07/07/2012 06:59 AM, Ronald S. Bultje wrote: They're part of fate. I don't understand the trouble part. I already did all the effort. What more trouble could there possibly be? Is deciding where to put getopt.c too much trouble? I said already

Re: [libav-devel] [PATCH] x86/cpu: implement get/set_eflags using intrinsics.

2012-07-08 Thread Måns Rullgård
Ronald S. Bultje rsbul...@gmail.com writes: From: Ronald S. Bultje rsbul...@gmail.com --- configure |3 +++ libavutil/x86/cpu.c | 14 ++ 2 files changed, 17 insertions(+) diff --git a/configure b/configure index 598b33c..bb4d55f 100755 --- a/configure +++

Re: [libav-devel] [PATCH] snow: refactor code to work around a compiler bug in MSVC.

2012-07-08 Thread Måns Rullgård
Ronald S. Bultje rsbul...@gmail.com writes: From: Ronald S. Bultje rsbul...@gmail.com This fixes the compiler error cannot convert from 'BlockNode' to 'int16_t'. WTF? -- Måns Rullgård m...@mansr.com ___ libav-devel mailing list

Re: [libav-devel] [PATCH] configure: check for math functions without linking to -lm also.

2012-07-08 Thread Måns Rullgård
Ronald S. Bultje rsbul...@gmail.com writes: Hi, On Sat, Jul 7, 2012 at 3:57 PM, Derek Buitenhuis derek.buitenh...@gmail.com wrote: On 07/07/2012 6:09 PM, Diego Biurrun wrote: And this is required - why? Libm / math is not A Thing in MSVC and isn't required. It complains about -lm, but

Re: [libav-devel] [PATCH] configure: support linkers that expect non-standard library pre/suffixes.

2012-07-08 Thread Måns Rullgård
Ronald S. Bultje rsbul...@gmail.com writes: Hi, On Sat, Jul 7, 2012 at 2:03 PM, Ronald S. Bultje rsbul...@gmail.com wrote: From: Ronald S. Bultje rsbul...@gmail.com This is similar in style to how it currently supports some exotic compilers (e.g. see CC_O). --- configure | 13

Re: [libav-devel] [PATCH] configure: Try linking against math library without -lm first

2012-07-08 Thread Måns Rullgård
Diego Biurrun di...@biurrun.de writes: On some systems -lm is not required to link against libmath. --- configure |3 ++- 1 files changed, 2 insertions(+), 1 deletions(-) diff --git a/configure b/configure index ec68648..7e38c9c 100755 --- a/configure +++ b/configure @@ -2931,9

Re: [libav-devel] [PATCH] configure: Try linking against math library without -lm first

2012-07-08 Thread Måns Rullgård
Måns Rullgård m...@mansr.com writes: Diego Biurrun di...@biurrun.de writes: On some systems -lm is not required to link against libmath. --- configure |3 ++- 1 files changed, 2 insertions(+), 1 deletions(-) diff --git a/configure b/configure index ec68648..7e38c9c 100755 ---

Re: [libav-devel] [PATCH] dct/fft-test: use a replacement getopt() if the system has none present.

2012-07-08 Thread Måns Rullgård
Diego Biurrun di...@biurrun.de writes: On Fri, Jul 06, 2012 at 10:33:19AM -0700, Ronald S. Bultje wrote: On Fri, Jul 6, 2012 at 10:27 AM, Luca Barbato lu_z...@gentoo.org wrote: On 07/06/2012 07:13 PM, Ronald S. Bultje wrote: On Wed, Jul 4, 2012 at 2:09 PM, Måns Rullgård m...@mansr.com

Re: [libav-devel] [PATCH] libavcodec: implement LOCAL_ALIGNED() without variadic macro arguments.

2012-07-08 Thread Måns Rullgård
Ronald S. Bultje rsbul...@gmail.com writes: From: Ronald S. Bultje rsbul...@gmail.com MSVC has incomplete variadic macro argument handling, meaning it does support it to some extend, but it will basically handle all variadic arguments as a single argument, including the comma that separates

Re: [libav-devel] [PATCH 1/2] Add av_calloc() helper.

2012-07-08 Thread Måns Rullgård
Tomas Härdin tomas.har...@codemill.se writes: Hi The attached patch adds av_calloc() to libavutil. See the mxfdec thread ([PATCH] mxfdec: replace xav_log2(sizeof(..)) by x/sizeof(..)). This allows for simpler, smaller code and smaller binaries. /Tomas From

[libav-devel] Holiday

2012-07-08 Thread Måns Rullgård
I'm away on holiday for a week, which means I'll be slow replying to email. Please consider this before pushing changes I would normally want to review. -- Måns Rullgård m...@mansr.com ___ libav-devel mailing list libav-devel@libav.org

Re: [libav-devel] [PATCH] libavcodec: implement LOCAL_ALIGNED() without variadic macro arguments.

2012-07-08 Thread Luca Barbato
On 07/06/2012 07:55 PM, Ronald S. Bultje wrote: From: Ronald S. Bultje rsbul...@gmail.com MSVC has incomplete variadic macro argument handling, meaning it does support it to some extend, but it will basically handle all variadic arguments as a single argument, including the comma that

Re: [libav-devel] [PATCH] configure: Try linking against math library without -lm first

2012-07-08 Thread Måns Rullgård
Diego Biurrun di...@biurrun.de writes: On some systems -lm is not required to link against libmath. --- configure |3 ++- 1 files changed, 2 insertions(+), 1 deletions(-) diff --git a/configure b/configure index 0b0d532..c014668 100755 --- a/configure +++ b/configure @@ -2934,9

Re: [libav-devel] [PATCH] configure: check for math functions without linking to -lm also.

2012-07-08 Thread Måns Rullgård
Ronald S. Bultje rsbul...@gmail.com writes: Hi, On Sun, Jul 8, 2012 at 12:48 PM, Måns Rullgård m...@mansr.com wrote: Ronald S. Bultje rsbul...@gmail.com writes: Hi, On Sat, Jul 7, 2012 at 3:57 PM, Derek Buitenhuis derek.buitenh...@gmail.com wrote: On 07/07/2012 6:09 PM, Diego Biurrun

Re: [libav-devel] [PATCH] snow: refactor code to work around a compiler bug in MSVC.

2012-07-08 Thread Benjamin Larsson
On 07/07/2012 06:06 PM, Ronald S. Bultje wrote: From: Ronald S. Bultje rsbul...@gmail.com This fixes the compiler error cannot convert from 'BlockNode' to 'int16_t'. Ok, but maybe add a comment on why the code has to stay that way. MvH Benjamin Larsson

Re: [libav-devel] [PATCH] configure: check for math functions without linking to -lm also.

2012-07-08 Thread Diego Biurrun
On Sat, Jul 07, 2012 at 01:58:38PM -0700, Ronald S. Bultje wrote: --- a/configure +++ b/configure @@ -2943,7 +2943,9 @@ for thread in $THREADS_LIST; do -check_lib math.h sin -lm LIBM=-lm +if ! check_func_headers math.h sin; then +check_lib math.h sin -lm LIBM=-lm +fi I fear this

Re: [libav-devel] [PATCH] configure: check for math functions without linking to -lm also.

2012-07-08 Thread Måns Rullgård
Diego Biurrun di...@biurrun.de writes: On Sat, Jul 07, 2012 at 01:58:38PM -0700, Ronald S. Bultje wrote: --- a/configure +++ b/configure @@ -2943,7 +2943,9 @@ for thread in $THREADS_LIST; do -check_lib math.h sin -lm LIBM=-lm +if ! check_func_headers math.h sin; then +check_lib

[libav-devel] MSVC intrinsics patchset

2012-07-08 Thread Diego Biurrun
These are the intrinsics patches for MSVC, cleaned up to account for review comments. Note that I do not have MSVC, so testing is welcome. ___ libav-devel mailing list libav-devel@libav.org https://lists.libav.org/mailman/listinfo/libav-devel

[libav-devel] [PATCH 2/5] lavu: use intrinsics for emms on systems lacking inline asm support

2012-07-08 Thread Diego Biurrun
From: Ronald S. Bultje rsbul...@gmail.com Signed-off-by: Diego Biurrun di...@biurrun.de --- configure|3 +++ libavutil/internal.h |7 +-- 2 files changed, 8 insertions(+), 2 deletions(-) diff --git a/configure b/configure index 3c82eaa..7ef1025 100755 --- a/configure +++

[libav-devel] [PATCH 1/5] configure: Change the rdtsc check to a linker check

2012-07-08 Thread Diego Biurrun
A compile-only test could succeed with an implicit function declaration. Also move the check to the x86 section; rdtsc is x86-only. --- configure |4 ++-- 1 files changed, 2 insertions(+), 2 deletions(-) diff --git a/configure b/configure index 0b0d532..3c82eaa 100755 --- a/configure +++

[libav-devel] [PATCH 3/5] x86/cpu: implement support for xgetbv through intrinsics

2012-07-08 Thread Diego Biurrun
From: Ronald S. Bultje rsbul...@gmail.com Signed-off-by: Diego Biurrun di...@biurrun.de --- configure |2 ++ libavutil/x86/cpu.c | 11 +++ 2 files changed, 13 insertions(+), 0 deletions(-) diff --git a/configure b/configure index 7ef1025..488fa7e 100755 --- a/configure

[libav-devel] [PATCH 4/5] x86/cpu: implement support for cpuid through intrinsics

2012-07-08 Thread Diego Biurrun
From: Ronald S. Bultje rsbul...@gmail.com Signed-off-by: Diego Biurrun di...@biurrun.de --- configure |2 ++ libavutil/x86/cpu.c | 14 ++ 2 files changed, 16 insertions(+), 0 deletions(-) diff --git a/configure b/configure index 488fa7e..657e6d8 100755 ---

[libav-devel] [PATCH 5/5] x86/cpu: implement get/set_eflags using intrinsics

2012-07-08 Thread Diego Biurrun
From: Ronald S. Bultje rsbul...@gmail.com Signed-off-by: Diego Biurrun di...@biurrun.de --- configure |2 ++ libavutil/x86/cpu.c | 14 ++ 2 files changed, 16 insertions(+), 0 deletions(-) diff --git a/configure b/configure index 657e6d8..32270db 100755 ---

Re: [libav-devel] [PATCH] TechSmith Screen Codec 2 decoder

2012-07-08 Thread Jason Garrett-Glaser
+#define UNQUANT(val, q) ((q * val + 0x80) 8) I'd call this DEQUANT, but that's just me. +for (i = 0; i 4; i++) { +if (i 1) { +t0 = UNQUANT(q[1], in[0 * 4 + i]); +t1 = UNQUANT(q[2], in[1 * 4 + i]); +t2 = UNQUANT(q[1], in[2 * 4 + i]); +

Re: [libav-devel] [PATCH 1/2] alac: add multichannel decode support

2012-07-08 Thread Kostya Shishkov
On Sun, Jul 08, 2012 at 05:28:21PM -0400, Justin Ruggles wrote: From: Andrew D'Addesio modchip...@gmail.com Signed-off-by: Justin Ruggles justin.rugg...@gmail.com --- libavcodec/alac.c | 183 +++- 1 files changed, 109 insertions(+), 74

[libav-devel] [PATCH] mss3: split DSP functions that are used in MTS2(MSS4) into separate file

2012-07-08 Thread Kostya Shishkov
--- I also took the liberty to get rid of floats in matrix generation. --- libavcodec/Makefile |2 +- libavcodec/mss3.c | 94 +--- libavcodec/mss34dsp.c | 113 + libavcodec/mss34dsp.h | 45