Re: [libav-devel] [PATCH 2/8] segment: Properly create new AVStreams for the chained muxer

2012-10-04 Thread Anton Khirnov
On Thu, 04 Oct 2012 00:12:38 +0200, Luca Barbato lu_z...@gentoo.org wrote: On 10/03/2012 11:37 PM, Martin Storsjö wrote: Before, the chained muxer reused the AVStreams array from the outer muxer, which made it impossible to use the proper public functions (such as av_write_frame) when

Re: [libav-devel] [PATCH 2/8] segment: Properly create new AVStreams for the chained muxer

2012-10-04 Thread Martin Storsjö
On Thu, 4 Oct 2012, Anton Khirnov wrote: On Thu, 04 Oct 2012 00:12:38 +0200, Luca Barbato lu_z...@gentoo.org wrote: On 10/03/2012 11:37 PM, Martin Storsjö wrote: Before, the chained muxer reused the AVStreams array from the outer muxer, which made it impossible to use the proper public

Re: [libav-devel] [PATCH 1/4] x86: ff_get_cpu_flags_x86(): Avoid a pointless variable indirection

2012-10-04 Thread Diego Biurrun
On Wed, Oct 03, 2012 at 10:11:37PM -0400, Sean McGovern wrote: On Wed, Oct 3, 2012 at 7:06 PM, Diego Biurrun di...@biurrun.de wrote: --- a/libavutil/x86/cpu.c +++ b/libavutil/x86/cpu.c @@ -108,10 +108,7 @@ int ff_get_cpu_flags_x86(void) -cpuid(0, max_std_level, ebx, ecx, edx); -

Re: [libav-devel] [PATCH 1/4] x86: ff_get_cpu_flags_x86(): Avoid a pointless variable indirection

2012-10-04 Thread Diego Biurrun
On Thu, Oct 04, 2012 at 12:44:12AM +0100, Måns Rullgård wrote: Diego Biurrun di...@biurrun.de writes: On Thu, Oct 04, 2012 at 12:27:09AM +0100, Måns Rullgård wrote: Diego Biurrun di...@biurrun.de writes: --- a/libavutil/x86/cpu.c +++ b/libavutil/x86/cpu.c @@ -108,10 +108,7 @@ int

Re: [libav-devel] [PATCH 2/2] vf_overlay: properly sync inputs.

2012-10-04 Thread Luca Barbato
On 10/04/2012 07:29 AM, Anton Khirnov wrote: On Thu, 04 Oct 2012 00:21:12 +0200, Luca Barbato lu_z...@gentoo.org wrote: On 10/03/2012 10:56 PM, Anton Khirnov wrote: Right now it incorrectly assumes that the frames are sent in the proper order, which worked with old ffmpeg and avconv versions

Re: [libav-devel] [PATCH 1/4] x86: ff_get_cpu_flags_x86(): Avoid a pointless variable indirection

2012-10-04 Thread Måns Rullgård
Diego Biurrun di...@biurrun.de writes: On Wed, Oct 03, 2012 at 10:11:37PM -0400, Sean McGovern wrote: On Wed, Oct 3, 2012 at 7:06 PM, Diego Biurrun di...@biurrun.de wrote: --- a/libavutil/x86/cpu.c +++ b/libavutil/x86/cpu.c @@ -108,10 +108,7 @@ int ff_get_cpu_flags_x86(void) -

Re: [libav-devel] [PATCH] doc: initial nut documentation

2012-10-04 Thread Luca Barbato
On 10/02/2012 11:28 PM, Luca Barbato wrote: --- Ping... doc/formats/nut.texi | 130 +++ 1 file changed, 130 insertions(+) create mode 100644 doc/formats/nut.texi diff --git a/doc/formats/nut.texi b/doc/formats/nut.texi new file mode

Re: [libav-devel] [PATCH 2/2] nut: add support for yuva422p and yuva444p

2012-10-04 Thread Luca Barbato
On 10/02/2012 11:21 AM, Luca Barbato wrote: --- libavcodec/raw.c | 2 ++ libavformat/nut.c | 2 ++ 2 files changed, 4 insertions(+) Ping. ___ libav-devel mailing list libav-devel@libav.org https://lists.libav.org/mailman/listinfo/libav-devel

Re: [libav-devel] [PATCH 2/8] segment: Properly create new AVStreams for the chained muxer

2012-10-04 Thread Anton Khirnov
On Thu, 4 Oct 2012 11:43:54 +0300 (EEST), Martin Storsjö mar...@martin.st wrote: On Thu, 4 Oct 2012, Anton Khirnov wrote: On Thu, 04 Oct 2012 00:12:38 +0200, Luca Barbato lu_z...@gentoo.org wrote: On 10/03/2012 11:37 PM, Martin Storsjö wrote: Before, the chained muxer reused the

Re: [libav-devel] [PATCH 6/8] segment: Set the resend_headers flag for each segment

2012-10-04 Thread Anton Khirnov
On Thu, 4 Oct 2012 00:37:22 +0300, Martin Storsjö mar...@martin.st wrote: This makes sure new inline headers are emitted when the next packet is written. This allows segmenting mpegts without calling write_header/write_trailer (nor freeing/reiniting the muxer) for each segment. ---

Re: [libav-devel] [PATCH 1/4] x86: ff_get_cpu_flags_x86(): Avoid a pointless variable indirection

2012-10-04 Thread Sean McGovern
On Thu, Oct 4, 2012 at 6:14 AM, Måns Rullgård m...@mansr.com wrote: Diego Biurrun di...@biurrun.de writes: On Wed, Oct 03, 2012 at 10:11:37PM -0400, Sean McGovern wrote: On Wed, Oct 3, 2012 at 7:06 PM, Diego Biurrun di...@biurrun.de wrote: --- a/libavutil/x86/cpu.c +++

[libav-devel] [PATCH] segment: Add comments about calls that only are relevant for some muxers

2012-10-04 Thread Martin Storsjö
--- libavformat/segment.c |4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/libavformat/segment.c b/libavformat/segment.c index 1e928a7..375df58 100644 --- a/libavformat/segment.c +++ b/libavformat/segment.c @@ -98,7 +98,7 @@ static int segment_start(AVFormatContext *s,

[libav-devel] [PATCH 2/5] x86: cpu: Break out test for cpuid capabilities into separate function

2012-10-04 Thread Diego Biurrun
--- The #define indirection will be reused by the yasm version of cpuid_test, so I'm adding it here already to avoid code churn. libavutil/x86/cpu.c | 27 ++- 1 files changed, 18 insertions(+), 9 deletions(-) diff --git a/libavutil/x86/cpu.c b/libavutil/x86/cpu.c

[libav-devel] [PATCH 4/5] x86: Drop CPU detection intrinsics

2012-10-04 Thread Diego Biurrun
Now that there is CPU detection in YASM, there will always be one of inline or external assembly enabled, which obviates the need to fall back on CPU detection through compiler intrinsics. --- configure |6 -- libavutil/x86/cpu.c | 39 --- 2

[libav-devel] [PATCH 1/5] x86: ff_get_cpu_flags_x86(): Avoid a pointless variable indirection

2012-10-04 Thread Diego Biurrun
--- This was done for old suncc versions, but these miscompile libav anyway. libavutil/x86/cpu.c |5 + 1 files changed, 1 insertions(+), 4 deletions(-) diff --git a/libavutil/x86/cpu.c b/libavutil/x86/cpu.c index 9acc867..9cf68e4 100644 --- a/libavutil/x86/cpu.c +++

[libav-devel] [PATCH 3/5] x86: Add YASM implementations of cpuid and xgetbv from x264

2012-10-04 Thread Diego Biurrun
This allows detecting CPU features with builds that have neither gcc inline assembly nor the right compiler intrinsics enabled. --- Changed to implement have_cpuid() instead of emulating the macros used by the inline/intrinsics versions. libavutil/x86/Makefile |3 +- libavutil/x86/cpu.c

[libav-devel] [PATCH 5/5] x86: get_cpu_flags: add necessary ifdefs around function body

2012-10-04 Thread Diego Biurrun
ff_get_cpu_flags_x86() requires cpuid(), which is conditionally defined elsewhere in the file. Surrounding the function body with ifdefs allows building even when cpuid is not defined. An empty cpuflags mask is returned in this case. --- libavutil/x86/cpu.c |5 + 1 files changed, 5

Re: [libav-devel] [PATCH 1/5] x86: ff_get_cpu_flags_x86(): Avoid a pointless variable indirection

2012-10-04 Thread Måns Rullgård
Diego Biurrun di...@biurrun.de writes: --- This was done for old suncc versions, but these miscompile libav anyway. libavutil/x86/cpu.c |5 + 1 files changed, 1 insertions(+), 4 deletions(-) diff --git a/libavutil/x86/cpu.c b/libavutil/x86/cpu.c index 9acc867..9cf68e4 100644

Re: [libav-devel] [PATCH 2/5] x86: cpu: Break out test for cpuid capabilities into separate function

2012-10-04 Thread Måns Rullgård
Diego Biurrun di...@biurrun.de writes: --- The #define indirection will be reused by the yasm version of cpuid_test, so I'm adding it here already to avoid code churn. libavutil/x86/cpu.c | 27 ++- 1 files changed, 18 insertions(+), 9 deletions(-) diff --git

Re: [libav-devel] [PATCH 4/5] x86: Drop CPU detection intrinsics

2012-10-04 Thread Måns Rullgård
Diego Biurrun di...@biurrun.de writes: Now that there is CPU detection in YASM, there will always be one of inline or external assembly enabled, which obviates the need to fall back on CPU detection through compiler intrinsics. --- configure |6 -- libavutil/x86/cpu.c |

Re: [libav-devel] [PATCH 5/5] x86: get_cpu_flags: add necessary ifdefs around function body

2012-10-04 Thread Måns Rullgård
Diego Biurrun di...@biurrun.de writes: ff_get_cpu_flags_x86() requires cpuid(), which is conditionally defined elsewhere in the file. Surrounding the function body with ifdefs allows building even when cpuid is not defined. An empty cpuflags mask is returned in this case. ---

Re: [libav-devel] [PATCH] segment: Add comments about calls that only are relevant for some muxers

2012-10-04 Thread Luca Barbato
Ok ___ libav-devel mailing list libav-devel@libav.org https://lists.libav.org/mailman/listinfo/libav-devel

[libav-devel] [PATCH] x86: get_cpu_flags: add necessary ifdefs around function body

2012-10-04 Thread Diego Biurrun
ff_get_cpu_flags_x86() requires cpuid(), which is conditionally defined elsewhere in the file. Surrounding the function body with ifdefs allows building even when cpuid is not defined. An empty cpuflags mask is returned in this case. --- libavutil/x86/cpu.c |5 + 1 files changed, 5

Re: [libav-devel] [PATCH] x86: get_cpu_flags: add necessary ifdefs around function body

2012-10-04 Thread Måns Rullgård
Diego Biurrun di...@biurrun.de writes: ff_get_cpu_flags_x86() requires cpuid(), which is conditionally defined elsewhere in the file. Surrounding the function body with ifdefs allows building even when cpuid is not defined. An empty cpuflags mask is returned in this case. ---

Re: [libav-devel] [PATCH] pthread: introduce default lock manager

2012-10-04 Thread Jindřich Makovička
On Sun, Sep 2, 2012 at 7:03 PM, Luca Barbato lu_z...@gentoo.org wrote: On systems supported have the default lock manager enabled by default. --- Now with free and _cp - _cb typo fixed. Yet to be discussed if/how it could be of use. libavcodec/pthread.c | 27 +++

Re: [libav-devel] [PATCH 2/5] x86: cpu: Break out test for cpuid capabilities into separate function

2012-10-04 Thread Diego Biurrun
On Thu, Oct 04, 2012 at 02:04:27PM +0100, Måns Rullgård wrote: Diego Biurrun di...@biurrun.de writes: --- a/libavutil/x86/cpu.c +++ b/libavutil/x86/cpu.c @@ -96,15 +113,7 @@ int ff_get_cpu_flags_x86(void) #if ARCH_X86_32 -x86_reg a, c; - -/* Check if CPUID is supported by

Re: [libav-devel] [PATCH 2/5] x86: cpu: Break out test for cpuid capabilities into separate function

2012-10-04 Thread Måns Rullgård
Diego Biurrun di...@biurrun.de writes: On Thu, Oct 04, 2012 at 02:04:27PM +0100, Måns Rullgård wrote: Diego Biurrun di...@biurrun.de writes: --- a/libavutil/x86/cpu.c +++ b/libavutil/x86/cpu.c @@ -96,15 +113,7 @@ int ff_get_cpu_flags_x86(void) #if ARCH_X86_32 -x86_reg a, c; -

Re: [libav-devel] [PATCH 2/5] x86: cpu: Break out test for cpuid capabilities into separate function

2012-10-04 Thread Diego Biurrun
On Thu, Oct 04, 2012 at 03:30:17PM +0100, Måns Rullgård wrote: Diego Biurrun di...@biurrun.de writes: On Thu, Oct 04, 2012 at 02:04:27PM +0100, Måns Rullgård wrote: Diego Biurrun di...@biurrun.de writes: --- a/libavutil/x86/cpu.c +++ b/libavutil/x86/cpu.c @@ -96,15 +113,7 @@ int

Re: [libav-devel] [PATCH 2/5] x86: cpu: Break out test for cpuid capabilities into separate function

2012-10-04 Thread Måns Rullgård
Diego Biurrun di...@biurrun.de writes: On Thu, Oct 04, 2012 at 03:30:17PM +0100, Måns Rullgård wrote: Diego Biurrun di...@biurrun.de writes: On Thu, Oct 04, 2012 at 02:04:27PM +0100, Måns Rullgård wrote: Diego Biurrun di...@biurrun.de writes: --- a/libavutil/x86/cpu.c +++

Re: [libav-devel] [PATCH 2/5] x86: cpu: Break out test for cpuid capabilities into separate function

2012-10-04 Thread Diego Biurrun
On Thu, Oct 04, 2012 at 03:36:40PM +0100, Måns Rullgård wrote: Diego Biurrun di...@biurrun.de writes: On Thu, Oct 04, 2012 at 03:30:17PM +0100, Måns Rullgård wrote: Diego Biurrun di...@biurrun.de writes: On Thu, Oct 04, 2012 at 02:04:27PM +0100, Måns Rullgård wrote: Diego Biurrun

Re: [libav-devel] [PATCH 2/5] x86: cpu: Break out test for cpuid capabilities into separate function

2012-10-04 Thread Måns Rullgård
Diego Biurrun di...@biurrun.de writes: On Thu, Oct 04, 2012 at 03:36:40PM +0100, Måns Rullgård wrote: Diego Biurrun di...@biurrun.de writes: On Thu, Oct 04, 2012 at 03:30:17PM +0100, Måns Rullgård wrote: Diego Biurrun di...@biurrun.de writes: On Thu, Oct 04, 2012 at 02:04:27PM +0100,

[libav-devel] [PATCH] x86: get_cpu_flags: add necessary ifdefs around function body

2012-10-04 Thread Diego Biurrun
ff_get_cpu_flags_x86() requires cpuid(), which is conditionally defined elsewhere in the file. Surrounding the function body with ifdefs allows building even when cpuid is not defined. An empty cpuflags mask is returned in this case. --- libavutil/x86/cpu.c |5 + 1 files changed, 5

Re: [libav-devel] [PATCH] x86: get_cpu_flags: add necessary ifdefs around function body

2012-10-04 Thread Måns Rullgård
Diego Biurrun di...@biurrun.de writes: ff_get_cpu_flags_x86() requires cpuid(), which is conditionally defined elsewhere in the file. Surrounding the function body with ifdefs allows building even when cpuid is not defined. An empty cpuflags mask is returned in this case. ---

[libav-devel] [PATCH] x86: cpu: Break out test for cpuid capabilities into separate function

2012-10-04 Thread Diego Biurrun
--- libavutil/x86/cpu.c | 34 +++--- 1 files changed, 23 insertions(+), 11 deletions(-) diff --git a/libavutil/x86/cpu.c b/libavutil/x86/cpu.c index 9cf68e4..bf1ed0b 100644 --- a/libavutil/x86/cpu.c +++ b/libavutil/x86/cpu.c @@ -86,16 +86,16 @@ #endif /*

Re: [libav-devel] [PATCH] x86: cpu: Break out test for cpuid capabilities into separate function

2012-10-04 Thread Måns Rullgård
Diego Biurrun di...@biurrun.de writes: --- libavutil/x86/cpu.c | 34 +++--- 1 files changed, 23 insertions(+), 11 deletions(-) diff --git a/libavutil/x86/cpu.c b/libavutil/x86/cpu.c index 9cf68e4..bf1ed0b 100644 --- a/libavutil/x86/cpu.c +++

Re: [libav-devel] [PATCH] build: Factor out mpegaudio dependencies to CONFIG_MPEGAUDIO

2012-10-04 Thread Diego Biurrun
On Wed, Sep 26, 2012 at 07:59:36PM +0200, Diego Biurrun wrote: On Tue, Sep 11, 2012 at 10:30:44AM +0200, Diego Biurrun wrote: On Fri, Sep 07, 2012 at 11:17:38AM +0200, Diego Biurrun wrote: On Fri, Aug 24, 2012 at 07:06:26PM +0200, Diego Biurrun wrote: A new hidden config variable is

Re: [libav-devel] [PATCH 2/2] nut: add support for yuva422p and yuva444p

2012-10-04 Thread Derek Buitenhuis
On 02/10/2012 5:21 AM, Luca Barbato wrote: --- libavcodec/raw.c | 2 ++ libavformat/nut.c | 2 ++ 2 files changed, 4 insertions(+) Looks OK. Matches what's in FFmpeg's nut code too. - Derek ___ libav-devel mailing list libav-devel@libav.org

[libav-devel] [PATCH] x86: get_cpu_flags: add necessary ifdefs around function body

2012-10-04 Thread Diego Biurrun
ff_get_cpu_flags_x86() requires cpuid(), which is conditionally defined elsewhere in the file. Surrounding the function body with ifdefs allows building even when cpuid is not defined. An empty cpuflags mask is returned in this case. --- libavutil/x86/cpu.c |5 + 1 files changed, 5

[libav-devel] [PATCH] x86: cpu: Break out test for cpuid capabilities into separate function

2012-10-04 Thread Diego Biurrun
--- libavutil/x86/cpu.c | 32 +--- 1 files changed, 21 insertions(+), 11 deletions(-) diff --git a/libavutil/x86/cpu.c b/libavutil/x86/cpu.c index 9cf68e4..5de6014 100644 --- a/libavutil/x86/cpu.c +++ b/libavutil/x86/cpu.c @@ -86,16 +86,14 @@ #endif /*

Re: [libav-devel] [PATCH] x86: cpu: Break out test for cpuid capabilities into separate function

2012-10-04 Thread Måns Rullgård
Diego Biurrun di...@biurrun.de writes: --- libavutil/x86/cpu.c | 32 +--- 1 files changed, 21 insertions(+), 11 deletions(-) diff --git a/libavutil/x86/cpu.c b/libavutil/x86/cpu.c index 9cf68e4..5de6014 100644 --- a/libavutil/x86/cpu.c +++

[libav-devel] [PATCH] avplay: Rework variable initialization to avoid gcc warning

2012-10-04 Thread Diego Biurrun
avplay.c:3015:20: warning: assignment discards ‘const’ qualifier from pointer target type --- avplay.c |3 ++- 1 files changed, 2 insertions(+), 1 deletions(-) diff --git a/avplay.c b/avplay.c index 5ba8155..cd8c386 100644 --- a/avplay.c +++ b/avplay.c @@ -2959,6 +2959,7 @@ static void

Re: [libav-devel] [PATCH] avplay: Rework variable initialization to avoid gcc warning

2012-10-04 Thread Måns Rullgård
Diego Biurrun di...@biurrun.de writes: avplay.c:3015:20: warning: assignment discards ‘const’ qualifier from pointer target type --- avplay.c |3 ++- 1 files changed, 2 insertions(+), 1 deletions(-) diff --git a/avplay.c b/avplay.c index 5ba8155..cd8c386 100644 --- a/avplay.c +++

[libav-devel] [PATCH 2/4] ppc: fmtconvert: kill VLA in float_to_int16_interleave_altivec()

2012-10-04 Thread Mans Rullgard
Signed-off-by: Mans Rullgard m...@mansr.com --- libavcodec/ppc/fmtconvert_altivec.c | 25 ++--- 1 file changed, 18 insertions(+), 7 deletions(-) diff --git a/libavcodec/ppc/fmtconvert_altivec.c b/libavcodec/ppc/fmtconvert_altivec.c index db2c258..129891a 100644 ---

[libav-devel] [PATCH 4/4] build: error on variable-length arrays

2012-10-04 Thread Mans Rullgard
With all the VLAs gone, make sure they never come back. Signed-off-by: Mans Rullgard m...@mansr.com --- configure | 1 + 1 file changed, 1 insertion(+) diff --git a/configure b/configure index 760b747..7032f98 100755 --- a/configure +++ b/configure @@ -3522,6 +3522,7 @@ elif enabled gcc; then

[libav-devel] [PATCH 1/4] x86: dsputil: kill VLA in gmc_mmx()

2012-10-04 Thread Mans Rullgard
Instead of using an evil VLA, fall back to C version when edge emulation is needed. MPEG4 GMC is a rarely used fringe feature so the speed loss is an acceptable cost for cleaner code. Signed-off-by: Mans Rullgard m...@mansr.com --- libavcodec/x86/dsputil_mmx.c | 58

[libav-devel] [PATCH 3/4] ppc: swscale: rework yuv2planeX_altivec()

2012-10-04 Thread Mans Rullgard
This gets rid of the variable-length scratch buffer by filtering 16 pixels at a time and writing directly to the destination. The extra loads this requires to load the source values are compensated by not doing a round-trip to memory before shifting. Signed-off-by: Mans Rullgard m...@mansr.com

Re: [libav-devel] [PATCH 4/4] build: error on variable-length arrays

2012-10-04 Thread Diego Biurrun
On Thu, Oct 04, 2012 at 05:43:57PM +0100, Mans Rullgard wrote: With all the VLAs gone, make sure they never come back. Fine with me once the rest of the set is in. Diego ___ libav-devel mailing list libav-devel@libav.org

Re: [libav-devel] [PATCH] avplay: Rework variable initialization to avoid gcc warning

2012-10-04 Thread Diego Biurrun
On Thu, Oct 04, 2012 at 05:36:58PM +0100, Måns Rullgård wrote: Diego Biurrun di...@biurrun.de writes: avplay.c:3015:20: warning: assignment discards ‘const’ qualifier from pointer target type --- a/avplay.c +++ b/avplay.c @@ -2959,6 +2959,7 @@ static void opt_input_file(void *optctx,

Re: [libav-devel] [PATCH] avplay: Rework variable initialization to avoid gcc warning

2012-10-04 Thread Måns Rullgård
Diego Biurrun di...@biurrun.de writes: On Thu, Oct 04, 2012 at 05:36:58PM +0100, Måns Rullgård wrote: Diego Biurrun di...@biurrun.de writes: avplay.c:3015:20: warning: assignment discards ‘const’ qualifier from pointer target type --- a/avplay.c +++ b/avplay.c @@ -2959,6 +2959,7 @@

Re: [libav-devel] [PATCH] avplay: Rework variable initialization to avoid gcc warning

2012-10-04 Thread Diego Biurrun
On Thu, Oct 04, 2012 at 06:03:57PM +0100, Måns Rullgård wrote: Diego Biurrun di...@biurrun.de writes: On Thu, Oct 04, 2012 at 05:36:58PM +0100, Måns Rullgård wrote: Diego Biurrun di...@biurrun.de writes: avplay.c:3015:20: warning: assignment discards ‘const’ qualifier from pointer

Re: [libav-devel] [PATCH 3/5] x86: Add YASM implementations of cpuid and xgetbv from x264

2012-10-04 Thread Måns Rullgård
Diego Biurrun di...@biurrun.de writes: This allows detecting CPU features with builds that have neither gcc inline assembly nor the right compiler intrinsics enabled. --- Changed to implement have_cpuid() instead of emulating the macros used by the inline/intrinsics versions.

Re: [libav-devel] [PATCH] avplay: Rework variable initialization to avoid gcc warning

2012-10-04 Thread Måns Rullgård
Diego Biurrun di...@biurrun.de writes: On Thu, Oct 04, 2012 at 06:03:57PM +0100, Måns Rullgård wrote: Diego Biurrun di...@biurrun.de writes: On Thu, Oct 04, 2012 at 05:36:58PM +0100, Måns Rullgård wrote: Diego Biurrun di...@biurrun.de writes: avplay.c:3015:20: warning: assignment

Re: [libav-devel] [PATCH 3/4] ppc: swscale: rework yuv2planeX_altivec()

2012-10-04 Thread Derek Buitenhuis
On 04/10/2012 12:43 PM, Mans Rullgard wrote: This gets rid of the variable-length scratch buffer by filtering 16 pixels at a time and writing directly to the destination. The extra loads this requires to load the source values are compensated by not doing a round-trip to memory before

[libav-devel] [PATCH 1/4] Move avutil tables only used in libavcodec to libavcodec.

2012-10-04 Thread Diego Biurrun
--- libavcodec/Makefile |7 + libavcodec/arm/mathops.h| 24 ++ libavcodec/inverse.c|1 - libavutil/inverse.c = libavcodec/mathops.c | 14 -- libavcodec/mathops.h|

[libav-devel] [PATCH 3/4] avutil: Do not make ff_ symbols globally visible.

2012-10-04 Thread Diego Biurrun
--- libavutil/libavutil.v |2 +- 1 files changed, 1 insertions(+), 1 deletions(-) diff --git a/libavutil/libavutil.v b/libavutil/libavutil.v index eb16ae1..e9f04cb 100644 --- a/libavutil/libavutil.v +++ b/libavutil/libavutil.v @@ -1,4 +1,4 @@ LIBAVUTIL_$MAJOR { -global: av*; ff_*; +

[libav-devel] [PATCH 4/4] swscale: Do not make ff_ symbols globally visible.

2012-10-04 Thread Diego Biurrun
--- libswscale/libswscale.v |2 +- 1 files changed, 1 insertions(+), 1 deletions(-) diff --git a/libswscale/libswscale.v b/libswscale/libswscale.v index c68f153..8b9a96a 100644 --- a/libswscale/libswscale.v +++ b/libswscale/libswscale.v @@ -1,4 +1,4 @@ LIBSWSCALE_$MAJOR { -global:

[libav-devel] [PATCH 2/4] avutil: Give symbols used across libraries avpriv_ instead of ff_ prefixes

2012-10-04 Thread Diego Biurrun
--- libavcodec/aacps.c|2 +- libavcodec/bmpenc.c |2 +- libavcodec/rawdec.c |2 +- libavcodec/tiertexseqv.c |2 +- libavcodec/utils.c|2 +- libavcodec/xsubdec.c |2 +- libavcodec/xsubenc.c |2 +- libavformat/matroskadec.c |4

Re: [libav-devel] [PATCH 1/4] x86: dsputil: kill VLA in gmc_mmx()

2012-10-04 Thread Derek Buitenhuis
On 04/10/2012 12:43 PM, Mans Rullgard wrote: Instead of using an evil VLA, fall back to C version when edge emulation is needed. MPEG4 GMC is a rarely used fringe feature so the speed loss is an acceptable cost for cleaner code. While, IIRC, 'Scene' rules ban GMC usage, I've seen plenty of

Re: [libav-devel] [PATCH 3/4] ppc: swscale: rework yuv2planeX_altivec()

2012-10-04 Thread Måns Rullgård
Derek Buitenhuis derek.buitenh...@gmail.com writes: On 04/10/2012 12:43 PM, Mans Rullgard wrote: This gets rid of the variable-length scratch buffer by filtering 16 pixels at a time and writing directly to the destination. The extra loads this requires to load the source values are

Re: [libav-devel] [PATCH 1/4] Move avutil tables only used in libavcodec to libavcodec.

2012-10-04 Thread Måns Rullgård
Diego Biurrun di...@biurrun.de writes: --- libavcodec/Makefile |7 + libavcodec/arm/mathops.h| 24 ++ libavcodec/inverse.c|1 - libavutil/inverse.c = libavcodec/mathops.c | 14 --

Re: [libav-devel] [PATCH 4/4] swscale: Do not make ff_ symbols globally visible.

2012-10-04 Thread Måns Rullgård
Diego Biurrun di...@biurrun.de writes: --- libswscale/libswscale.v |2 +- 1 files changed, 1 insertions(+), 1 deletions(-) diff --git a/libswscale/libswscale.v b/libswscale/libswscale.v index c68f153..8b9a96a 100644 --- a/libswscale/libswscale.v +++ b/libswscale/libswscale.v @@ -1,4

Re: [libav-devel] [PATCH 4/4] swscale: Do not make ff_ symbols globally visible.

2012-10-04 Thread Derek Buitenhuis
On 04/10/2012 1:44 PM, Måns Rullgård wrote: Why are they visible now? The file was never touched after its creation... by you. Perhaps you should ask yourself :D - Derek ___ libav-devel mailing list libav-devel@libav.org

Re: [libav-devel] [PATCH 4/4] swscale: Do not make ff_ symbols globally visible.

2012-10-04 Thread Måns Rullgård
Derek Buitenhuis derek.buitenh...@gmail.com writes: On 04/10/2012 1:44 PM, Måns Rullgård wrote: Why are they visible now? The file was never touched after its creation... by you. I may have committed the file, but I did not create it. Now stop being snide. -- Måns Rullgård m...@mansr.com

Re: [libav-devel] [PATCH 4/4] swscale: Do not make ff_ symbols globally visible.

2012-10-04 Thread Diego Biurrun
On Thu, Oct 04, 2012 at 06:44:06PM +0100, Måns Rullgård wrote: Diego Biurrun di...@biurrun.de writes: --- a/libswscale/libswscale.v +++ b/libswscale/libswscale.v @@ -1,4 +1,4 @@ LIBSWSCALE_$MAJOR { -global: swscale_*; sws_*; ff_*; +global: swscale_*; sws_*;

Re: [libav-devel] [PATCH 1/4] Move avutil tables only used in libavcodec to libavcodec.

2012-10-04 Thread Diego Biurrun
On Thu, Oct 04, 2012 at 06:43:16PM +0100, Måns Rullgård wrote: Diego Biurrun di...@biurrun.de writes: --- libavcodec/Makefile |7 + libavcodec/arm/mathops.h| 24 ++ libavcodec/inverse.c|1

[libav-devel] [PATCH] movenc: calculate the average bitrate

2012-10-04 Thread Luca Barbato
From: Baptiste Coudurier baptiste.coudur...@gmail.com Setting the average bitrate to 0 in esds makes the produced files not compatible with certain players. According to 14496-1:2010 7.2.6.6 DecoderConfigDescriptor: avgBitrate – is the average bitrate in bits per second of this elementary

Re: [libav-devel] [PATCH] avplay: Rework variable initialization to avoid gcc warning

2012-10-04 Thread Luca Barbato
On 10/04/2012 07:14 PM, Måns Rullgård wrote: Diego Biurrun di...@biurrun.de writes: On Thu, Oct 04, 2012 at 06:03:57PM +0100, Måns Rullgård wrote: Diego Biurrun di...@biurrun.de writes: On Thu, Oct 04, 2012 at 05:36:58PM +0100, Måns Rullgård wrote: Diego Biurrun di...@biurrun.de writes:

Re: [libav-devel] [PATCH 2/4] ppc: fmtconvert: kill VLA in float_to_int16_interleave_altivec()

2012-10-04 Thread Luca Barbato
On 10/04/2012 06:43 PM, Mans Rullgard wrote: Signed-off-by: Mans Rullgard m...@mansr.com --- libavcodec/ppc/fmtconvert_altivec.c | 25 ++--- 1 file changed, 18 insertions(+), 7 deletions(-) Ok. ___ libav-devel mailing list

Re: [libav-devel] [PATCH 3/4] ppc: swscale: rework yuv2planeX_altivec()

2012-10-04 Thread Luca Barbato
On 10/04/2012 07:38 PM, Måns Rullgård wrote: Derek Buitenhuis derek.buitenh...@gmail.com writes: On 04/10/2012 12:43 PM, Mans Rullgard wrote: This gets rid of the variable-length scratch buffer by filtering 16 pixels at a time and writing directly to the destination. The extra loads this

Re: [libav-devel] [PATCH] movenc: calculate the average bitrate

2012-10-04 Thread Måns Rullgård
Luca Barbato lu_z...@gentoo.org writes: From: Baptiste Coudurier baptiste.coudur...@gmail.com Setting the average bitrate to 0 in esds makes the produced files not compatible with certain players. According to 14496-1:2010 7.2.6.6 DecoderConfigDescriptor: avgBitrate – is the average

[libav-devel] [PATCH v2] ppc: swscale: rework yuv2planeX_altivec()

2012-10-04 Thread Mans Rullgard
This gets rid of the variable-length scratch buffer by filtering 16 pixels at a time and writing directly to the destination. The extra loads this requires to load the source values are compensated by not doing a round-trip to memory before shifting. Signed-off-by: Mans Rullgard m...@mansr.com

Re: [libav-devel] [PATCH] movenc: calculate the average bitrate

2012-10-04 Thread Luca Barbato
On 10/04/2012 08:52 PM, Måns Rullgård wrote: Luca Barbato lu_z...@gentoo.org writes: From: Baptiste Coudurier baptiste.coudur...@gmail.com Setting the average bitrate to 0 in esds makes the produced files not compatible with certain players. According to 14496-1:2010 7.2.6.6

[libav-devel] [PATCH 2/2] avutil: Give symbols used across libraries avpriv_ instead of ff_ prefixes

2012-10-04 Thread Diego Biurrun
--- libavcodec/bmpenc.c |2 +- libavcodec/rawdec.c |2 +- libavcodec/utils.c |2 +- libavutil/imgutils.c |4 ++-- libavutil/imgutils.h |2 +- 5 files changed, 6 insertions(+), 6 deletions(-) diff --git a/libavcodec/bmpenc.c b/libavcodec/bmpenc.c index 37cf6e1..791d846

[libav-devel] [PATCH 1/2] avutil: Duplicate ff_log2_tab instead of sharing it across libs

2012-10-04 Thread Diego Biurrun
The table is so small that the space gain is not worth the performance overhead of cross-library access. --- libavcodec/aacps.c |1 + libavcodec/mathops.c|2 ++ libavcodec/mathops.h|1 + libavutil/Makefile |1 + libavutil/log2_tab.c| 30

Re: [libav-devel] [PATCH] h264: don't touch H264Context-ref_count[] during MB decoding.

2012-10-04 Thread Luca Barbato
On 10/04/2012 01:46 AM, Luca Barbato wrote: From: Ronald S. Bultje rsbul...@gmail.com The variable is copied to subsequent threads at the same time, so this may cause wrong ref_count[] values to be copied to subsequent threads. This bug was found using TSAN. Signed-off-by: Luca Barbato

[libav-devel] [PATCH] avutil: Duplicate ff_log2_tab instead of sharing it across libs

2012-10-04 Thread Diego Biurrun
The table is so small that the space gain is not worth the performance overhead of cross-library access. --- Now with each duplicated table in a separate file. libavcodec/Makefile |1 + libavcodec/log2_tab.c |1 + libavutil/Makefile |1 + libavutil/log2_tab.c| 30

Re: [libav-devel] [PATCH 5/5] fate.sh: add support for build-only FATE instances

2012-10-04 Thread Martin Storsjö
On Tue, 20 Sep 2011, Måns Rullgård wrote: Diego Biurrun di...@biurrun.de writes: If the build_only variable is set in the configuration file, the FATE client will skip running tests and just compile all targets. --- tests/fate.sh |1 + 1 files changed, 1 insertions(+), 0 deletions(-)

[libav-devel] [PATCH] Rename ff_set_systematic_pal2 - av_image_get_palette and make it public

2012-10-04 Thread Diego Biurrun
--- Now making the function public instead of marking it avpriv_. libavcodec/bmpenc.c |2 +- libavcodec/rawdec.c |2 +- libavcodec/utils.c |2 +- libavutil/imgutils.c |4 ++-- libavutil/imgutils.h |2 +- 5 files changed, 6 insertions(+), 6 deletions(-) diff --git

Re: [libav-devel] [PATCH] Rename ff_set_systematic_pal2 - av_image_get_palette and make it public

2012-10-04 Thread Måns Rullgård
Diego Biurrun di...@biurrun.de writes: --- Now making the function public instead of marking it avpriv_. I'm not at all convinced this function needs to be exposed at all. It probably shouldn't even exist. -- Måns Rullgård m...@mansr.com ___

Re: [libav-devel] [PATCH] configure: x86: improve ebp availability check

2012-10-04 Thread Diego Biurrun
On Thu, Sep 13, 2012 at 08:18:56PM +0100, Mans Rullgard wrote: Some compilers are extra strict about register usage in main(), disallowing ebp in inline asm there while allowing it elsewhere. This change makes the test better reflect actual usage. --- a/configure +++ b/configure @@ -865,6

Re: [libav-devel] [PATCH] configure: x86: improve ebp availability check

2012-10-04 Thread Måns Rullgård
Diego Biurrun di...@biurrun.de writes: On Thu, Sep 13, 2012 at 08:18:56PM +0100, Mans Rullgard wrote: Some compilers are extra strict about register usage in main(), disallowing ebp in inline asm there while allowing it elsewhere. This change makes the test better reflect actual usage. ---

Re: [libav-devel] [PATCH 1/4] x86: dsputil: kill VLA in gmc_mmx()

2012-10-04 Thread Måns Rullgård
Derek Buitenhuis derek.buitenh...@gmail.com writes: On 04/10/2012 12:43 PM, Mans Rullgard wrote: Instead of using an evil VLA, fall back to C version when edge emulation is needed. MPEG4 GMC is a rarely used fringe feature so the speed loss is an acceptable cost for cleaner code. While,

Re: [libav-devel] [PATCH 1/4] x86: dsputil: kill VLA in gmc_mmx()

2012-10-04 Thread Derek Buitenhuis
On 04/10/2012 5:42 PM, Måns Rullgård wrote: The gmc function itself is only mmx. The old code has multiple wrappers using mmx or sse versions of the edge emulation function. Since the patch changes it to fall back to C code when edge emulation is needed, these wrappers are gone, leaving only

[libav-devel] [PATCH] Fix ff_log2_run[] declaration

2012-10-04 Thread Mans Rullgard
This updates the declaration to match the actual size of the table. --- This is a troll-patch in the hope that someone will suggest a better home for that declaration. --- libavcodec/jpegls.h | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/libavcodec/jpegls.h

Re: [libav-devel] [PATCH] h264: don't touch H264Context-ref_count[] during MB decoding.

2012-10-04 Thread Ronald S. Bultje
Hi, On Thu, Oct 4, 2012 at 12:59 PM, Luca Barbato lu_z...@gentoo.org wrote: On 10/04/2012 01:46 AM, Luca Barbato wrote: From: Ronald S. Bultje rsbul...@gmail.com The variable is copied to subsequent threads at the same time, so this may cause wrong ref_count[] values to be copied to

Re: [libav-devel] [PATCH] h264: don't touch H264Context-ref_count[] during MB decoding.

2012-10-04 Thread Ronald S. Bultje
On Thu, Oct 4, 2012 at 4:27 PM, Ronald S. Bultje rsbul...@gmail.com wrote: Hi, On Thu, Oct 4, 2012 at 12:59 PM, Luca Barbato lu_z...@gentoo.org wrote: On 10/04/2012 01:46 AM, Luca Barbato wrote: From: Ronald S. Bultje rsbul...@gmail.com The variable is copied to subsequent threads at the

Re: [libav-devel] [PATCH] h264: don't touch H264Context-ref_count[] during MB decoding.

2012-10-04 Thread Luca Barbato
On 10/05/2012 01:29 AM, Ronald S. Bultje wrote: On Thu, Oct 4, 2012 at 4:27 PM, Ronald S. Bultje rsbul...@gmail.com wrote: Oh right you mean for performance? fate-h264-conformance-cama2_vtc_b fate-h264-conformance-cama3_vtc_b should be ok. I'd push your patch + my style nits soon, I'll

[libav-devel] AVFMT_GLOBALHEADER and chained muxers

2012-10-04 Thread Luca Barbato
Segment set AVFMT_GLOBALHEADER since it is assumed that global headers might be used, it interacts badly with x264 assumption on that flag and mpegts. The two patches aim to partially workaround another option is adding a AVFMT_ANYHEADER or making the flag not part of the immutable format

[libav-devel] [PATCH 1/2] x264: expose repeat-headers

2012-10-04 Thread Luca Barbato
Currently this option is bound to the container format requesting global headers or not, in a number of situation global headers and and repeated headers are not mutually exclusive. --- libavcodec/libx264.c | 5 + 1 file changed, 5 insertions(+) diff --git a/libavcodec/libx264.c

[libav-devel] [PATCH 2/2] segment: drop global headers setting

2012-10-04 Thread Luca Barbato
avconv provides an option to set it externally and there isn't a way for the outer demuxer to report the inner demuxer flag. Solves a bad interaction between mpegts and x264, but requires additional setting for the user. --- libavformat/segment.c | 2 +- 1 file changed, 1 insertion(+), 1

[libav-devel] [PATCH 1/5] lavc: deprecate the audio resampling API.

2012-10-04 Thread Anton Khirnov
It has been superseded by lavr. --- libavcodec/avcodec.h | 10 ++ libavcodec/resample.c |4 libavcodec/resample2.c |4 libavcodec/version.h |3 +++ 4 files changed, 21 insertions(+) diff --git a/libavcodec/avcodec.h b/libavcodec/avcodec.h index

[libav-devel] [PATCH 2/5] lavr: change the type of the data buffers to uint8_t**.

2012-10-04 Thread Anton Khirnov
This is more consistent with what the rest of Libav does. This breaks API. --- avplay.c|4 ++-- libavfilter/af_asyncts.c|8 libavfilter/af_resample.c |6 +++--- libavresample/audio_data.c |2 +- libavresample/audio_data.h |

[libav-devel] [PATCH 5/5] Update the Changelog and RELEASE for 9 beta1.

2012-10-04 Thread Anton Khirnov
--- Changelog |2 ++ RELEASE |2 +- 2 files changed, 3 insertions(+), 1 deletion(-) diff --git a/Changelog b/Changelog index cc5eaeb..935cd49 100644 --- a/Changelog +++ b/Changelog @@ -3,6 +3,8 @@ releases are sorted from youngest to oldest. version next: +version 9_beta1: + -

[libav-devel] [PATCH 4/5] doc/RELEASE_NOTES: update for the 9 release.

2012-10-04 Thread Anton Khirnov
--- doc/RELEASE_NOTES | 109 ++--- 1 file changed, 61 insertions(+), 48 deletions(-) diff --git a/doc/RELEASE_NOTES b/doc/RELEASE_NOTES index 8c642b9..6acb230 100644 --- a/doc/RELEASE_NOTES +++ b/doc/RELEASE_NOTES @@ -1,29 +1,53 @@ Release Notes

Re: [libav-devel] [PATCH 2/5] lavr: change the type of the data buffers to uint8_t**.

2012-10-04 Thread Justin Ruggles
On 10/05/2012 01:27 AM, Anton Khirnov wrote: This is more consistent with what the rest of Libav does. This breaks API. --- avplay.c|4 ++-- libavfilter/af_asyncts.c|8 libavfilter/af_resample.c |6 +++---

Re: [libav-devel] [PATCH 3/5] lavr: bump major to 1 and declare it stable.

2012-10-04 Thread Justin Ruggles
On 10/05/2012 01:27 AM, Anton Khirnov wrote: --- doc/APIchanges |5 + libavresample/version.h |4 ++-- 2 files changed, 7 insertions(+), 2 deletions(-) diff --git a/doc/APIchanges b/doc/APIchanges index 4558ca7..314622d 100644 --- a/doc/APIchanges +++

Re: [libav-devel] [PATCH] FATE: remove the lavf-ffm test

2012-10-04 Thread Anton Khirnov
On Mon, 1 Oct 2012 01:42:24 -0400, Justin Ruggles justin.rugg...@gmail.com wrote: --- alternative patch tests/lavf-regression.sh |4 tests/ref/lavf/ffm |3 --- 2 files changed, 0 insertions(+), 7 deletions(-) delete mode 100644 tests/ref/lavf/ffm diff --git