[libav-devel] [PATCH] swscale: sse scaler optimizations are not bitexact

2012-10-30 Thread Luca Barbato
--- ./avconv -filter_complex color="white" -s 640x480 -c rawvideo -pix_fmt yuv444p9le shows the problem easily. libswscale/x86/swscale.c | 18 -- tests/ref/lavfi/pixfmts_scale | 8 2 files changed, 16 insertions(+), 10 deletions(-) diff --git a/libswscale/x86/swsc

Re: [libav-devel] [PATCH] rtsp: generate payload type from stream id

2012-10-30 Thread Martin Storsjö
On Tue, 30 Oct 2012, Luca Barbato wrote: On 10/30/2012 02:54 PM, Martin Storsjö wrote: On Tue, 30 Oct 2012, Luca Barbato wrote: Support multiple video/audio streams with different format in the same session. --- This time it keeps the previous behaviour on stream id equal to 0, so nothing un

Re: [libav-devel] [PATCH 4/6] cngdec: Allow flushing the decoder

2012-10-30 Thread Kostya Shishkov
On Tue, Oct 30, 2012 at 12:39:46PM +0200, Martin Storsjö wrote: > After a flush, don't average the output envelope and energy with > previous iterations. > > Also start directly from the target values for the first iteration > at startup. > --- > libavcodec/cngdec.c | 20 +--- >

Re: [libav-devel] [PATCH 3/6] cngdec: Make the dbov variable have the right unit

2012-10-30 Thread Kostya Shishkov
On Tue, Oct 30, 2012 at 12:39:45PM +0200, Martin Storsjö wrote: > Previously the unit of the variable was Bov, not dBov. > --- > libavcodec/cngdec.c |4 ++-- > 1 file changed, 2 insertions(+), 2 deletions(-) > > diff --git a/libavcodec/cngdec.c b/libavcodec/cngdec.c > index c05bfd9..fca4b9e 1

Re: [libav-devel] [PATCH 2/6] cngdec: Fix the memset size to cover the full array

2012-10-30 Thread Kostya Shishkov
On Tue, Oct 30, 2012 at 12:39:44PM +0200, Martin Storsjö wrote: > This was a leftover from previous iterations of the code, where the > refl coef arrays were statically allocated. > --- > libavcodec/cngdec.c |2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/libavcodec/cn

Re: [libav-devel] [PATCH 1/6] cngdec: Update the LPC coefficients after averaging the reflection coefficients

2012-10-30 Thread Kostya Shishkov
On Tue, Oct 30, 2012 at 12:39:43PM +0200, Martin Storsjö wrote: > I somehow messed up the placement of this one. > --- > libavcodec/cngdec.c |2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/libavcodec/cngdec.c b/libavcodec/cngdec.c > index c22fd55..8cfe9cf 100644 > ---

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

2012-10-30 Thread Diego Biurrun
On Mon, Oct 29, 2012 at 08:42:18PM -0700, Ronald S. Bultje wrote: > On Mon, Oct 29, 2012 at 7:28 PM, Diego Biurrun 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

Re: [libav-devel] [PATCH] rtsp: generate payload type from stream id

2012-10-30 Thread Luca Barbato
On 10/30/2012 02:54 PM, Martin Storsjö wrote: > On Tue, 30 Oct 2012, Luca Barbato wrote: > >> Support multiple video/audio streams with different format in the same >> session. >> --- >> >> This time it keeps the previous behaviour on stream id equal to 0, so >> nothing unexpected should happen. >

Re: [libav-devel] [PATCH] rtsp: generate payload type from stream id

2012-10-30 Thread Martin Storsjö
On Tue, 30 Oct 2012, Luca Barbato wrote: Support multiple video/audio streams with different format in the same session. --- This time it keeps the previous behaviour on stream id equal to 0, so nothing unexpected should happen. libavformat/internal.h | 4 ++-- libavformat/movenc.c |

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

2012-10-30 Thread Luca Barbato
On 10/30/2012 04:42 AM, Ronald S. Bultje wrote: > Hi, > > On Mon, Oct 29, 2012 at 7:28 PM, Diego Biurrun 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. The

[libav-devel] [PATCH] rtsp: generate payload type from stream id

2012-10-30 Thread Luca Barbato
Support multiple video/audio streams with different format in the same session. --- This time it keeps the previous behaviour on stream id equal to 0, so nothing unexpected should happen. libavformat/internal.h | 4 ++-- libavformat/movenc.c | 2 +- libavformat/rtp.c | 8 ++

Re: [libav-devel] [PATCH] rtsp: generate payload type from stream id

2012-10-30 Thread Luca Barbato
On 10/30/2012 01:34 PM, Martin Storsjö wrote: > On Tue, 30 Oct 2012, Luca Barbato wrote: > >> Support multiple video/audio streams with different format in the same >> session. > > While this is a worthwhile target in itself, this suddely requires all > callers of anything RTP/RTSP related to set

Re: [libav-devel] [PATCH] rtsp: generate payload type from stream id

2012-10-30 Thread Martin Storsjö
On Tue, 30 Oct 2012, Luca Barbato wrote: Support multiple video/audio streams with different format in the same session. While this is a worthwhile target in itself, this suddely requires all callers of anything RTP/RTSP related to set ->id to unique values on all streams, a requirement that

[libav-devel] [PATCH] rtsp: generate payload type from stream id

2012-10-30 Thread Luca Barbato
Support multiple video/audio streams with different format in the same session. --- libavformat/internal.h | 4 ++-- libavformat/movenc.c | 2 +- libavformat/rtp.c | 6 -- libavformat/rtp.h | 9 ++--- libavformat/rtpenc.c | 2 +- libavformat/rtpenc_ch

Re: [libav-devel] [PATCH 5/6] cng: Reindent some incorrectly indented lines

2012-10-30 Thread Diego Biurrun
On Tue, Oct 30, 2012 at 12:39:47PM +0200, Martin Storsjö wrote: > --- > libavcodec/cngdec.c |2 +- > libavcodec/cngenc.c |2 +- > 2 files changed, 2 insertions(+), 2 deletions(-) OK Diego ___ libav-devel mailing list libav-devel@libav.org https

Re: [libav-devel] [PATCH 6/6] cngdec: Add a comment to explain assumptions on how to use it

2012-10-30 Thread Luca Barbato
On 10/30/2012 11:39 AM, Martin Storsjö wrote: > --- > libavcodec/cngdec.c |4 > 1 file changed, 4 insertions(+) > > diff --git a/libavcodec/cngdec.c b/libavcodec/cngdec.c > index caaa9c1..da74e2e 100644 > --- a/libavcodec/cngdec.c > +++ b/libavcodec/cngdec.c > @@ -112,6 +112,10 @@ static

Re: [libav-devel] [PATCH] configure: fix print_config() with broke awks

2012-10-30 Thread Martin Storsjö
On Tue, 30 Oct 2012, Mans Rullgard wrote: Some awk versions do not treat the result of unary + on a (numeric) string as numeric, giving wrong results when used in a boolean context Using unary - instead is logically equivalent works as expected. Signed-off-by: Mans Rullgard --- Tested on Tru64

Re: [libav-devel] [PATCH] configure: fix print_config() with broke awks

2012-10-30 Thread Kostya Shishkov
On Tue, Oct 30, 2012 at 12:01:03PM +, Mans Rullgard wrote: > Some awk versions do not treat the result of unary + on a (numeric) > string as numeric, giving wrong results when used in a boolean context > Using unary - instead is logically equivalent works as expected. > > Signed-off-by: Mans R

[libav-devel] [PATCH] configure: fix print_config() with broke awks

2012-10-30 Thread Mans Rullgard
Some awk versions do not treat the result of unary + on a (numeric) string as numeric, giving wrong results when used in a boolean context Using unary - instead is logically equivalent works as expected. Signed-off-by: Mans Rullgard --- Tested on Tru64. --- configure | 2 +- 1 file changed, 1 in

Re: [libav-devel] [PATCH 6/6] cngdec: Add a comment to explain assumptions on how to use it

2012-10-30 Thread Martin Storsjö
On Tue, 30 Oct 2012, Martin Storsjö wrote: --- libavcodec/cngdec.c |4 1 file changed, 4 insertions(+) diff --git a/libavcodec/cngdec.c b/libavcodec/cngdec.c index caaa9c1..da74e2e 100644 --- a/libavcodec/cngdec.c +++ b/libavcodec/cngdec.c @@ -112,6 +112,10 @@ static int cng_decode_fram

[libav-devel] [PATCH 5/6] cng: Reindent some incorrectly indented lines

2012-10-30 Thread Martin Storsjö
--- libavcodec/cngdec.c |2 +- libavcodec/cngenc.c |2 +- 2 files changed, 2 insertions(+), 2 deletions(-) diff --git a/libavcodec/cngdec.c b/libavcodec/cngdec.c index 4fe7839..caaa9c1 100644 --- a/libavcodec/cngdec.c +++ b/libavcodec/cngdec.c @@ -102,7 +102,7 @@ static void cng_decode_fl

[libav-devel] [PATCH 6/6] cngdec: Add a comment to explain assumptions on how to use it

2012-10-30 Thread Martin Storsjö
--- libavcodec/cngdec.c |4 1 file changed, 4 insertions(+) diff --git a/libavcodec/cngdec.c b/libavcodec/cngdec.c index caaa9c1..da74e2e 100644 --- a/libavcodec/cngdec.c +++ b/libavcodec/cngdec.c @@ -112,6 +112,10 @@ static int cng_decode_frame(AVCodecContext *avctx, void *data, f

[libav-devel] [PATCH 4/6] cngdec: Allow flushing the decoder

2012-10-30 Thread Martin Storsjö
After a flush, don't average the output envelope and energy with previous iterations. Also start directly from the target values for the first iteration at startup. --- libavcodec/cngdec.c | 20 +--- 1 file changed, 17 insertions(+), 3 deletions(-) diff --git a/libavcodec/cngde

[libav-devel] [PATCH 2/6] cngdec: Fix the memset size to cover the full array

2012-10-30 Thread Martin Storsjö
This was a leftover from previous iterations of the code, where the refl coef arrays were statically allocated. --- libavcodec/cngdec.c |2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/libavcodec/cngdec.c b/libavcodec/cngdec.c index 8cfe9cf..c05bfd9 100644 --- a/libavcodec/cn

[libav-devel] [PATCH 3/6] cngdec: Make the dbov variable have the right unit

2012-10-30 Thread Martin Storsjö
Previously the unit of the variable was Bov, not dBov. --- libavcodec/cngdec.c |4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/libavcodec/cngdec.c b/libavcodec/cngdec.c index c05bfd9..fca4b9e 100644 --- a/libavcodec/cngdec.c +++ b/libavcodec/cngdec.c @@ -106,8 +106,8 @@

[libav-devel] [PATCH 1/6] cngdec: Update the LPC coefficients after averaging the reflection coefficients

2012-10-30 Thread Martin Storsjö
I somehow messed up the placement of this one. --- libavcodec/cngdec.c |2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/libavcodec/cngdec.c b/libavcodec/cngdec.c index c22fd55..8cfe9cf 100644 --- a/libavcodec/cngdec.c +++ b/libavcodec/cngdec.c @@ -112,12 +112,12 @@ static int

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

2012-10-30 Thread Martin Storsjö
On Tue, 30 Oct 2012, Luca Barbato wrote: On 10/30/2012 11:06 AM, Martin Storsjö wrote: On Tue, 30 Oct 2012, Luca Barbato wrote: On 10/30/2012 10:55 AM, Martin Storsjö wrote: On Tue, 30 Oct 2012, Luca Barbato wrote: On 10/30/2012 10:31 AM, Martin Storsjö wrote: On Tue, 30 Oct 2012, Luca Ba

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

2012-10-30 Thread Luca Barbato
On 10/30/2012 11:06 AM, Martin Storsjö wrote: > On Tue, 30 Oct 2012, Luca Barbato wrote: > >> On 10/30/2012 10:55 AM, Martin Storsjö wrote: >>> On Tue, 30 Oct 2012, Luca Barbato wrote: >>> On 10/30/2012 10:31 AM, Martin Storsjö wrote: > On Tue, 30 Oct 2012, Luca Barbato wrote: > >

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

2012-10-30 Thread Martin Storsjö
On Tue, 30 Oct 2012, Luca Barbato wrote: On 10/30/2012 10:55 AM, Martin Storsjö wrote: On Tue, 30 Oct 2012, Luca Barbato wrote: On 10/30/2012 10:31 AM, Martin Storsjö wrote: On Tue, 30 Oct 2012, Luca Barbato wrote: Makes possible to support multiple video/audio streams with different fmtp.

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

2012-10-30 Thread Kostya Shishkov
On Tue, Oct 30, 2012 at 09:59:47AM +0100, Luca Barbato wrote: > On 10/30/2012 04:40 AM, Ronald S. Bultje wrote: > > Hi, > > > > On Mon, Oct 29, 2012 at 7:14 PM, Luca Barbato wrote: > >> With the input of Kostya and Ronald. > >> --- > >> libswscale/swscale.c | 38

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

2012-10-30 Thread Luca Barbato
On 10/30/2012 10:55 AM, Martin Storsjö wrote: > On Tue, 30 Oct 2012, Luca Barbato wrote: > >> On 10/30/2012 10:31 AM, Martin Storsjö wrote: >>> On Tue, 30 Oct 2012, Luca Barbato wrote: >>> Makes possible to support multiple video/audio streams with different fmtp. --- >> >>> This is

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

2012-10-30 Thread Martin Storsjö
On Tue, 30 Oct 2012, Luca Barbato wrote: On 10/30/2012 10:31 AM, Martin Storsjö wrote: On Tue, 30 Oct 2012, Luca Barbato wrote: Makes possible to support multiple video/audio streams with different fmtp. --- This is problematic, since you want unique RTP payload types for audio and video,

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

2012-10-30 Thread Luca Barbato
On 10/30/2012 10:31 AM, Martin Storsjö wrote: > On Tue, 30 Oct 2012, Luca Barbato wrote: > >> Makes possible to support multiple video/audio streams with different >> fmtp. >> --- > This is problematic, since you want unique RTP payload types for audio > and video, and these might be in two diffe

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

2012-10-30 Thread Martin Storsjö
On Tue, 30 Oct 2012, Luca Barbato wrote: 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

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

2012-10-30 Thread Luca Barbato
On 10/30/2012 04:42 AM, Ronald S. Bultje wrote: > Hi, > > On Mon, Oct 29, 2012 at 7:28 PM, Diego Biurrun 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 chang

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

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

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

2012-10-30 Thread Luca Barbato
On 10/30/2012 04:40 AM, Ronald S. Bultje wrote: > Hi, > > On Mon, Oct 29, 2012 at 7:14 PM, Luca Barbato wrote: >> With the input of Kostya and Ronald. >> --- >> libswscale/swscale.c | 38 -- >> libswscale/swscale_unscaled.c | 32 ++

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

2012-10-30 Thread Måns Rullgård
Loren Merritt writes: > On Mon, 29 Oct 2012, Ronald S. Bultje wrote: >> On Mon, Oct 29, 2012 at 5:15 PM, Diego Biurrun wrote: >> >>> This allows using "mmxext" as name in Libav while staying compatible >>> with changes to the YASM macro infrastructure imported from x264. >>> --- >>> libavutil/x

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

2012-10-30 Thread Måns Rullgård
Diego Biurrun writes: > --- > 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