[libav-devel] [PATCH 1/8] vf_overlay: prevent premature freeing of cur_buf

2012-07-25 Thread Anton Khirnov
From: Michael Niedermayer michae...@gmx.at Signed-off-by: Anton Khirnov an...@khirnov.net --- libavfilter/vf_overlay.c |1 + 1 file changed, 1 insertion(+) diff --git a/libavfilter/vf_overlay.c b/libavfilter/vf_overlay.c index 14304c1..5f34895 100644 --- a/libavfilter/vf_overlay.c +++

[libav-devel] [PATCH 2/8] vf_overlay: ensure the overlay frame does not get leaked.

2012-07-25 Thread Anton Khirnov
--- libavfilter/vf_overlay.c |1 + 1 file changed, 1 insertion(+) diff --git a/libavfilter/vf_overlay.c b/libavfilter/vf_overlay.c index 5f34895..5a8da27 100644 --- a/libavfilter/vf_overlay.c +++ b/libavfilter/vf_overlay.c @@ -239,6 +239,7 @@ static int start_frame_overlay(AVFilterLink

[libav-devel] [PATCH 3/8] vf_yadif: unset cur_buf on the input link.

2012-07-25 Thread Anton Khirnov
The buffer is stored internally, so this prevents it from being unreffed automatically. --- libavfilter/vf_yadif.c |1 + 1 file changed, 1 insertion(+) diff --git a/libavfilter/vf_yadif.c b/libavfilter/vf_yadif.c index c6d78a5..a2b7337 100644 --- a/libavfilter/vf_yadif.c +++

[libav-devel] [PATCH 4/8] audiogen: allow specifying random seed as a commandline parameter.

2012-07-25 Thread Anton Khirnov
--- tests/audiogen.c |7 +-- 1 file changed, 5 insertions(+), 2 deletions(-) diff --git a/tests/audiogen.c b/tests/audiogen.c index 8d27dc2..acb380d 100644 --- a/tests/audiogen.c +++ b/tests/audiogen.c @@ -144,8 +144,8 @@ int main(int argc, char **argv) int nb_channels = 2;

[libav-devel] [PATCH 5/8] FATE: add a test for amix audio filter.

2012-07-25 Thread Anton Khirnov
--- tests/fate/filter.mak | 22 ++ 1 file changed, 22 insertions(+) diff --git a/tests/fate/filter.mak b/tests/fate/filter.mak index 41916c9..2ba51ed 100644 --- a/tests/fate/filter.mak +++ b/tests/fate/filter.mak @@ -1,3 +1,25 @@ +FATE_AMIX += fate-filter-amix-simple

[libav-devel] [PATCH 6/8] FATE: add a test for delogo video filter.

2012-07-25 Thread Anton Khirnov
--- tests/fate/filter.mak|5 ++ tests/ref/fate/filter-delogo | 110 ++ 2 files changed, 115 insertions(+) create mode 100644 tests/ref/fate/filter-delogo diff --git a/tests/fate/filter.mak b/tests/fate/filter.mak index 2ba51ed..f1185ed 100644

[libav-devel] [PATCH 8/8] lavfi: bump minor to mark stabilizing the ABI.

2012-07-25 Thread Anton Khirnov
--- libavfilter/version.h |2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/libavfilter/version.h b/libavfilter/version.h index c1292bb..0e72a47 100644 --- a/libavfilter/version.h +++ b/libavfilter/version.h @@ -29,7 +29,7 @@ #include libavutil/avutil.h #define

[libav-devel] [PATCH 7/8] FATE: add tests for yadif.

2012-07-25 Thread Anton Khirnov
--- tests/fate/filter.mak |9 ++ tests/ref/fate/filter-yadif-mode0 | 32 +++ tests/ref/fate/filter-yadif-mode1 | 63 + 3 files changed, 104 insertions(+) create mode 100644 tests/ref/fate/filter-yadif-mode0 create mode

Re: [libav-devel] [PATCH 1/7] vf_hqdn3d: cosmetics

2012-07-25 Thread Anton Khirnov
On Wed, 25 Jul 2012 02:45:34 +, Loren Merritt lor...@u.washington.edu wrote: Change code style to match the rest of libav. --- libavfilter/vf_hqdn3d.c | 309 +++ 1 files changed, 153 insertions(+), 156 deletions(-) diff --git

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

2012-07-25 Thread Anton Khirnov
On Wed, 25 Jul 2012 02:45:35 +, Loren Merritt lor...@u.washington.edu wrote: 14% faster on penryn, 2% on sandybridge, 9% on bulldozer --- libavfilter/vf_hqdn3d.c | 166 +++--- 1 files changed, 54 insertions(+), 112 deletions(-) diff --git

Re: [libav-devel] [PATCH 6/7] vf_hqdn3d: allow outpic==inpic to reduce cache pressure

2012-07-25 Thread Anton Khirnov
On Wed, 25 Jul 2012 02:45:39 +, Loren Merritt lor...@u.washington.edu wrote: --- libavfilter/vf_hqdn3d.c | 21 - 1 files changed, 20 insertions(+), 1 deletions(-) diff --git a/libavfilter/vf_hqdn3d.c b/libavfilter/vf_hqdn3d.c index cbc4ab9..5a90df3 100644 ---

Re: [libav-devel] [PATCH 6/7] vf_hqdn3d: allow outpic==inpic to reduce cache pressure

2012-07-25 Thread Anton Khirnov
On Wed, 25 Jul 2012 02:53:09 + (UTC), Loren Merritt lor...@u.washington.edu wrote: On Wed, 25 Jul 2012, Loren Merritt wrote: --- a/libavfilter/vf_hqdn3d.c +++ b/libavfilter/vf_hqdn3d.c @@ -294,6 +294,23 @@ static int config_input(AVFilterLink *inlink) return 0; }

Re: [libav-devel] [PATCH 3/4] avformat: Drop pointless format from container long names

2012-07-25 Thread Diego Biurrun
On Tue, Jul 24, 2012 at 03:30:47PM -0700, Diego Elio Pettenò wrote: Il 24/07/2012 15:08, Diego Biurrun ha scritto: +.long_name = NULL_IF_CONFIG_SMALL(Apple Core Audio), Shouldn't this be Apple CAF (Core Audio Format) ? Yes, good catch. Amended locally and squashed this change into

Re: [libav-devel] [PATCH] Replace x*155/100 by x*10158116.

2012-07-25 Thread Diego Biurrun
On Tue, Jul 24, 2012 at 03:23:12PM -0700, Ronald S. Bultje wrote: --- a/libavcodec/vp8.c +++ b/libavcodec/vp8.c @@ -249,12 +249,12 @@ static void get_quants(VP8Context *s) -s-qmat[i].luma_qmul[0]= vp8_dc_qlookup[av_clip_uintp2(base_qi + ydc_delta , 7)]; -

Re: [libav-devel] [PATCH 7/8] FATE: add tests for yadif.

2012-07-25 Thread Diego Biurrun
On Wed, Jul 25, 2012 at 08:42:31AM +0200, Anton Khirnov wrote: --- tests/fate/filter.mak |9 ++ tests/ref/fate/filter-yadif-mode0 | 32 +++ tests/ref/fate/filter-yadif-mode1 | 63 + 3 files changed, 104

Re: [libav-devel] [PATCH 6/8] FATE: add a test for delogo video filter.

2012-07-25 Thread Diego Biurrun
On Wed, Jul 25, 2012 at 08:42:30AM +0200, Anton Khirnov wrote: --- tests/fate/filter.mak|5 ++ tests/ref/fate/filter-delogo | 110 ++ 2 files changed, 115 insertions(+) create mode 100644 tests/ref/fate/filter-delogo Should be OK.

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

2012-07-25 Thread Diego Biurrun
On Wed, Jul 25, 2012 at 08:42:29AM +0200, Anton Khirnov wrote: --- tests/fate/filter.mak | 22 ++ 1 file changed, 22 insertions(+) Probably OK. Diego ___ libav-devel mailing list libav-devel@libav.org

Re: [libav-devel] [PATCH] H264: Remove 3dnow qpel code.

2012-07-25 Thread Diego Biurrun
On Mon, Jul 23, 2012 at 05:37:33PM -0700, Daniel Kang wrote: On Mon, Jul 23, 2012 at 5:21 PM, Diego Biurrun di...@biurrun.de wrote: On Mon, Jul 23, 2012 at 05:12:23PM -0700, Daniel Kang wrote: From: Daniel Kang daniel.d.k...@gmail.com The only CPUs that have 3dnow and don't have mmxext

[libav-devel] [PATCH 0/1] Libav urldecode support

2012-07-25 Thread Antti Seppälä
Hello. Here is a patch which allows libav to support urlencoded credentials with http(s) connections. E.g. avplay http://email%40address.foo:password@video-url/video-file.ts does not succeed without this patch. This patch comes especially handy if the username happens to contain '@' character.

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

2012-07-25 Thread Antti Seppälä
It should be possible to specify usernames in http requests containing urlencoded characters. This patch adds support for decoding the auth strings. Signed-off-by: Antti Seppälä a.sepp...@gmail.com --- libavformat/Makefile| 11 +++--- libavformat/httpauth.c | 23 +---

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

2012-07-25 Thread Måns Rullgård
Ronald S. Bultje rsbul...@gmail.com writes: Hi, On Tue, Jul 24, 2012 at 7:45 PM, Loren Merritt lor...@u.washington.edu wrote: -long x, y; -uint32_t pixel; +uint32_t tmp; -for (y = 0; y h; y++) { -for (x = 0; x w; x++) { -pixel =

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

2012-07-25 Thread Diego Biurrun
On Wed, Jul 25, 2012 at 12:43:39PM +0300, Antti Seppälä wrote: It should be possible to specify usernames in http requests containing urlencoded characters. This patch adds support for decoding the auth strings. --- /dev/null +++ b/libavformat/urldecode.c @@ -0,0 +1,86 @@ +/* + * Simple

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

2012-07-25 Thread Martin Storsjö
On Wed, 25 Jul 2012, Antti Seppälä wrote: It should be possible to specify usernames in http requests containing urlencoded characters. This patch adds support for decoding the auth strings. Signed-off-by: Antti Seppälä a.sepp...@gmail.com --- libavformat/Makefile| 11 +++---

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

2012-07-25 Thread Uoti Urpala
On Wed, 2012-07-25 at 10:53 +0100, Måns Rullgård wrote: Ronald S. Bultje rsbul...@gmail.com writes: On Tue, Jul 24, 2012 at 7:45 PM, Loren Merritt lor...@u.washington.edu wrote: -long x, y; -uint32_t pixel; +uint32_t tmp; -for (y = 0; y h; y++) { -for (x

Re: [libav-devel] [PATCH 3/4] avformat: Drop pointless format from container long names

2012-07-25 Thread Anton Khirnov
On Wed, 25 Jul 2012 00:08:56 +0200, Diego Biurrun di...@biurrun.de wrote: --- a/libavformat/crcenc.c +++ b/libavformat/crcenc.c @@ -56,7 +56,7 @@ static int crc_write_trailer(struct AVFormatContext *s) AVOutputFormat ff_crc_muxer = { .name = crc, -.long_name

Re: [libav-devel] [PATCH 3/4] avformat: Drop pointless format from container long names

2012-07-25 Thread Diego Biurrun
On Wed, Jul 25, 2012 at 01:30:42PM +0200, Anton Khirnov wrote: On Wed, 25 Jul 2012 00:08:56 +0200, Diego Biurrun di...@biurrun.de wrote: --- a/libavformat/crcenc.c +++ b/libavformat/crcenc.c @@ -56,7 +56,7 @@ static int crc_write_trailer(struct AVFormatContext *s) AVOutputFormat

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

2012-07-25 Thread Måns Rullgård
Uoti Urpala uoti.urp...@pp1.inet.fi writes: On Wed, 2012-07-25 at 10:53 +0100, Måns Rullgård wrote: Ronald S. Bultje rsbul...@gmail.com writes: On Tue, Jul 24, 2012 at 7:45 PM, Loren Merritt lor...@u.washington.edu wrote: -long x, y; -uint32_t pixel; +uint32_t tmp; -

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

2012-07-25 Thread Diego Biurrun
On Wed, Jul 25, 2012 at 12:46:02PM +0100, Måns Rullgård wrote: Uoti Urpala uoti.urp...@pp1.inet.fi writes: [... flame ...] [... flame ...] Compiler trollwar incoming? Come on guys, save us all the spectacle... Diego ___ libav-devel mailing list

Re: [libav-devel] [PATCH 3/4] avformat: Drop pointless format from container long names

2012-07-25 Thread Anton Khirnov
On Wed, 25 Jul 2012 13:39:56 +0200, Diego Biurrun di...@biurrun.de wrote: Not quite - notice the capitalization. We might also use mkv as short name, it would make sense... Now that I look at it, we have a matroska muxer and matroska_audio muxer, both sharing the name matroska. Is that

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

2012-07-25 Thread Uoti Urpala
On Wed, 2012-07-25 at 00:08 +0200, Diego Biurrun wrote: --- a/libavcodec/bmpenc.c -.long_name = NULL_IF_CONFIG_SMALL(BMP image), +.long_name = NULL_IF_CONFIG_SMALL(Bitmap image), Bitmap image sounds generic. BMP more clearly identifies a particular format IMO. This only

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

2012-07-25 Thread Kostya Shishkov
On Wed, Jul 25, 2012 at 02:59:57PM +0300, Uoti Urpala wrote: On Wed, 2012-07-25 at 00:08 +0200, Diego Biurrun wrote: --- a/libavcodec/bmpenc.c -.long_name = NULL_IF_CONFIG_SMALL(BMP image), +.long_name = NULL_IF_CONFIG_SMALL(Bitmap image), Bitmap image sounds generic.

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

2012-07-25 Thread Uoti Urpala
On Wed, 2012-07-25 at 12:46 +0100, Måns Rullgård wrote: Uoti Urpala uoti.urp...@pp1.inet.fi writes: On Wed, 2012-07-25 at 10:53 +0100, Måns Rullgård wrote: Same goes for a number of other compilers. It's unfortunate, but a small price to pay for portability. What compilers are those

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

2012-07-25 Thread Mans Rullgard
This adds a full identification probe of CC, AS, LD and HOSTCC, and sets up correct flags and dependency tracking for each. Signed-off-by: Mans Rullgard m...@mansr.com --- Now adds $_flags to the host cflags/ldflags. This variable contains flags common to all invocations of the tool. ---

Re: [libav-devel] [PATCH 1/8] vf_overlay: prevent premature freeing of cur_buf

2012-07-25 Thread Luca Barbato
On 07/25/2012 08:42 AM, Anton Khirnov wrote: From: Michael Niedermayer michae...@gmx.at Signed-off-by: Anton Khirnov an...@khirnov.net --- libavfilter/vf_overlay.c |1 + 1 file changed, 1 insertion(+) Ok. -- Luca Barbato Gentoo/linux http://dev.gentoo.org/~lu_zero

Re: [libav-devel] [PATCH 2/8] vf_overlay: ensure the overlay frame does not get leaked.

2012-07-25 Thread Luca Barbato
On 07/25/2012 08:42 AM, Anton Khirnov wrote: --- libavfilter/vf_overlay.c |1 + 1 file changed, 1 insertion(+) Ok. -- Luca Barbato Gentoo/linux http://dev.gentoo.org/~lu_zero ___ libav-devel mailing list libav-devel@libav.org

Re: [libav-devel] [PATCH 3/8] vf_yadif: unset cur_buf on the input link.

2012-07-25 Thread Luca Barbato
On 07/25/2012 08:42 AM, Anton Khirnov wrote: The buffer is stored internally, so this prevents it from being unreffed automatically. --- libavfilter/vf_yadif.c |1 + 1 file changed, 1 insertion(+) diff --git a/libavfilter/vf_yadif.c b/libavfilter/vf_yadif.c index c6d78a5..a2b7337

Re: [libav-devel] [PATCH 4/8] audiogen: allow specifying random seed as a commandline parameter.

2012-07-25 Thread Luca Barbato
On 07/25/2012 08:42 AM, Anton Khirnov wrote: --- tests/audiogen.c |7 +-- 1 file changed, 5 insertions(+), 2 deletions(-) Ok. -- Luca Barbato Gentoo/linux http://dev.gentoo.org/~lu_zero ___ libav-devel mailing list

Re: [libav-devel] [PATCH 8/8] lavfi: bump minor to mark stabilizing the ABI.

2012-07-25 Thread Luca Barbato
On 07/25/2012 08:42 AM, Anton Khirnov wrote: --- libavfilter/version.h |2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/libavfilter/version.h b/libavfilter/version.h index c1292bb..0e72a47 100644 --- a/libavfilter/version.h +++ b/libavfilter/version.h @@ -29,7 +29,7

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

2012-07-25 Thread Luca Barbato
On 07/25/2012 12:33 PM, Uoti Urpala wrote: This kind of syntax issues could be automatically handled by a conversion wrapper for the deficient compilers. It is commonly accepted not to mix variable declaration and code and usually it helps readability and avoids shadowing variables in long/too

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

2012-07-25 Thread Luca Barbato
On 07/25/2012 02:12 PM, Kostya Shishkov wrote: On Wed, Jul 25, 2012 at 02:59:57PM +0300, Uoti Urpala wrote: Bitmap image sounds generic. BMP more clearly identifies a particular format IMO. It's not only generic but also a tautology. Windows Bitmap is more recognizable but doesn't cover

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

2012-07-25 Thread Luca Barbato
On 07/25/2012 11:59 AM, Diego Biurrun wrote: On Wed, Jul 25, 2012 at 12:43:39PM +0300, Antti Seppälä wrote: It should be possible to specify usernames in http requests containing urlencoded characters. This patch adds support for decoding the auth strings. --- /dev/null +++

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

2012-07-25 Thread Luca Barbato
On 07/25/2012 07:40 AM, Jason Garrett-Glaser wrote: Do the x264 functions sign-extend all their integer arguments? Or put differently, does the problem occur for 32-bit builds also, or only for 64-bit builds? Yes they do, and such a problem wouldn't target solely _avx functions; that

Re: [libav-devel] [PATCH 2/2] vp3: don't use calls to inline asm in yasm code.

2012-07-25 Thread Luca Barbato
On 07/25/2012 07:25 AM, Ronald S. Bultje wrote: Hi, On Mon, Jul 23, 2012 at 7:12 AM, Ronald S. Bultje rsbul...@gmail.com wrote: On Sun, Jul 22, 2012 at 2:38 PM, Ronald S. Bultje rsbul...@gmail.com wrote: From: Ronald S. Bultje rsbul...@gmail.com Mixing yasm and inline asm is a bad idea,

[libav-devel] [PATCH] msvc: workaround __VA_ARGS__ missing tokenization

2012-07-25 Thread Luca Barbato
A second expansion forces the preprocessor to tokenize properly. --- Not sure if we have other broken preprocessors to mention there. libavcodec/dsputil.h | 10 +++--- 1 files changed, 7 insertions(+), 3 deletions(-) diff --git a/libavcodec/dsputil.h b/libavcodec/dsputil.h index

Re: [libav-devel] [PATCH] msvc: workaround __VA_ARGS__ missing tokenization

2012-07-25 Thread Måns Rullgård
Luca Barbato lu_z...@gentoo.org writes: A second expansion forces the preprocessor to tokenize properly. --- Not sure if we have other broken preprocessors to mention there. I've never seen or heard of anything else being that completely broken. libavcodec/dsputil.h | 10 +++--- 1

Re: [libav-devel] [PATCH 2/2] lavr: add x86-optimized mixing functions

2012-07-25 Thread Loren Merritt
On Tue, 24 Jul 2012, Justin Ruggles wrote: Adds optimized functions for mixing 3 through 8 input channels to 1 and 2 output channels in fltp or s16p format with flt coeffs. --- libavresample/utils.c |7 +- libavresample/x86/audio_mix.asm| 293

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

2012-07-25 Thread Luca Barbato
On 07/25/2012 03:10 PM, 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. Signed-off-by: Mans Rullgard m...@mansr.com --- Now adds $_flags to the host cflags/ldflags. This variable contains

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

2012-07-25 Thread Loren Merritt
On Wed, 25 Jul 2012, Luca Barbato wrote: On 07/25/2012 07:40 AM, Jason Garrett-Glaser wrote: Do the x264 functions sign-extend all their integer arguments? Or put differently, does the problem occur for 32-bit builds also, or only for 64-bit builds? Yes they do, and such a problem wouldn't

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

2012-07-25 Thread John Stebbins
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 wrote: Jason Garrett-Glaser ja...@x264.com

Re: [libav-devel] [PATCH 01/10] rtmp: Factorize the code by adding handle_server_bw

2012-07-25 Thread Martin Storsjö
On Sat, 21 Jul 2012, Luca Barbato wrote: On 07/21/2012 12:59 PM, Samuel Pitoiset wrote: +return AVERROR(EINVAL); AVERROR_INVALIDDATA while you are at it =) I'd rather keep it as is in this one, to keep it as a clean refactoring. I'm ok with this one and will push it soon, the

Re: [libav-devel] [PATCH 05/10] rtmp: Return a proper error value when the chunk size is incorrect

2012-07-25 Thread Martin Storsjö
On Sat, 21 Jul 2012, Samuel Pitoiset wrote: --- libavformat/rtmpproto.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/libavformat/rtmpproto.c b/libavformat/rtmpproto.c index 4747899..f3353b8 100644 --- a/libavformat/rtmpproto.c +++ b/libavformat/rtmpproto.c @@ -834,7 +834,7

Re: [libav-devel] [PATCH 05/10] rtmp: Return a proper error value when the chunk size is incorrect

2012-07-25 Thread Martin Storsjö
On Wed, 25 Jul 2012, Martin Storsjö wrote: On Sat, 21 Jul 2012, Samuel Pitoiset wrote: --- libavformat/rtmpproto.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/libavformat/rtmpproto.c b/libavformat/rtmpproto.c index 4747899..f3353b8 100644 --- a/libavformat/rtmpproto.c

Re: [libav-devel] [PATCH 09/10] rtmp: Return a proper error value when a client bandwith packet is not 5 bytes long

2012-07-25 Thread Martin Storsjö
On Sat, 21 Jul 2012, Samuel Pitoiset wrote: --- libavformat/rtmpproto.c | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/libavformat/rtmpproto.c b/libavformat/rtmpproto.c index c003b37..5af03c4 100644 --- a/libavformat/rtmpproto.c +++ b/libavformat/rtmpproto.c @@ -859,11

Re: [libav-devel] [PATCH 10/10] rtmp: Factorize the code by adding handle_invoke

2012-07-25 Thread Martin Storsjö
On Sat, 21 Jul 2012, Samuel Pitoiset wrote: This function is used for parsing invoke packets. --- libavformat/rtmpproto.c | 143 ++-- 1 file changed, 77 insertions(+), 66 deletions(-) Ok - applied this one and the other refactorings, awaiting a

Re: [libav-devel] [PATCH] dsputil_mmx: fix incorrect assembly code

2012-07-25 Thread Ronald S. Bultje
Hi, On Mon, Jul 23, 2012 at 5:30 PM, Derek Buitenhuis derek.buitenh...@gmail.com wrote: From: Yang Wang yang.y.w...@intel.com In ff_put_pixels_clamped_mmx(), there are two assembly code blocks. In the first block (in the unrolled loop), the instructions movq 8%3, %%mm1 \n\t, and so forth,

Re: [libav-devel] [PATCH 1/2] x86: add support for fmaddps fma4 instruction with abstraction to avx/sse

2012-07-25 Thread Ronald S. Bultje
Hi, On Tue, Jul 24, 2012 at 2:03 PM, Justin Ruggles justin.rugg...@gmail.com wrote: --- configure|5 + libavutil/x86/x86inc.asm | 16 +++- 2 files changed, 16 insertions(+), 5 deletions(-) OK. Ronald ___

Re: [libav-devel] [PATCH 01/10] rtmp: Factorize the code by adding handle_server_bw

2012-07-25 Thread Luca Barbato
On 07/25/2012 07:47 PM, Martin Storsjö wrote: On Sat, 21 Jul 2012, Luca Barbato wrote: On 07/21/2012 12:59 PM, Samuel Pitoiset wrote: +return AVERROR(EINVAL); AVERROR_INVALIDDATA while you are at it =) I'd rather keep it as is in this one, to keep it as a clean refactoring.

Re: [libav-devel] [PATCH 04/15] lavr: x86: optimized 6-channel s16p to flt conversion

2012-07-25 Thread Justin Ruggles
On 07/24/2012 11:58 PM, Ronald S. Bultje wrote: Hi, On Sat, Jul 21, 2012 at 12:12 PM, Justin Ruggles justin.rugg...@gmail.com wrote: +%if cpuflag(ssse3) +pshufb m3, m0, unpack_odd ; m3 = 12, 13, 14, 15 +pshufb m0, unpack_even ; m0 = 0, 1, 2,

[libav-devel] [PATCH 1/4] rtmp: Return proper error codes in handle_chunk_size

2012-07-25 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 31d1f69..5aa6a54 100644 --- a/libavformat/rtmpproto.c +++ b/libavformat/rtmpproto.c @@ -889,7 +889,7 @@ static int

[libav-devel] [PATCH] lavr: x86: add missing vzeroupper in ff_mix_1_to_2_fltp_flt()

2012-07-25 Thread Justin Ruggles
--- libavresample/x86/audio_mix.asm |4 1 files changed, 4 insertions(+), 0 deletions(-) diff --git a/libavresample/x86/audio_mix.asm b/libavresample/x86/audio_mix.asm index 4b0434d..6193031 100644 --- a/libavresample/x86/audio_mix.asm +++ b/libavresample/x86/audio_mix.asm @@ -175,6

[libav-devel] [PATCH 3/4] rtmp: Return proper error code in handle_server_bw

2012-07-25 Thread Samuel Pitoiset
--- libavformat/rtmpproto.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/libavformat/rtmpproto.c b/libavformat/rtmpproto.c index be61074..5ac60d2 100644 --- a/libavformat/rtmpproto.c +++ b/libavformat/rtmpproto.c @@ -946,7 +946,7 @@ static int handle_server_bw(URLContext

[libav-devel] [PATCH 4/4] rtmp: Return an error when the client bandwidth is incorrect

2012-07-25 Thread Samuel Pitoiset
--- libavformat/rtmpproto.c | 12 ++-- 1 file changed, 10 insertions(+), 2 deletions(-) diff --git a/libavformat/rtmpproto.c b/libavformat/rtmpproto.c index 5ac60d2..a2efe38 100644 --- a/libavformat/rtmpproto.c +++ b/libavformat/rtmpproto.c @@ -932,8 +932,16 @@ static int

Re: [libav-devel] [PATCH 09/10] rtmp: Return a proper error value when a client bandwith packet is not 5 bytes long

2012-07-25 Thread Samuel Pitoiset
On Wed, Jul 25, 2012 at 8:05 PM, Martin Storsjö mar...@martin.st wrote: On Sat, 21 Jul 2012, Samuel Pitoiset wrote: --- libavformat/rtmpproto.c | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/libavformat/rtmpproto.c b/libavformat/rtmpproto.c index c003b37..5af03c4

Re: [libav-devel] [PATCH 4/4] rtmp: Return an error when the client bandwidth is incorrect

2012-07-25 Thread Luca Barbato
On 07/25/2012 08:51 PM, Samuel Pitoiset wrote: +rt-client_report_size = 1; Why? -- 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] lavr: x86: add missing vzeroupper in ff_mix_1_to_2_fltp_flt()

2012-07-25 Thread Luca Barbato
On 07/25/2012 08:51 PM, Justin Ruggles wrote: --- libavresample/x86/audio_mix.asm |4 1 files changed, 4 insertions(+), 0 deletions(-) I guess it is fine. -- Luca Barbato Gentoo/linux http://dev.gentoo.org/~lu_zero ___ libav-devel

Re: [libav-devel] [PATCH 14/15] lavr: x86: optimized 2-channel flt to fltp conversion

2012-07-25 Thread Justin Ruggles
On 07/25/2012 01:03 AM, Ronald S. Bultje wrote: Hi, On Sat, Jul 14, 2012 at 9:29 PM, Justin Ruggles justin.rugg...@gmail.com wrote: --- libavresample/x86/audio_convert.asm| 34 libavresample/x86/audio_convert_init.c |9 2 files

Re: [libav-devel] [PATCH 14/15] lavr: x86: optimized 2-channel flt to fltp conversion

2012-07-25 Thread Justin Ruggles
On 07/25/2012 03:19 PM, Justin Ruggles wrote: On 07/25/2012 01:03 AM, Ronald S. Bultje wrote: Hi, On Sat, Jul 14, 2012 at 9:29 PM, Justin Ruggles justin.rugg...@gmail.com wrote: --- libavresample/x86/audio_convert.asm| 34

Re: [libav-devel] [PATCH 4/4] rtmp: Return an error when the client bandwidth is incorrect

2012-07-25 Thread Samuel Pitoiset
On Wed, Jul 25, 2012 at 9:10 PM, Luca Barbato lu_z...@gentoo.org wrote: On 07/25/2012 08:51 PM, Samuel Pitoiset wrote: +rt-client_report_size = 1; Why? That instruction already existed before these changes, I just factorized it. =) Otherwise, it's used to send the client bytes report

Re: [libav-devel] [PATCH 14/15] lavr: x86: optimized 2-channel flt to fltp conversion

2012-07-25 Thread Loren Merritt
On Wed, 25 Jul 2012, Justin Ruggles wrote: On 07/25/2012 01:03 AM, Ronald S. Bultje wrote: (Can this be implemented in YMM with the current instructions available?) Not really sure if it would help. Maybe something like this? vpermilps m0, [srcq+2*lenq], q0213 vextractf128

Re: [libav-devel] round of stray commits

2012-07-25 Thread Sean McGovern
On Monday, July 23, 2012, Diego Biurrun di...@biurrun.de wrote: Ahem, it seems to be the brown paperbag season again... So, I slipped up and accidentally pushed some ongoing work I had on my master branch to origin/master, namely the following commits:

Re: [libav-devel] [PATCH 2/2] rtmp: add rtmp_listen support

2012-07-25 Thread Martin Storsjö
On Wed, 4 Jul 2012, Jordi Ortiz wrote: This code allows receiving a stream from a client with the PUBLISH command. --- libavformat/rtmpproto.c | 470 --- 1 file changed, 449 insertions(+), 21 deletions(-) FWIW, this didn't work for me (returning

Re: [libav-devel] [PATCH 4/4] rtmp: Return an error when the client bandwidth is incorrect

2012-07-25 Thread Martin Storsjö
On Wed, 25 Jul 2012, Samuel Pitoiset wrote: --- libavformat/rtmpproto.c | 12 ++-- 1 file changed, 10 insertions(+), 2 deletions(-) All 4 look good to me, pushed. // Martin ___ libav-devel mailing list libav-devel@libav.org

Re: [libav-devel] [PATCH 09/10] rtmp: Return a proper error value when a client bandwith packet is not 5 bytes long

2012-07-25 Thread Martin Storsjö
On Wed, 25 Jul 2012, Samuel Pitoiset wrote: On Wed, Jul 25, 2012 at 8:05 PM, Martin Storsjö mar...@martin.st wrote: On Sat, 21 Jul 2012, Samuel Pitoiset wrote: --- libavformat/rtmpproto.c | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/libavformat/rtmpproto.c

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

2012-07-25 Thread Diego Biurrun
On Wed, Jul 25, 2012 at 04:04:03PM +0200, Luca Barbato wrote: On 07/25/2012 11:59 AM, Diego Biurrun wrote: On Wed, Jul 25, 2012 at 12:43:39PM +0300, Antti Seppälä wrote: It should be possible to specify usernames in http requests containing urlencoded characters. This patch adds support for

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

2012-07-25 Thread Diego Biurrun
On Tue, Jul 24, 2012 at 03:29:17PM -0700, Diego Elio Pettenò wrote: Honestly I'm a bit confused, you have Flash Video (FLV) and SWF (ShockWave Flash) maybe choose between the two? Umm, SWF and FLV files are not the same... Diego ___

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

2012-07-25 Thread Diego Biurrun
On Wed, Jul 25, 2012 at 03:59:34PM +0200, Luca Barbato wrote: On 07/25/2012 02:12 PM, Kostya Shishkov wrote: On Wed, Jul 25, 2012 at 02:59:57PM +0300, Uoti Urpala wrote: Bitmap image sounds generic. BMP more clearly identifies a particular format IMO. It's not only generic but also a

Re: [libav-devel] [PATCH 09/10] rtmp: Return a proper error value when a client bandwith packet is not 5 bytes long

2012-07-25 Thread Samuel Pitoiset
On Wed, Jul 25, 2012 at 11:00 PM, Martin Storsjö mar...@martin.st wrote: On Wed, 25 Jul 2012, Samuel Pitoiset wrote: On Wed, Jul 25, 2012 at 8:05 PM, Martin Storsjö mar...@martin.st wrote: On Sat, 21 Jul 2012, Samuel Pitoiset wrote: --- libavformat/rtmpproto.c | 6 +++--- 1 file changed, 3

Re: [libav-devel] [PATCH 09/10] rtmp: Return a proper error value when a client bandwith packet is not 5 bytes long

2012-07-25 Thread Martin Storsjö
On Wed, 25 Jul 2012, Samuel Pitoiset wrote: On Wed, Jul 25, 2012 at 11:00 PM, Martin Storsjö mar...@martin.st wrote: On Wed, 25 Jul 2012, Samuel Pitoiset wrote: On Wed, Jul 25, 2012 at 8:05 PM, Martin Storsjö mar...@martin.st wrote: On Sat, 21 Jul 2012, Samuel Pitoiset wrote: ---

Re: [libav-devel] [PATCH 09/10] rtmp: Return a proper error value when a client bandwith packet is not 5 bytes long

2012-07-25 Thread Samuel Pitoiset
On Wed, Jul 25, 2012 at 11:15 PM, Martin Storsjö mar...@martin.st wrote: On Wed, 25 Jul 2012, Samuel Pitoiset wrote: On Wed, Jul 25, 2012 at 11:00 PM, Martin Storsjö mar...@martin.st wrote: On Wed, 25 Jul 2012, Samuel Pitoiset wrote: On Wed, Jul 25, 2012 at 8:05 PM, Martin Storsjö

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

2012-07-25 Thread Martin Storsjö
On Wed, 25 Jul 2012, Diego Biurrun wrote: On Wed, Jul 25, 2012 at 04:04:03PM +0200, Luca Barbato wrote: On 07/25/2012 11:59 AM, Diego Biurrun wrote: On Wed, Jul 25, 2012 at 12:43:39PM +0300, Antti Seppälä wrote: It should be possible to specify usernames in http requests containing

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

2012-07-25 Thread Diego Elio Pettenò
Il 25/07/2012 14:07, Diego Biurrun ha scritto: Umm, SWF and FLV files are not the same... I know that (*sighs thinking about the actionscript he's writing*). I mean that for some formats you have the extended name first then the acronym in parenthesis; for other formats you have the acronym and

[libav-devel] [PATCH 1/2] rtmp: Return an error when a server bandwidth packet is less than 4 bytes long

2012-07-25 Thread Samuel Pitoiset
This check is used to make things more consistent because we already do this test for client bandwidth packets. --- libavformat/rtmpproto.c | 7 +++ 1 file changed, 7 insertions(+) diff --git a/libavformat/rtmpproto.c b/libavformat/rtmpproto.c index a2efe38..8d388ab 100644 ---

Re: [libav-devel] [PATCH 1/2] rtmp: Return an error when a server bandwidth packet is less than 4 bytes long

2012-07-25 Thread Martin Storsjö
On Wed, 25 Jul 2012, Samuel Pitoiset wrote: This check is used to make things more consistent because we already do this test for client bandwidth packets. No, this is not a good commit message. The commit message does not need to include every tiny irrelevant detail of the actual change.

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

2012-07-25 Thread Diego Biurrun
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. Give me a bit more time to review this please, I have some ideas I would like to try. --- a/configure +++

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

2012-07-25 Thread Luca Barbato
On 07/25/2012 11:11 PM, Diego Biurrun wrote: I settled for ASF (Advanced / Active Streaming Format) Good to go IMHO. lu -- Luca Barbato Gentoo/linux http://dev.gentoo.org/~lu_zero ___ libav-devel mailing list libav-devel@libav.org

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

2012-07-25 Thread Luca Barbato
On 07/26/2012 12:10 AM, Diego Biurrun wrote: 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. Give me a bit more time to review this please, I have some

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

2012-07-25 Thread Måns Rullgård
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. Give me a bit more time to review this please, I have some ideas

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

2012-07-25 Thread Derek Buitenhuis
On 25/07/2012 6:35 PM, Måns Rullgård wrote: There is no llvm assembler. Clang invokes the gnu assembler for .s files. You actually can use its own assembler with -integrated-as. - Derek ___ libav-devel mailing list libav-devel@libav.org

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

2012-07-25 Thread Ronald S. Bultje
From: Ronald S. Bultje rsbul...@gmail.com --- libswscale/swscale.c |2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/libswscale/swscale.c b/libswscale/swscale.c index 5cfa7f2..0f8ef2b 100644 --- a/libswscale/swscale.c +++ b/libswscale/swscale.c @@ -661,7 +661,7 @@ static int

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

2012-07-25 Thread Ronald S. Bultje
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 | 10 -- libavresample/x86/audio_mix.asm |

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

2012-07-25 Thread Luca Barbato
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. -- Luca Barbato Gentoo/linux http://dev.gentoo.org/~lu_zero ___

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

2012-07-25 Thread Luca Barbato
On 07/26/2012 04:59 AM, 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] [PATCH] x86/dsputilenc: bury inline asm under HAVE_INLINE_ASM.

2012-07-25 Thread Ronald S. Bultje
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 |6 +++ libavcodec/x86/mpegvideo_mmx.c |6

[libav-devel] [PATCH] mpegaudio: bury inline asm under HAVE_INLINE_ASM.

2012-07-25 Thread Ronald S. Bultje
From: Ronald S. Bultje rsbul...@gmail.com --- libavcodec/x86/mpegaudiodec_mmx.c |5 + 1 file changed, 5 insertions(+) diff --git a/libavcodec/x86/mpegaudiodec_mmx.c b/libavcodec/x86/mpegaudiodec_mmx.c index f51a06d..88a3477 100644 --- a/libavcodec/x86/mpegaudiodec_mmx.c +++

[libav-devel] [PATCH] eval: add workaround for broken strtod() in MSVS.

2012-07-25 Thread Ronald S. Bultje
From: Ronald S. Bultje rsbul...@gmail.com --- libavutil/eval.c | 35 +++ 1 file changed, 35 insertions(+) diff --git a/libavutil/eval.c b/libavutil/eval.c index ff3191d..ef37ad8 100644 --- a/libavutil/eval.c +++ b/libavutil/eval.c @@ -26,6 +26,7 @@ * see

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

2012-07-25 Thread Ronald S. Bultje
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 insertions(+) diff --git a/libavutil/eval.c b/libavutil/eval.c index ef37ad8..6131263

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

2012-07-25 Thread Loren Merritt
On Wed, 25 Jul 2012, Ronald S. Bultje wrote: diff --git a/libavutil/x86/x86inc.asm b/libavutil/x86/x86inc.asm index 42ba97a..4b523e9 100644 --- a/libavutil/x86/x86inc.asm +++ b/libavutil/x86/x86inc.asm @@ -369,11 +369,14 @@ DECLARE_REG 14, R15, R15D, R15W, R15B, 120 %macro RET 0

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

2012-07-25 Thread Ronald S. Bultje
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 | 10 -- libavresample/x86/audio_mix.asm |

[libav-devel] [PATCH] avprobe/avconv: fix tentative declaration compile errors on MSVS.

2012-07-25 Thread Ronald S. Bultje
From: Ronald S. Bultje rsbul...@gmail.com --- avconv.c |5 +++-- avprobe.c |5 +++-- 2 files changed, 6 insertions(+), 4 deletions(-) diff --git a/avconv.c b/avconv.c index 7142ab4..439672a 100644 --- a/avconv.c +++ b/avconv.c @@ -104,7 +104,7 @@ typedef struct MetadataMap { int

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

2012-07-25 Thread Måns Rullgård
Derek Buitenhuis derek.buitenh...@gmail.com writes: On 25/07/2012 6:35 PM, Måns Rullgård wrote: There is no llvm assembler. Clang invokes the gnu assembler for .s files. You actually can use its own assembler with -integrated-as. That's something else. With -integrated-as, clang/llvm

  1   2   >