Re: [libav-devel] [PATCH 41/45] x86: Rename SPLATD macro to SPLATD_OFFSET

2012-11-16 Thread Diego Biurrun
On Wed, Aug 01, 2012 at 12:18:05AM +0200, Diego Biurrun wrote: This will allow porting the optimized versions to cpuflags. --- libavutil/x86/x86util.asm |2 +- libswscale/x86/output.asm |2 +- 2 files changed, 2 insertions(+), 2 deletions(-) ping - trivial and blocks the porting of

Re: [libav-devel] [PATCH 2/2] configure: Add separate list for libraries and use where appropriate

2012-11-16 Thread Diego Biurrun
On Sun, Nov 11, 2012 at 02:59:06PM +0100, Diego Biurrun wrote: --- configure | 33 + 1 files changed, 13 insertions(+), 20 deletions(-) ping Diego ___ libav-devel mailing list libav-devel@libav.org

Re: [libav-devel] [PATCH] configure: Rename --disable-everything -- --disable-components

2012-11-16 Thread Diego Biurrun
On Wed, Nov 14, 2012 at 02:56:05PM +0100, Luca Barbato wrote: On 11/14/2012 02:38 PM, Diego Biurrun wrote: Now the difference between all and everything seems arbitrary. Why don't we just set clear semantics on what we call parts and components or components and subcomponents? From an

Re: [libav-devel] [PATCH 2/2] configure: Add separate list for libraries and use where appropriate

2012-11-16 Thread Måns Rullgård
Diego Biurrun di...@biurrun.de writes: --- configure | 33 + 1 files changed, 13 insertions(+), 20 deletions(-) diff --git a/configure b/configure index 2894d6f..57a3891 100755 --- a/configure +++ b/configure @@ -980,6 +980,15 @@ COMPONENT_LIST=

Re: [libav-devel] [PATCH] configure: Rename --disable-everything -- --disable-components

2012-11-16 Thread Måns Rullgård
Diego Biurrun di...@biurrun.de writes: On Wed, Nov 14, 2012 at 02:56:05PM +0100, Luca Barbato wrote: On 11/14/2012 02:38 PM, Diego Biurrun wrote: Now the difference between all and everything seems arbitrary. Why don't we just set clear semantics on what we call parts and components or

Re: [libav-devel] [PATCH] configure: Rename --disable-everything -- --disable-components

2012-11-16 Thread Diego Biurrun
On Fri, Nov 16, 2012 at 10:06:16AM +, Måns Rullgård wrote: Diego Biurrun di...@biurrun.de writes: On Wed, Nov 14, 2012 at 02:56:05PM +0100, Luca Barbato wrote: On 11/14/2012 02:38 PM, Diego Biurrun wrote: Now the difference between all and everything seems arbitrary. Why don't we

Re: [libav-devel] [PATCH 2/2] configure: Add separate list for libraries and use where appropriate

2012-11-16 Thread Diego Biurrun
On Fri, Nov 16, 2012 at 10:05:13AM +, Måns Rullgård wrote: Diego Biurrun di...@biurrun.de writes: --- a/configure +++ b/configure @@ -980,6 +980,15 @@ COMPONENT_LIST= +LIBRARY_LIST= +avcodec +avdevice +avfilter +avformat +avresample +swscale +

Re: [libav-devel] [PATCH] configure: Rename --disable-everything -- --disable-components

2012-11-16 Thread Måns Rullgård
Diego Biurrun di...@biurrun.de writes: On Fri, Nov 16, 2012 at 10:06:16AM +, Måns Rullgård wrote: Diego Biurrun di...@biurrun.de writes: On Wed, Nov 14, 2012 at 02:56:05PM +0100, Luca Barbato wrote: On 11/14/2012 02:38 PM, Diego Biurrun wrote: Now the difference between all and

Re: [libav-devel] [PATCH 2/2] configure: Add separate list for libraries and use where appropriate

2012-11-16 Thread Måns Rullgård
Diego Biurrun di...@biurrun.de writes: On Fri, Nov 16, 2012 at 10:05:13AM +, Måns Rullgård wrote: Diego Biurrun di...@biurrun.de writes: --- a/configure +++ b/configure @@ -980,6 +980,15 @@ COMPONENT_LIST= +LIBRARY_LIST= +avcodec +avdevice +avfilter +

Re: [libav-devel] [PATCH] configure: Rename --disable-everything -- --disable-components

2012-11-16 Thread Luca Barbato
On 11/16/2012 10:14 AM, Diego Biurrun wrote: On Wed, Nov 14, 2012 at 02:56:05PM +0100, Luca Barbato wrote: On 11/14/2012 02:38 PM, Diego Biurrun wrote: Now the difference between all and everything seems arbitrary. Why don't we just set clear semantics on what we call parts and components or

Re: [libav-devel] [PATCH 2/2] configure: Add separate list for libraries and use where appropriate

2012-11-16 Thread Diego Biurrun
On Fri, Nov 16, 2012 at 10:34:37AM +, Måns Rullgård wrote: Diego Biurrun di...@biurrun.de writes: On Fri, Nov 16, 2012 at 10:05:13AM +, Måns Rullgård wrote: Diego Biurrun di...@biurrun.de writes: --- a/configure +++ b/configure @@ -980,6 +980,15 @@ COMPONENT_LIST=

Re: [libav-devel] [PATCH] configure: Rename --disable-everything -- --disable-components

2012-11-16 Thread Diego Biurrun
On Fri, Nov 16, 2012 at 10:33:38AM +, Måns Rullgård wrote: Diego Biurrun di...@biurrun.de writes: On Fri, Nov 16, 2012 at 10:06:16AM +, Måns Rullgård wrote: Diego Biurrun di...@biurrun.de writes: On Wed, Nov 14, 2012 at 02:56:05PM +0100, Luca Barbato wrote: On 11/14/2012 02:38

Re: [libav-devel] [PATCH 1/2] x86: PMINSD/PMAXSD: port to cpuflags

2012-11-16 Thread Diego Biurrun
On Wed, Nov 14, 2012 at 05:28:53PM -0500, Justin Ruggles wrote: On 11/14/2012 04:16 PM, Diego Biurrun wrote: --- a/libavutil/x86/x86util.asm +++ b/libavutil/x86/x86util.asm @@ -599,25 +599,37 @@ -%macro PMINSD_MMX 3 ; dst, src, tmp +%macro PMINSD 3 ; dst, src, tmp/unused +%if

Re: [libav-devel] [PATCH] configure: Rename --disable-everything -- --disable-components

2012-11-16 Thread Måns Rullgård
Diego Biurrun di...@biurrun.de writes: On Fri, Nov 16, 2012 at 10:33:38AM +, Måns Rullgård wrote: Diego Biurrun di...@biurrun.de writes: On Fri, Nov 16, 2012 at 10:06:16AM +, Måns Rullgård wrote: Diego Biurrun di...@biurrun.de writes: On Wed, Nov 14, 2012 at 02:56:05PM +0100,

[libav-devel] [PATCH 1/1] cpu: define AV_CPU_FLAG_MMX2 for libavutil major 52

2012-11-16 Thread Janne Grunau
--- libavutil/cpu.h | 2 +- libavutil/version.h | 3 +++ 2 files changed, 4 insertions(+), 1 deletion(-) diff --git a/libavutil/cpu.h b/libavutil/cpu.h index 01f7201..4929512 100644 --- a/libavutil/cpu.h +++ b/libavutil/cpu.h @@ -28,7 +28,7 @@ /* lower 16 bits - CPU features */ #define

Re: [libav-devel] [PATCH 1/1] cpu: define AV_CPU_FLAG_MMX2 for libavutil major 52

2012-11-16 Thread Luca Barbato
On 11/16/2012 02:56 PM, Janne Grunau wrote: --- libavutil/cpu.h | 2 +- libavutil/version.h | 3 +++ 2 files changed, 4 insertions(+), 1 deletion(-) Thanks, patch ok and unbreaks vlc-2.0.4 and other. lu ___ libav-devel mailing list

[libav-devel] [PATCH 1/1] libswscale: remove unnecessary direct #if LIBSWSCALE_VERSION_MAJOR

2012-11-16 Thread Janne Grunau
SWS_CPU_CAPS are deprecated and slated to removed with libswscale major version 3. No need to provide a SWS_CPU_CAPS_MMX2 as backward compatibility define under the same explicit condition. --- libswscale/swscale.h | 2 -- 1 file changed, 2 deletions(-) diff --git a/libswscale/swscale.h

Re: [libav-devel] [PATCH 1/1] libswscale: remove unnecessary direct #if LIBSWSCALE_VERSION_MAJOR

2012-11-16 Thread Luca Barbato
On 11/16/2012 03:04 PM, Janne Grunau wrote: SWS_CPU_CAPS are deprecated and slated to removed with libswscale major version 3. No need to provide a SWS_CPU_CAPS_MMX2 as backward compatibility define under the same explicit condition. --- libswscale/swscale.h | 2 -- 1 file changed, 2

Re: [libav-devel] [PATCH] configure: Rename --disable-everything -- --disable-components

2012-11-16 Thread Diego Biurrun
On Fri, Nov 16, 2012 at 11:55:34AM +, Måns Rullgård wrote: Diego Biurrun di...@biurrun.de writes: On Fri, Nov 16, 2012 at 10:33:38AM +, Måns Rullgård wrote: Diego Biurrun di...@biurrun.de writes: On Fri, Nov 16, 2012 at 10:06:16AM +, Måns Rullgård wrote: Diego Biurrun

Re: [libav-devel] [PATCH] configure: Rename --disable-everything -- --disable-components

2012-11-16 Thread Luca Barbato
On 11/16/2012 04:19 PM, Diego Biurrun wrote: It does not disable everything, contrary to what the name suggests. As long it is documented (and it is) I see no problem. The result of this discussion ended with an useful option (--disable-programs) but I don't see why we should break people

[libav-devel] [RFC PATCH 1/2] h264: set parameters from SPS whenever it changes

2012-11-16 Thread Janne Grunau
I'm not entirely convinced that is the the correct/best way to fix the problem with this sample. It crashes in draw_edges_10_c because the image format and parameters has just 8 bit per pixels. The main problem I see is that it does far too many context re-inits. Janne ---8--- Fixes a crash in

[libav-devel] [PATCH 2/2] h264: reset has_b_frames after enabling low_delay from SPS

2012-11-16 Thread Janne Grunau
Fixes a crash in fuzzed file nasa-8s2.ts_s20033 caused by a too large has_b_frames value. low_delay keeps getting re-enabled from the the presumely broken SPS. --- libavcodec/h264.c | 4 +++- 1 file changed, 3 insertions(+), 1 deletion(-) diff --git a/libavcodec/h264.c b/libavcodec/h264.c index

Re: [libav-devel] [PATCH] PGS subtitles: Set AVSubtitle pts value

2012-11-16 Thread Anton Khirnov
On Fri, 2 Nov 2012 09:32:51 -0700, John Stebbins stebb...@jetheaddev.com wrote: On 11/02/2012 08:20 AM, Diego Biurrun wrote: On Sun, Oct 14, 2012 at 04:52:12PM +0200, John Stebbins wrote: pts should be that of the packet containing the presentation segment. --- libavcodec/pgssubdec.c |

Re: [libav-devel] [PATCH 1/2] x86: PMINSD/PMAXSD: port to cpuflags

2012-11-16 Thread Justin Ruggles
On 11/16/2012 06:31 AM, Diego Biurrun wrote: On Wed, Nov 14, 2012 at 05:28:53PM -0500, Justin Ruggles wrote: On 11/14/2012 04:16 PM, Diego Biurrun wrote: --- a/libavutil/x86/x86util.asm +++ b/libavutil/x86/x86util.asm @@ -599,25 +599,37 @@ -%macro PMINSD_MMX 3 ; dst, src, tmp +%macro

Re: [libav-devel] [PATCH 41/45] x86: Rename SPLATD macro to SPLATD_OFFSET

2012-11-16 Thread Justin Ruggles
On 07/31/2012 06:18 PM, Diego Biurrun wrote: This will allow porting the optimized versions to cpuflags. --- libavutil/x86/x86util.asm |2 +- libswscale/x86/output.asm |2 +- 2 files changed, 2 insertions(+), 2 deletions(-) diff --git a/libavutil/x86/x86util.asm

[libav-devel] [PATCH 1/2] build: set -U__STRICT_ANSI__ for newlib

2012-11-16 Thread Mans Rullgard
This is (erroneously) required to enable various things in the newlib headers. As cygwin uses newlib, it is covered by this. Signed-off-by: Mans Rullgard m...@mansr.com --- configure | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/configure b/configure index af22b10..cfc7010

[libav-devel] [PATCH 2/2] configure: add basic support for ARM AArch64

2012-11-16 Thread Mans Rullgard
Signed-off-by: Mans Rullgard m...@mansr.com --- configure | 15 +++ 1 file changed, 15 insertions(+) diff --git a/configure b/configure index cfc7010..fbc49dd 100755 --- a/configure +++ b/configure @@ -1068,6 +1068,7 @@ THREADS_LIST=' ' ARCH_LIST=' +aarch64 alpha

Re: [libav-devel] [PATCH 1/2] build: set -U__STRICT_ANSI__ for newlib

2012-11-16 Thread Luca Barbato
On 11/17/2012 12:19 AM, Mans Rullgard wrote: This is (erroneously) required to enable various things in the newlib headers. As cygwin uses newlib, it is covered by this. Signed-off-by: Mans Rullgard m...@mansr.com --- configure | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) Both

Re: [libav-devel] [PATCH 2/2] h264: reset has_b_frames after enabling low_delay from SPS

2012-11-16 Thread Ronald S. Bultje
Hi, On Fri, Nov 16, 2012 at 8:43 AM, Janne Grunau janne-li...@jannau.netwrote: Fixes a crash in fuzzed file nasa-8s2.ts_s20033 caused by a too large has_b_frames value. low_delay keeps getting re-enabled from the the presumely broken SPS. --- libavcodec/h264.c | 4 +++- 1 file changed, 3

[libav-devel] [PATCH] pixdesc: add PIX_FMT_ALPHA flag

2012-11-16 Thread Luca Barbato
--- Now with pixdesc updated. libavutil/pixdesc.c | 49 + libavutil/pixdesc.h | 3 +++ 2 files changed, 28 insertions(+), 24 deletions(-) diff --git a/libavutil/pixdesc.c b/libavutil/pixdesc.c index d0889b4..439c550 100644 ---