Re: [libav-devel] [PATCH 2/2] h264: convert 8-bit qpel inlined assembly to yasm

2012-08-01 Thread Daniel Kang
On Wed, Aug 1, 2012 at 1:03 AM, Loren Merritt lor...@u.washington.eduwrote: On Wed, 1 Aug 2012, Luca Barbato wrote: +%macro OP_MOVH_MMX 3 +movh %3, %2 +pavgb %1, %3 +movh %2, %1 +%endmacro + +%macro MOVH_MMX 3 +movh %2, %1 +%endmacro + +%macro

[libav-devel] [PATCH] api-example: update to new audio encoding API.

2012-08-01 Thread Anton Khirnov
--- libavcodec/api-example.c | 137 -- 1 file changed, 120 insertions(+), 17 deletions(-) diff --git a/libavcodec/api-example.c b/libavcodec/api-example.c index 93d6c22..bef15dd 100644 --- a/libavcodec/api-example.c +++ b/libavcodec/api-example.c @@

Re: [libav-devel] [PATCH 1/2] h264: remove 3dnow qpel code

2012-08-01 Thread Luca Barbato
On 08/01/2012 07:14 AM, Daniel Kang wrote: On Wed, Aug 1, 2012 at 1:03 AM, Daniel Kang daniel.d.k...@gmail.com wrote: On Wed, Aug 1, 2012 at 12:57 AM, Loren Merritt lor...@u.washington.eduwrote: libavcodec/x86/dsputil_mmx_avg_template.c:58:1: warning: `put_pixels4_l2_3dnow' defined but

Re: [libav-devel] [PATCH] dct-test: allow to compile without HAVE_INLINE_ASM.

2012-08-01 Thread Måns Rullgård
Ronald S. Bultje rsbul...@gmail.com writes: From: Ronald S. Bultje rsbul...@gmail.com --- libavcodec/dct-test.c |2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/libavcodec/dct-test.c b/libavcodec/dct-test.c index 5046544..9e19e0c 100644 --- a/libavcodec/dct-test.c

Re: [libav-devel] [PATCH] dct-test: allow to compile without HAVE_INLINE_ASM.

2012-08-01 Thread Luca Barbato
On 08/01/2012 05:30 AM, Ronald S. Bultje wrote: From: Ronald S. Bultje rsbul...@gmail.com --- libavcodec/dct-test.c |2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/libavcodec/dct-test.c b/libavcodec/dct-test.c index 5046544..9e19e0c 100644 ---

Re: [libav-devel] [libav-commits] dct-test: allow to compile without HAVE_INLINE_ASM.

2012-08-01 Thread Måns Rullgård
Ronald S. Bultje g...@libav.org writes: Module: libav Branch: master Commit: ddbe71b44fc810cc39b576bf0047f89090cfabcb Author:Ronald S. Bultje rsbul...@gmail.com Committer: Ronald S. Bultje rsbul...@gmail.com Date: Tue Jul 31 20:30:29 2012 -0700 dct-test: allow to compile

Re: [libav-devel] [PATCH] dct-test: allow to compile without HAVE_INLINE_ASM.

2012-08-01 Thread Måns Rullgård
Luca Barbato lu_z...@gentoo.org writes: On 08/01/2012 05:30 AM, Ronald S. Bultje wrote: From: Ronald S. Bultje rsbul...@gmail.com --- libavcodec/dct-test.c |2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/libavcodec/dct-test.c b/libavcodec/dct-test.c index

Re: [libav-devel] [PATCH] dct-test: allow to compile without HAVE_INLINE_ASM.

2012-08-01 Thread Luca Barbato
On 08/01/2012 11:12 AM, Måns Rullgård wrote: Luca Barbato lu_z...@gentoo.org writes: On 08/01/2012 05:30 AM, Ronald S. Bultje wrote: From: Ronald S. Bultje rsbul...@gmail.com --- libavcodec/dct-test.c |2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git

Re: [libav-devel] [PATCH] dct-test: allow to compile without HAVE_INLINE_ASM.

2012-08-01 Thread Kostya Shishkov
On Wed, Aug 01, 2012 at 10:12:01AM +0100, Måns Rullgård wrote: Luca Barbato lu_z...@gentoo.org writes: On 08/01/2012 05:30 AM, Ronald S. Bultje wrote: From: Ronald S. Bultje rsbul...@gmail.com --- libavcodec/dct-test.c |2 +- 1 file changed, 1 insertion(+), 1 deletion(-)

Re: [libav-devel] [libav-commits] dct-test: allow to compile without HAVE_INLINE_ASM.

2012-08-01 Thread Luca Barbato
On 08/01/2012 11:11 AM, Måns Rullgård wrote: Ronald S. Bultje g...@libav.org writes: Module: libav Branch: master Commit: ddbe71b44fc810cc39b576bf0047f89090cfabcb Author:Ronald S. Bultje rsbul...@gmail.com Committer: Ronald S. Bultje rsbul...@gmail.com Date: Tue Jul 31 20:30:29

Re: [libav-devel] [PATCH 2/2] h264: convert 8-bit qpel inlined assembly to yasm

2012-08-01 Thread Måns Rullgård
Daniel Kang daniel.d.k...@gmail.com writes: Can I have access to a setup that doesn't have inlined assembly? --extra-cflags=-D__asm__=error should make it fail nicely. -- Måns Rullgård m...@mansr.com ___ libav-devel mailing list

Re: [libav-devel] [PATCH 2/2] rtmp: Prevent handling of an incorrect invoke packet

2012-08-01 Thread Samuel Pitoiset
On Wed, Aug 1, 2012 at 12:34 AM, Derek Buitenhuis derek.buitenh...@gmail.com wrote: On 31/07/2012 12:17 PM, Samuel Pitoiset wrote: +if (memcmp(pkt-data, \002, 1)) { +av_log(s, AV_LOG_ERROR, No string method found in invoke packet\n); +return AVERROR_INVALIDDATA; +}

[libav-devel] [PATCH] rtmp: add functions for reading AMF values

2012-08-01 Thread Jordi Ortiz
--- libavformat/rtmppkt.c | 45 + libavformat/rtmppkt.h | 42 ++ 2 files changed, 87 insertions(+) diff --git a/libavformat/rtmppkt.c b/libavformat/rtmppkt.c index 4ce238d..8228e28 100644 ---

Re: [libav-devel] [PATCH 1/2] (WIP) rtmp: Add message tracking

2012-08-01 Thread Samuel Pitoiset
On Tue, Jul 31, 2012 at 9:22 PM, Luca Barbato lu_z...@gentoo.org wrote: On 07/31/2012 06:16 PM, Samuel Pitoiset wrote: +if ((ret = find_tracked_method(rt, pkt_id, tracked_method)) 0) { +/* Ignore this reply when the method is not tracked. */ +return 0; +

Re: [libav-devel] [PATCH] dct-test: allow to compile without HAVE_INLINE_ASM.

2012-08-01 Thread Måns Rullgård
Luca Barbato lu_z...@gentoo.org writes: On 08/01/2012 11:12 AM, Måns Rullgård wrote: Luca Barbato lu_z...@gentoo.org writes: On 08/01/2012 05:30 AM, Ronald S. Bultje wrote: From: Ronald S. Bultje rsbul...@gmail.com --- libavcodec/dct-test.c |2 +- 1 file changed, 1 insertion(+), 1

[libav-devel] [PATCH] dca: Switch dca_sample_rates to avpriv_ prefix; it is used across libs

2012-08-01 Thread Diego Biurrun
--- libavcodec/dca.c|2 +- libavcodec/dca.h|2 +- libavcodec/dca_parser.c |2 +- libavcodec/dcadec.c |2 +- libavformat/spdifenc.c |2 +- 5 files changed, 5 insertions(+), 5 deletions(-) diff --git a/libavcodec/dca.c b/libavcodec/dca.c index

Re: [libav-devel] [PATCH] dca: Switch dca_sample_rates to avpriv_ prefix; it is used across libs

2012-08-01 Thread Kostya Shishkov
On Wed, Aug 01, 2012 at 11:29:40AM +0200, Diego Biurrun wrote: --- libavcodec/dca.c|2 +- libavcodec/dca.h|2 +- libavcodec/dca_parser.c |2 +- libavcodec/dcadec.c |2 +- libavformat/spdifenc.c |2 +- 5 files changed, 5 insertions(+), 5 deletions(-)

[libav-devel] [PATCH 1/2] (WIP) rtmp: Add message tracking

2012-08-01 Thread Samuel Pitoiset
--- libavformat/rtmpproto.c | 278 1 file changed, 161 insertions(+), 117 deletions(-) diff --git a/libavformat/rtmpproto.c b/libavformat/rtmpproto.c index 4a35319..6044425 100644 --- a/libavformat/rtmpproto.c +++ b/libavformat/rtmpproto.c @@

[libav-devel] [PATCH 2/2] rtmp: Prevent handling of an incorrect invoke packet

2012-08-01 Thread Samuel Pitoiset
--- libavformat/rtmpproto.c | 6 +- 1 file changed, 5 insertions(+), 1 deletion(-) diff --git a/libavformat/rtmpproto.c b/libavformat/rtmpproto.c index 6044425..8c0ecb2 100644 --- a/libavformat/rtmpproto.c +++ b/libavformat/rtmpproto.c @@ -1038,7 +1038,11 @@ static int

Re: [libav-devel] [PATCH] dct-test: allow to compile without HAVE_INLINE_ASM.

2012-08-01 Thread Luca Barbato
On 08/01/2012 11:27 AM, Måns Rullgård wrote: You're supposed to actually *read* patches before approving them. Agreed as I said, I overlooked it. lu -- Luca Barbato Gentoo/linux http://dev.gentoo.org/~lu_zero ___ libav-devel mailing list

[libav-devel] [PATCH] x86: remove libmpeg2 mmx(ext) idct functions

2012-08-01 Thread Mans Rullgard
These functions are not faster than other mmx implementations on any hardware I have been able to test on, and they are horribly inaccurate. There is thus no reason to ever use them. Signed-off-by: Mans Rullgard m...@mansr.com --- libavcodec/dct-test.c| 4 - libavcodec/options_table.h

Re: [libav-devel] [PATCH 30/45] x86: vp8dsp: port FILTER_BILINEAR macro to cpuflags

2012-08-01 Thread Diego Biurrun
On Tue, Jul 31, 2012 at 04:33:37PM -0700, Ronald S. Bultje wrote: On Tue, Jul 31, 2012 at 3:17 PM, Diego Biurrun di...@biurrun.de wrote: --- libavcodec/x86/vp8dsp.asm | 143 + 1 files changed, 66 insertions(+), 77 deletions(-) This is

Re: [libav-devel] [PATCH] x86: remove libmpeg2 mmx(ext) idct functions

2012-08-01 Thread Kostya Shishkov
On Wed, Aug 01, 2012 at 12:15:05PM +0100, Mans Rullgard wrote: These functions are not faster than other mmx implementations on any hardware I have been able to test on, and they are horribly inaccurate. There is thus no reason to ever use them. Signed-off-by: Mans Rullgard m...@mansr.com

[libav-devel] [RFC] doxy for internal API

2012-08-01 Thread anton
Hi, I have a disagreement with Diego about generating doxygen docs. Currently we generate doxygen for all files in Libav, which IMO leads to horribly cluttered output. See e.g. https://libav.org/doxygen/master/files.html https://libav.org/doxygen/master/annotated.html

[libav-devel] [PATCH] x86: vp8dsp: port FILTER_BILINEAR macro to cpuflags

2012-08-01 Thread Diego Biurrun
--- libavcodec/x86/vp8dsp.asm | 141 + 1 files changed, 53 insertions(+), 88 deletions(-) diff --git a/libavcodec/x86/vp8dsp.asm b/libavcodec/x86/vp8dsp.asm index e0041ec..9d1a9fd 100644 --- a/libavcodec/x86/vp8dsp.asm +++ b/libavcodec/x86/vp8dsp.asm

Re: [libav-devel] [PATCH] x86: remove libmpeg2 mmx(ext) idct functions

2012-08-01 Thread Diego Biurrun
On Wed, Aug 01, 2012 at 12:15:05PM +0100, Mans Rullgard wrote: These functions are not faster than other mmx implementations on any hardware I have been able to test on, and they are horribly inaccurate. There is thus no reason to ever use them. Signed-off-by: Mans Rullgard m...@mansr.com

[libav-devel] [PATCH] x86: Refactor PSWAPD fallback implementations and port to cpuflags

2012-08-01 Thread Diego Biurrun
--- libavcodec/x86/fft_mmx.asm| 19 --- libavcodec/x86/fmtconvert.asm | 17 ++--- libavutil/x86/x86util.asm | 12 3 files changed, 18 insertions(+), 30 deletions(-) diff --git a/libavcodec/x86/fft_mmx.asm b/libavcodec/x86/fft_mmx.asm index

Re: [libav-devel] [RFC] doxy for internal API

2012-08-01 Thread Mashiat Sarker Shakkhar
On 8/1/2012 5:48 PM, an...@khirnov.net wrote: [...] If there indeed are people who read generated doxy for non-public API, s/ for non-public API// ;P I'd like them to speak up and prove their existence. Speaking as a user, I just open the header file and read the relevant part. (Also from

Re: [libav-devel] [PATCH 02/45] x86: build: replace mmx2 by mmxext

2012-08-01 Thread Janne Grunau
On 2012-08-01 01:33:47 +0200, Diego Biurrun wrote: On Wed, Aug 01, 2012 at 12:00:58AM +0100, Måns Rullgård wrote: Ronald S. Bultje rsbul...@gmail.com writes: On Tue, Jul 31, 2012 at 3:17 PM, Diego Biurrun di...@biurrun.de wrote: [..] The most important thing is missing: why?

[libav-devel] [PATCH] vp8: pack struct VP8ThreadData more efficiently

2012-08-01 Thread Mans Rullgard
Reordering the members in this struct reduces the holes required to maintain alignment. With this order, the only remaining, and unavoidable, hole is 3 bytes following left_nnz. Signed-off-by: Mans Rullgard m...@mansr.com --- libavcodec/vp8.h | 34 +- 1 file

Re: [libav-devel] [PATCH 01/45] x86: Use consistent 3dnowext file/function/macro name suffixes

2012-08-01 Thread Janne Grunau
On 2012-08-01 00:17:25 +0200, Diego Biurrun wrote: Currently there is a wild mix of 3dn2/3dnow2/3dnowext. Switching to 3dnowext, which is the actual name of the CPU flag unifies this. define actual name it is the same name as the linux kernel reports in /proc/cpuinfo which is also just a name

Re: [libav-devel] [PATCH] vp8: pack struct VP8ThreadData more efficiently

2012-08-01 Thread Kostya Shishkov
On Wed, Aug 01, 2012 at 02:01:11PM +0100, Mans Rullgard wrote: Reordering the members in this struct reduces the holes required to maintain alignment. With this order, the only remaining, and unavoidable, hole is 3 bytes following left_nnz. Signed-off-by: Mans Rullgard m...@mansr.com ---

Re: [libav-devel] [PATCH] vp8: pack struct VP8ThreadData more efficiently

2012-08-01 Thread Janne Grunau
On 2012-08-01 14:01:11 +0100, Måns Rullgård wrote: Reordering the members in this struct reduces the holes required to maintain alignment. With this order, the only remaining, and unavoidable, hole is 3 bytes following left_nnz. Signed-off-by: Mans Rullgard m...@mansr.com ---

Re: [libav-devel] [PATCH 02/45] x86: build: replace mmx2 by mmxext

2012-08-01 Thread Diego Biurrun
On Wed, Aug 01, 2012 at 02:46:36PM +0200, Janne Grunau wrote: On 2012-08-01 01:33:47 +0200, Diego Biurrun wrote: On Wed, Aug 01, 2012 at 12:00:58AM +0100, Måns Rullgård wrote: Ronald S. Bultje rsbul...@gmail.com writes: On Tue, Jul 31, 2012 at 3:17 PM, Diego Biurrun di...@biurrun.de

[libav-devel] [PATCH] x86: Use consistent 3dnowext function and macro name suffixes

2012-08-01 Thread Diego Biurrun
Currently there is a wild mix of 3dn2/3dnow2/3dnowext. Switching to 3dnowext, which is a more common name of the CPU flag, as reported e.g. by the Linux kernel, unifies this. --- libavcodec/x86/dsputil_mmx.c| 14 ++-- libavcodec/x86/fft.c|6 ++--

[libav-devel] [PATCH] dct-test: always link with aandcttab.o

2012-08-01 Thread Mans Rullgard
This allows building dct-test even if aandcttab.o is not pulled in by any enabled codec. The DCT with which these tables are used does not use them directly, so building it without the tables is possible. Signed-off-by: Mans Rullgard m...@mansr.com --- libavcodec/Makefile | 2 +- 1 file

Re: [libav-devel] [PATCH] dct-test: always link with aandcttab.o

2012-08-01 Thread Kostya Shishkov
On Wed, Aug 01, 2012 at 02:32:24PM +0100, Mans Rullgard wrote: This allows building dct-test even if aandcttab.o is not pulled in by any enabled codec. The DCT with which these tables are used does not use them directly, so building it without the tables is possible. Signed-off-by: Mans

Re: [libav-devel] [PATCH] dct-test: always link with aandcttab.o

2012-08-01 Thread Diego Biurrun
On Wed, Aug 01, 2012 at 02:32:24PM +0100, Mans Rullgard wrote: This allows building dct-test even if aandcttab.o is not pulled in by any enabled codec. The DCT with which these tables are used does not use them directly, so building it without the tables is possible. --- libavcodec/Makefile

Re: [libav-devel] [PATCH 02/45] x86: build: replace mmx2 by mmxext

2012-08-01 Thread Janne Grunau
On 2012-08-01 15:24:37 +0200, Diego Biurrun wrote: On Wed, Aug 01, 2012 at 02:46:36PM +0200, Janne Grunau wrote: On 2012-08-01 01:33:47 +0200, Diego Biurrun wrote: On Wed, Aug 01, 2012 at 12:00:58AM +0100, Måns Rullgård wrote: Ronald S. Bultje rsbul...@gmail.com writes: On Tue, Jul

Re: [libav-devel] [PATCH 02/45] x86: build: replace mmx2 by mmxext

2012-08-01 Thread Diego Biurrun
On Wed, Aug 01, 2012 at 04:04:50PM +0200, Janne Grunau wrote: On 2012-08-01 15:24:37 +0200, Diego Biurrun wrote: On Wed, Aug 01, 2012 at 02:46:36PM +0200, Janne Grunau wrote: On 2012-08-01 01:33:47 +0200, Diego Biurrun wrote: On Wed, Aug 01, 2012 at 12:00:58AM +0100, Måns Rullgård wrote:

Re: [libav-devel] [RFC] doxy for internal API

2012-08-01 Thread Justin Ruggles
On 08/01/2012 07:48 AM, an...@khirnov.net wrote: Hi, I have a disagreement with Diego about generating doxygen docs. Currently we generate doxygen for all files in Libav, which IMO leads to horribly cluttered output. See e.g. https://libav.org/doxygen/master/files.html

Re: [libav-devel] [PATCH 33/45] x86: PALIGNR: port to cpuflags

2012-08-01 Thread Justin Ruggles
On 07/31/2012 06:17 PM, Diego Biurrun wrote: --- libavcodec/x86/h264_intrapred.asm | 27 --- libavcodec/x86/h264_intrapred_10bit.asm | 16 libavcodec/x86/h264_qpel_10bit.asm |4 libavutil/x86/x86util.asm | 16

Re: [libav-devel] [PATCH 02/45] x86: build: replace mmx2 by mmxext

2012-08-01 Thread Ronald S. Bultje
Hi, On Wed, Aug 1, 2012 at 6:24 AM, Diego Biurrun di...@biurrun.de wrote: On Wed, Aug 01, 2012 at 02:46:36PM +0200, Janne Grunau wrote: On 2012-08-01 01:33:47 +0200, Diego Biurrun wrote: On Wed, Aug 01, 2012 at 12:00:58AM +0100, Måns Rullgård wrote: Ronald S. Bultje rsbul...@gmail.com

Re: [libav-devel] [PATCH 34/45] x86: PABSW: port to cpuflags

2012-08-01 Thread Justin Ruggles
On 07/31/2012 06:17 PM, Diego Biurrun wrote: --- libavcodec/x86/vc1dsp_yasm.asm |3 --- libavutil/x86/x86util.asm | 22 ++ 2 files changed, 10 insertions(+), 15 deletions(-) patch looks fine. But I think ideally PABSW should be merged with ABS1. It's quite

Re: [libav-devel] [PATCH] x86: vp8dsp: port FILTER_BILINEAR macro to cpuflags

2012-08-01 Thread Ronald S. Bultje
Hi, On Wed, Aug 1, 2012 at 5:08 AM, Diego Biurrun di...@biurrun.de wrote: +FILTER_BILINEAR 8, 7, m2 INIT_MMX ssse3 -FILTER_BILINEAR_SSSE3 4 +FILTER_BILINEAR 4, 7, m2 INIT_XMM ssse3 -FILTER_BILINEAR_SSSE3 8 +FILTER_BILINEAR 8, 5, m1 So, uhm, we should probably discuss this on IRC before

Re: [libav-devel] [PATCH] x86: Refactor PSWAPD fallback implementations and port to cpuflags

2012-08-01 Thread Ronald S. Bultje
Hi, On Wed, Aug 1, 2012 at 5:30 AM, Diego Biurrun di...@biurrun.de wrote: --- libavcodec/x86/fft_mmx.asm| 19 --- libavcodec/x86/fmtconvert.asm | 17 ++--- libavutil/x86/x86util.asm | 12 3 files changed, 18 insertions(+), 30

[libav-devel] [PATCH 08/15] lavr: x86: optimized 2-channel s16 to s16p conversion

2012-08-01 Thread Justin Ruggles
--- libavresample/x86/audio_convert.asm| 51 libavresample/x86/audio_convert_init.c | 15 + libavresample/x86/util.asm |6 3 files changed, 72 insertions(+), 0 deletions(-) diff --git a/libavresample/x86/audio_convert.asm

[libav-devel] [PATCH 10/15] lavr: x86: optimized 2-channel s16 to fltp conversion

2012-08-01 Thread Justin Ruggles
--- libavresample/x86/audio_convert.asm| 40 libavresample/x86/audio_convert_init.c | 13 ++ 2 files changed, 53 insertions(+), 0 deletions(-) diff --git a/libavresample/x86/audio_convert.asm b/libavresample/x86/audio_convert.asm index

[libav-devel] [PATCH 12/15] lavr: x86: optimized 2-channel flt to s16p conversion

2012-08-01 Thread Justin Ruggles
--- libavresample/x86/audio_convert.asm| 47 libavresample/x86/audio_convert_init.c |9 ++ 2 files changed, 56 insertions(+), 0 deletions(-) diff --git a/libavresample/x86/audio_convert.asm b/libavresample/x86/audio_convert.asm index

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

2012-08-01 Thread Justin Ruggles
--- libavresample/x86/audio_convert.asm| 33 libavresample/x86/audio_convert_init.c |9 2 files changed, 42 insertions(+), 0 deletions(-) diff --git a/libavresample/x86/audio_convert.asm b/libavresample/x86/audio_convert.asm index

[libav-devel] [PATCH] idcin: check for integer overflow when calling av_get_packet()

2012-08-01 Thread Justin Ruggles
chunk_size is unsigned 32-bit, but av_get_packet() takes a signed int as the packet size. --- libavformat/idcin.c |5 + 1 files changed, 5 insertions(+), 0 deletions(-) diff --git a/libavformat/idcin.c b/libavformat/idcin.c index 6a9b0ca..3acbc92 100644 --- a/libavformat/idcin.c +++

[libav-devel] [PATCH 1/2] rtmp: Add support for subscribing live streams

2012-08-01 Thread Samuel Pitoiset
When streaming live streams using the Akamai, Edgecast or Limelight CDN, players cannot simply connect to the live stream. Instead, they have to subscribe to it, by sending an FC Subscribe call to the server. --- libavformat/rtmpproto.c | 28 1 file changed, 28

[libav-devel] [PATCH 2/2] rtmp: Add a new option 'rtmp_subscribe'

2012-08-01 Thread Samuel Pitoiset
This option specifies the name of live stream to subscribe. Defaults to rtmp_playpath. --- doc/protocols.texi | 3 +++ libavformat/rtmpproto.c | 9 +++-- libavformat/version.h | 2 +- 3 files changed, 11 insertions(+), 3 deletions(-) diff --git a/doc/protocols.texi

Re: [libav-devel] [PATCH 2/2] rtmp: Prevent handling of an incorrect invoke packet

2012-08-01 Thread Derek Buitenhuis
On 01/08/2012 6:13 AM, Samuel Pitoiset wrote: --- libavformat/rtmpproto.c | 6 +- 1 file changed, 5 insertions(+), 1 deletion(-) Looks OK, but I have no idea where the magic numbers come from for this or the lines following it. Perhaps you can add comments in a separate patch? - Derek

Re: [libav-devel] [PATCH 02/45] x86: build: replace mmx2 by mmxext

2012-08-01 Thread Diego Biurrun
On Wed, Aug 01, 2012 at 07:36:09AM -0700, Ronald S. Bultje wrote: On Wed, Aug 1, 2012 at 6:24 AM, Diego Biurrun di...@biurrun.de wrote: On Wed, Aug 01, 2012 at 02:46:36PM +0200, Janne Grunau wrote: On 2012-08-01 01:33:47 +0200, Diego Biurrun wrote: On Wed, Aug 01, 2012 at 12:00:58AM +0100,

[libav-devel] [PATCH] vc1dec: remove useless #include simple_idct.h

2012-08-01 Thread Mans Rullgard
Signed-off-by: Mans Rullgard m...@mansr.com --- libavcodec/vc1dec.c | 1 - 1 file changed, 1 deletion(-) diff --git a/libavcodec/vc1dec.c b/libavcodec/vc1dec.c index c6cbfc1..e36cc0d 100644 --- a/libavcodec/vc1dec.c +++ b/libavcodec/vc1dec.c @@ -36,7 +36,6 @@ #include vc1acdata.h #include

[libav-devel] [PATCH] x86: PALIGNR: port to cpuflags

2012-08-01 Thread Diego Biurrun
--- libavcodec/x86/h264_intrapred.asm | 27 --- libavcodec/x86/h264_intrapred_10bit.asm | 16 libavcodec/x86/h264_qpel_10bit.asm |4 libavutil/x86/x86util.asm | 16 libswscale/x86/output.asm

Re: [libav-devel] [PATCH] idcin: check for integer overflow when calling av_get_packet()

2012-08-01 Thread Kostya Shishkov
On Wed, Aug 01, 2012 at 11:31:23AM -0400, Justin Ruggles wrote: chunk_size is unsigned 32-bit, but av_get_packet() takes a signed int as the packet size. --- libavformat/idcin.c |5 + 1 files changed, 5 insertions(+), 0 deletions(-) diff --git a/libavformat/idcin.c

Re: [libav-devel] [PATCH 2/2] rtmp: Prevent handling of an incorrect invoke packet

2012-08-01 Thread Samuel Pitoiset
On Wed, Aug 1, 2012 at 6:09 PM, Derek Buitenhuis derek.buitenh...@gmail.com wrote: On 01/08/2012 6:13 AM, Samuel Pitoiset wrote: --- libavformat/rtmpproto.c | 6 +- 1 file changed, 5 insertions(+), 1 deletion(-) Looks OK, but I have no idea where the magic numbers come from for this or

Re: [libav-devel] [PATCH] vc1dec: remove useless #include simple_idct.h

2012-08-01 Thread Diego Biurrun
On Wed, Aug 01, 2012 at 05:15:47PM +0100, Mans Rullgard wrote: Signed-off-by: Mans Rullgard m...@mansr.com --- libavcodec/vc1dec.c | 1 - 1 file changed, 1 deletion(-) OK Diego ___ libav-devel mailing list libav-devel@libav.org

[libav-devel] [PATCH] Revert dct-test: allow to compile without HAVE_INLINE_ASM.

2012-08-01 Thread Ronald S. Bultje
From: Ronald S. Bultje rsbul...@gmail.com This reverts commit ddbe71b44fc810cc39b576bf0047f89090cfabcb. I forgot to remove it from my tree before pushing another patch. --- libavcodec/dct-test.c |2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/libavcodec/dct-test.c

[libav-devel] [PATCH] x86: dct32: port to cpuflags

2012-08-01 Thread Diego Biurrun
--- libavcodec/x86/dct32_sse.asm | 74 ++--- 1 files changed, 25 insertions(+), 49 deletions(-) diff --git a/libavcodec/x86/dct32_sse.asm b/libavcodec/x86/dct32_sse.asm index 20a5c7b..65fac09 100644 --- a/libavcodec/x86/dct32_sse.asm +++

Re: [libav-devel] [PATCH] Revert dct-test: allow to compile without HAVE_INLINE_ASM.

2012-08-01 Thread Diego Biurrun
On Wed, Aug 01, 2012 at 07:15:13PM +0200, Kostya Shishkov wrote: On Wed, Aug 01, 2012 at 09:30:33AM -0700, Ronald S. Bultje wrote: From: Ronald S. Bultje rsbul...@gmail.com This reverts commit ddbe71b44fc810cc39b576bf0047f89090cfabcb. I forgot to remove it from my tree before pushing

Re: [libav-devel] [PATCH] api-example: update to new audio encoding API.

2012-08-01 Thread Justin Ruggles
On 08/01/2012 02:35 AM, Anton Khirnov wrote: --- libavcodec/api-example.c | 137 -- 1 file changed, 120 insertions(+), 17 deletions(-) LGTM -Justin ___ libav-devel mailing list libav-devel@libav.org

Re: [libav-devel] [PATCH] x86: PALIGNR: port to cpuflags

2012-08-01 Thread Justin Ruggles
On 08/01/2012 12:18 PM, Diego Biurrun wrote: --- libavcodec/x86/h264_intrapred.asm | 27 --- libavcodec/x86/h264_intrapred_10bit.asm | 16 libavcodec/x86/h264_qpel_10bit.asm |4 libavutil/x86/x86util.asm | 16

Re: [libav-devel] [PATCH 2/2] rtmp: Prevent handling of an incorrect invoke packet

2012-08-01 Thread Derek Buitenhuis
On 01/08/2012 12:27 PM, Samuel Pitoiset wrote: 0x02 is a magic number defined in the rtmp specification which indicates that data is string. I don't think a comment is needed here because it's already specified in the error message. OK. - Derek ___

[libav-devel] [PATCH v3] Canopus Lossless decoder

2012-08-01 Thread Derek Buitenhuis
At the moment it only does BGR24, but I plan to add the rest after. Signed-off-by: Derek Buitenhuis derek.buitenh...@gmail.com --- Changelog |1 + doc/general.texi |1 + libavcodec/Makefile|1 + libavcodec/allcodecs.c |1 + libavcodec/avcodec.h |1 +

[libav-devel] [PATCH] x86: Refactor PSWAPD fallback implementations and port to cpuflags

2012-08-01 Thread Diego Biurrun
--- libavcodec/x86/fft_mmx.asm| 13 - libavcodec/x86/fmtconvert.asm | 17 ++--- libavutil/x86/x86util.asm | 12 3 files changed, 14 insertions(+), 28 deletions(-) diff --git a/libavcodec/x86/fft_mmx.asm b/libavcodec/x86/fft_mmx.asm index

Re: [libav-devel] [PATCH] x86: Refactor PSWAPD fallback implementations and port to cpuflags

2012-08-01 Thread Ronald S. Bultje
Hi, On Wed, Aug 1, 2012 at 10:53 AM, Diego Biurrun di...@biurrun.de wrote: --- a/libavcodec/x86/fft_mmx.asm [..] -%elifidn %1, %2 -movd [r0+12], %1 -punpckhdq %1, [r0+8] -%else If you remove this macro from this file, the callers with %1==%2 need to use the code directly, or call a

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

2012-08-01 Thread Ronald S. Bultje
Hi, On Wed, Jul 25, 2012 at 10:35 PM, Måns Rullgård m...@mansr.com wrote: Luca Barbato lu_z...@gentoo.org writes: From: Ronald S. Bultje rsbul...@gmail.com --- Here my initial twist about it, ideally I'd consider moving os_support in libavu and include it automagically from config.h

Re: [libav-devel] [PATCH v3] Canopus Lossless decoder

2012-08-01 Thread Kostya Shishkov
On Wed, Aug 01, 2012 at 01:50:32PM -0400, Derek Buitenhuis wrote: At the moment it only does BGR24, but I plan to add the rest after. Signed-off-by: Derek Buitenhuis derek.buitenh...@gmail.com --- Changelog |1 + doc/general.texi |1 + libavcodec/Makefile|

Re: [libav-devel] [PATCH v3] Canopus Lossless decoder

2012-08-01 Thread Johan Andersson
On Wed, Aug 01, 2012 at 01:50:32PM -0400, Derek Buitenhuis wrote: diff --git a/libavcodec/version.h b/libavcodec/version.h index acad4d4..a74b0ad 100644 --- a/libavcodec/version.h +++ b/libavcodec/version.h @@ -27,7 +27,7 @@ */ #define LIBAVCODEC_VERSION_MAJOR 54 -#define

[libav-devel] [PATCH v4] Canopus Lossless decoder

2012-08-01 Thread Derek Buitenhuis
At the moment it only does BGR24, but I plan to add the rest after. Signed-off-by: Derek Buitenhuis derek.buitenh...@gmail.com --- Changelog |1 + doc/general.texi |1 + libavcodec/Makefile|1 + libavcodec/allcodecs.c |1 + libavcodec/avcodec.h |1 +

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

2012-08-01 Thread Måns Rullgård
Luca Barbato lu_z...@gentoo.org writes: +#if !HAVE_SNPRINTF +#ifdef _MSC_VER +#define vsnprintf _vsnprintf +#endif + +int snprintf(char *buffer, size_t bufsize, const char *fmt, ...) +{ +va_list ap; +int ret; + +if ((int)bufsize = 0) return -1; +va_start(ap, fmt); +

Re: [libav-devel] [PATCH v4] Canopus Lossless decoder

2012-08-01 Thread Kostya Shishkov
On Wed, Aug 01, 2012 at 02:46:53PM -0400, Derek Buitenhuis wrote: At the moment it only does BGR24, but I plan to add the rest after. Signed-off-by: Derek Buitenhuis derek.buitenh...@gmail.com --- Changelog |1 + doc/general.texi |1 + libavcodec/Makefile|

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

2012-08-01 Thread Ronald S. Bultje
Hi, On Wed, Aug 1, 2012 at 11:58 AM, Måns Rullgård m...@mansr.com wrote: Luca Barbato lu_z...@gentoo.org writes: +#if !HAVE_SNPRINTF +#ifdef _MSC_VER +#define vsnprintf _vsnprintf +#endif + +int snprintf(char *buffer, size_t bufsize, const char *fmt, ...) +{ +va_list ap; +int

Re: [libav-devel] [PATCH 1/3] api-example: update to new audio encoding API.

2012-08-01 Thread Kieran Kunhya
+/* the codec gives us the frame size, in samples, + * we calculate the size of the samples buffer in bytes */ +buffer_size = av_samples_get_buffer_size(NULL, c-channels, c-frame_size, + c-sample_fmt, 0); +samples =

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

2012-08-01 Thread Uoti Urpala
On Thu, 2012-07-26 at 07:32 +0200, Luca Barbato wrote: +int snprintf(char *buffer, size_t bufsize, const char *fmt, ...) +{ +va_list ap; +int ret; + +if ((int)bufsize = 0) return -1; bufsize == 0 is valid. +ret = vsnprintf(buffer, bufsize-1, fmt, ap); IIRC MSVC (or the C

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

2012-08-01 Thread Måns Rullgård
Ronald S. Bultje rsbul...@gmail.com writes: Hi, On Wed, Aug 1, 2012 at 11:58 AM, Måns Rullgård m...@mansr.com wrote: Luca Barbato lu_z...@gentoo.org writes: +#if !HAVE_SNPRINTF +#ifdef _MSC_VER +#define vsnprintf _vsnprintf +#endif + +int snprintf(char *buffer, size_t bufsize, const

[libav-devel] [PATCH] eamad: Use dsputils instead of its own bswap16_buf

2012-08-01 Thread Derek Buitenhuis
Signed-off-by: Derek Buitenhuis derek.buitenh...@gmail.com --- libavcodec/eamad.c |9 + 1 file changed, 1 insertion(+), 8 deletions(-) diff --git a/libavcodec/eamad.c b/libavcodec/eamad.c index ca74393..d7e65db 100644 --- a/libavcodec/eamad.c +++ b/libavcodec/eamad.c @@ -57,13 +57,6

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

2012-08-01 Thread Måns Rullgård
Ronald S. Bultje rsbul...@gmail.com writes: Hi, On Wed, Aug 1, 2012 at 11:58 AM, Måns Rullgård m...@mansr.com wrote: Luca Barbato lu_z...@gentoo.org writes: +#if !HAVE_SNPRINTF +#ifdef _MSC_VER +#define vsnprintf _vsnprintf +#endif + +int snprintf(char *buffer, size_t bufsize, const

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

2012-08-01 Thread Måns Rullgård
Uoti Urpala uoti.urp...@pp1.inet.fi writes: On Thu, 2012-07-26 at 07:32 +0200, Luca Barbato wrote: +int snprintf(char *buffer, size_t bufsize, const char *fmt, ...) +{ +va_list ap; +int ret; + +if ((int)bufsize = 0) return -1; bufsize == 0 is valid. Of course zero is a

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

2012-08-01 Thread Uoti Urpala
On Wed, 2012-08-01 at 21:22 +0100, Måns Rullgård wrote: +int snprintf(char *buffer, size_t bufsize, const char *fmt, ...) +if ((int)bufsize = 0) return -1; If bufsize INT_MAX, that cast has unspecified behaviour. No, it's implementation-defined (you're probably confusing it with

Re: [libav-devel] [PATCH 2/2] rtmp: Prevent handling of an incorrect invoke packet

2012-08-01 Thread Luca Barbato
On 01/08/12 12:13, Samuel Pitoiset wrote: --- libavformat/rtmpproto.c | 6 +- 1 file changed, 5 insertions(+), 1 deletion(-) diff --git a/libavformat/rtmpproto.c b/libavformat/rtmpproto.c index 6044425..8c0ecb2 100644 --- a/libavformat/rtmpproto.c +++ b/libavformat/rtmpproto.c @@

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

2012-08-01 Thread Uoti Urpala
On Wed, 2012-08-01 at 21:34 +0100, Måns Rullgård wrote: Uoti Urpala uoti.urp...@pp1.inet.fi writes: +ret = vsnprintf(buffer, bufsize-1, fmt, ap); IIRC MSVC (or the C library) does not support all standard format modifiers, so unless this vsnprintf comes from elsewhere, you need more

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

2012-08-01 Thread Måns Rullgård
Uoti Urpala uoti.urp...@pp1.inet.fi writes: On Wed, 2012-08-01 at 21:22 +0100, Måns Rullgård wrote: +int snprintf(char *buffer, size_t bufsize, const char *fmt, ...) +if ((int)bufsize = 0) return -1; If bufsize INT_MAX, that cast has unspecified behaviour. No, it's

Re: [libav-devel] [PATCH 1/3] api-example: update to new audio encoding API.

2012-08-01 Thread Anton Khirnov
On Wed, 1 Aug 2012 20:39:36 +0100, Kieran Kunhya kier...@ob-encoder.com wrote: +/* the codec gives us the frame size, in samples, + * we calculate the size of the samples buffer in bytes */ +buffer_size = av_samples_get_buffer_size(NULL, c-channels, c-frame_size, +

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

2012-08-01 Thread Måns Rullgård
Uoti Urpala uoti.urp...@pp1.inet.fi writes: On Wed, 2012-08-01 at 21:34 +0100, Måns Rullgård wrote: Uoti Urpala uoti.urp...@pp1.inet.fi writes: +ret = vsnprintf(buffer, bufsize-1, fmt, ap); IIRC MSVC (or the C library) does not support all standard format modifiers, so unless this

Re: [libav-devel] [PATCH 1/2] (WIP) rtmp: Add message tracking

2012-08-01 Thread Luca Barbato
On 01/08/12 12:12, Samuel Pitoiset wrote: --- libavformat/rtmpproto.c | 278 1 file changed, 161 insertions(+), 117 deletions(-) Seems better. Let me try it. ___ libav-devel mailing list

Re: [libav-devel] [PATCH] rtmp: add functions for reading AMF values

2012-08-01 Thread Luca Barbato
On 01/08/12 11:25, Jordi Ortiz wrote: --- libavformat/rtmppkt.c | 45 + libavformat/rtmppkt.h | 42 ++ 2 files changed, 87 insertions(+) Seems fine to me. I'd commit it tomorrow. lu

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

2012-08-01 Thread Uoti Urpala
On Wed, 2012-08-01 at 21:49 +0100, Måns Rullgård wrote: Uoti Urpala uoti.urp...@pp1.inet.fi writes: On Wed, 2012-08-01 at 21:22 +0100, Måns Rullgård wrote: +int snprintf(char *buffer, size_t bufsize, const char *fmt, ...) +if ((int)bufsize = 0) return -1; If bufsize INT_MAX,

Re: [libav-devel] [PATCH] eamad: Use dsputils instead of its own bswap16_buf

2012-08-01 Thread Justin Ruggles
On 08/01/2012 04:22 PM, Derek Buitenhuis wrote: Signed-off-by: Derek Buitenhuis derek.buitenh...@gmail.com --- libavcodec/eamad.c |9 + 1 file changed, 1 insertion(+), 8 deletions(-) diff --git a/libavcodec/eamad.c b/libavcodec/eamad.c index ca74393..d7e65db 100644 ---

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

2012-08-01 Thread Uoti Urpala
On Wed, 2012-08-01 at 21:55 +0100, Måns Rullgård wrote: Uoti Urpala uoti.urp...@pp1.inet.fi writes: On Wed, 2012-08-01 at 21:34 +0100, Måns Rullgård wrote: It is not possible to get those semantics using the regular Windows functions. Of course it is, with enough workarounds (this is

[libav-devel] [PATCH 01/10] idcinvideo: correctly set AVFrame defaults

2012-08-01 Thread Justin Ruggles
--- libavcodec/idcinvideo.c |2 +- 1 files changed, 1 insertions(+), 1 deletions(-) diff --git a/libavcodec/idcinvideo.c b/libavcodec/idcinvideo.c index c936ebb..65414ee 100644 --- a/libavcodec/idcinvideo.c +++ b/libavcodec/idcinvideo.c @@ -166,7 +166,7 @@ static av_cold int

[libav-devel] [PATCH 02/10] idcin: validate header parameters

2012-08-01 Thread Justin Ruggles
Avoids using unsupported parameters and signed integer overflows. --- libavformat/idcin.c | 19 +++ 1 files changed, 19 insertions(+), 0 deletions(-) diff --git a/libavformat/idcin.c b/libavformat/idcin.c index 6a9b0ca..f278f42 100644 --- a/libavformat/idcin.c +++

[libav-devel] [PATCH 03/10] idcin: fix check for presence of an audio stream

2012-08-01 Thread Justin Ruggles
--- libavformat/idcin.c | 10 ++ 1 files changed, 6 insertions(+), 4 deletions(-) diff --git a/libavformat/idcin.c b/libavformat/idcin.c index f278f42..113c66e 100644 --- a/libavformat/idcin.c +++ b/libavformat/idcin.c @@ -170,6 +170,10 @@ static int idcin_read_header(AVFormatContext

[libav-devel] [PATCH 05/10] idcin: set start_time and packet duration instead of manually tracking pts.

2012-08-01 Thread Justin Ruggles
Also, use 1 / sample_rate for audio stream time_base. --- libavformat/idcin.c | 13 +++-- 1 files changed, 7 insertions(+), 6 deletions(-) diff --git a/libavformat/idcin.c b/libavformat/idcin.c index c8b8225..0e93a59 100644 --- a/libavformat/idcin.c +++ b/libavformat/idcin.c @@ -82,6

[libav-devel] [PATCH 06/10] idcin: set AV_PKT_FLAG_KEY for video packets with a palette

2012-08-01 Thread Justin Ruggles
--- libavformat/idcin.c |1 + 1 files changed, 1 insertions(+), 0 deletions(-) diff --git a/libavformat/idcin.c b/libavformat/idcin.c index 0e93a59..638377c 100644 --- a/libavformat/idcin.c +++ b/libavformat/idcin.c @@ -292,6 +292,7 @@ static int idcin_read_packet(AVFormatContext *s,

[libav-devel] [PATCH 07/10] idcin: allow seeking back to the first packet

2012-08-01 Thread Justin Ruggles
--- libavformat/idcin.c | 21 - 1 files changed, 20 insertions(+), 1 deletions(-) diff --git a/libavformat/idcin.c b/libavformat/idcin.c index 638377c..67915e6 100644 --- a/libavformat/idcin.c +++ b/libavformat/idcin.c @@ -89,7 +89,7 @@ typedef struct IdcinDemuxContext {

[libav-devel] [PATCH 08/10] idcin: check for integer overflow when calling av_get_packet()

2012-08-01 Thread Justin Ruggles
chunk_size is unsigned 32-bit, but av_get_packet() takes a signed int as the packet size. --- libavformat/idcin.c |4 1 files changed, 4 insertions(+), 0 deletions(-) diff --git a/libavformat/idcin.c b/libavformat/idcin.c index 67915e6..2d506ff 100644 --- a/libavformat/idcin.c +++

  1   2   >