Re: [libav-devel] [PATCH 1/3] alacenc: support 24-bit encoding

2012-11-17 Thread Justin Ruggles
On 11/09/2012 05:14 PM, Justin Ruggles wrote: > --- > libavcodec/alacenc.c | 101 > +- > 1 files changed, 75 insertions(+), 26 deletions(-) ping -Justin ___ libav-devel mailing list libav-devel@libav.o

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

2012-11-17 Thread John Stebbins
On 11/17/2012 11:30 AM, Anton Khirnov wrote: > > On Sat, 17 Nov 2012 11:12:34 -0800, John Stebbins > wrote: >> On 11/16/2012 09:46 AM, Anton Khirnov wrote: >>> >>> On Fri, 2 Nov 2012 09:32:51 -0700, John Stebbins >>> wrote: On 11/02/2012 08:20 AM, Diego Biurrun wrote: > On Sun, Oct 14

[libav-devel] [RFC] imgconvert: remove PixFmtInfo

2012-11-17 Thread Luca Barbato
It is pleonastic and was used in stale functions pending replacement. --- A good stab at imgconvert, the code is mostly rfc. libavcodec/imgconvert.c | 496 1 file changed, 35 insertions(+), 461 deletions(-) diff --git a/libavcodec/imgconvert.c b/

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

2012-11-17 Thread Anton Khirnov
On Sat, 17 Nov 2012 11:12:34 -0800, John Stebbins wrote: > On 11/16/2012 09:46 AM, Anton Khirnov wrote: > > > > On Fri, 2 Nov 2012 09:32:51 -0700, John Stebbins > > wrote: > >> On 11/02/2012 08:20 AM, Diego Biurrun wrote: > >>> On Sun, Oct 14, 2012 at 04:52:12PM +0200, John Stebbins wrote: >

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

2012-11-17 Thread John Stebbins
On 11/16/2012 09:46 AM, Anton Khirnov wrote: > > On Fri, 2 Nov 2012 09:32:51 -0700, John Stebbins > 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.

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

2012-11-17 Thread Luca Barbato
On 11/17/2012 06:56 PM, Diego Biurrun wrote: > On Sat, Nov 17, 2012 at 05:31:51PM +, Måns Rullgård wrote: >> Diego Biurrun writes: >>> On Fri, Nov 16, 2012 at 11:19:47PM +, Mans Rullgard wrote: This is (erroneously) required to enable various things in the newlib headers. As cyg

Re: [libav-devel] [PATCH] configure: Refactor CPPFLAGS settings for glibc/uclibc

2012-11-17 Thread Luca Barbato
On 11/17/2012 07:10 PM, Mans Rullgard wrote: > From: Diego Biurrun > > Signed-off-by: Mans Rullgard > --- > configure | 6 +++--- > 1 file changed, 3 insertions(+), 3 deletions(-) Looks fine, remind me to play with musl and add stuff if required. lu __

Re: [libav-devel] [PATCH] lavr: fix the decision for writing directly to the output buffer

2012-11-17 Thread Luca Barbato
On 11/08/2012 11:19 PM, Justin Ruggles wrote: > If there are any samples remaining in the output fifo from previous conversion > calls, we have to output those samples first instead of doing direct output > of the current samples. > --- > libavresample/utils.c |9 + > 1 files changed,

[libav-devel] [PATCH] configure: Refactor CPPFLAGS settings for glibc/uclibc

2012-11-17 Thread Mans Rullgard
From: Diego Biurrun Signed-off-by: Mans Rullgard --- configure | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/configure b/configure index fbc49dd..dbd4caf 100755 --- a/configure +++ b/configure @@ -2886,7 +2886,6 @@ case $target_os in add_cppflags -U__STRICT_

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

2012-11-17 Thread Måns Rullgård
Diego Biurrun writes: > On Sat, Nov 17, 2012 at 05:31:51PM +, Måns Rullgård wrote: >> Diego Biurrun writes: >> > On Fri, Nov 16, 2012 at 11:19:47PM +, Mans Rullgard wrote: >> >> This is (erroneously) required to enable various things in the >> >> newlib headers. As cygwin uses newlib, i

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

2012-11-17 Thread Diego Biurrun
On Sat, Nov 17, 2012 at 05:31:51PM +, Måns Rullgård wrote: > Diego Biurrun writes: > > On Fri, Nov 16, 2012 at 11:19:47PM +, Mans Rullgard wrote: > >> This is (erroneously) required to enable various things in the > >> newlib headers. As cygwin uses newlib, it is covered by this. > >> >

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

2012-11-17 Thread Diego Biurrun
On Sat, Nov 17, 2012 at 05:31:15PM +, Måns Rullgård wrote: > Diego Biurrun writes: > > On Fri, Nov 16, 2012 at 11:19:48PM +, Mans Rullgard wrote: > >> --- a/configure > >> +++ b/configure > >> @@ -2666,6 +2670,17 @@ elif enabled avr32; then > >> ;; > >> esac > >> > >> +elif

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

2012-11-17 Thread Måns Rullgård
Diego Biurrun writes: > On Fri, Nov 16, 2012 at 11:19:47PM +, Mans Rullgard wrote: >> This is (erroneously) required to enable various things in the >> newlib headers. As cygwin uses newlib, it is covered by this. >> >> --- a/configure >> +++ b/configure >> @@ -2863,7 +2863,6 @@ case $targe

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

2012-11-17 Thread Måns Rullgård
Diego Biurrun writes: > On Fri, Nov 16, 2012 at 11:19:48PM +, Mans Rullgard wrote: >> --- a/configure >> +++ b/configure >> @@ -2666,6 +2670,17 @@ elif enabled avr32; then >> ;; >> esac >> >> +elif enabled aarch64; then >> + >> +case $cpu in >> +armv*) >> +

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

2012-11-17 Thread Justin Ruggles
On 11/17/2012 12:13 PM, Diego Biurrun wrote: > --- > This version takes also changes the SPLATD in libswscale and > drops the SPLATD macro that provides an offset, as no instance > actually uses an offset. > > libavcodec/x86/dsputil.asm|2 -- > libavcodec/x86/fmtconvert.asm |3 --- >

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

2012-11-17 Thread Diego Biurrun
On Fri, Nov 16, 2012 at 11:19:47PM +, Mans Rullgard wrote: > This is (erroneously) required to enable various things in the > newlib headers. As cygwin uses newlib, it is covered by this. > > --- a/configure > +++ b/configure > @@ -2863,7 +2863,6 @@ case $target_os in > SHFLAGS='-sha

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

2012-11-17 Thread Diego Biurrun
On Fri, Nov 16, 2012 at 11:19:48PM +, Mans Rullgard wrote: > --- a/configure > +++ b/configure > @@ -2666,6 +2670,17 @@ elif enabled avr32; then > ;; > esac > > +elif enabled aarch64; then > + > +case $cpu in > +armv*) > +cpuflags="-march=$cpu" > +

Re: [libav-devel] [PATCH] lavr: fix the decision for writing directly to the output buffer

2012-11-17 Thread Justin Ruggles
On 11/08/2012 05:19 PM, Justin Ruggles wrote: > If there are any samples remaining in the output fifo from previous conversion > calls, we have to output those samples first instead of doing direct output > of the current samples. > --- > libavresample/utils.c |9 + > 1 files changed,

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

2012-11-17 Thread Diego Biurrun
--- This version takes also changes the SPLATD in libswscale and drops the SPLATD macro that provides an offset, as no instance actually uses an offset. libavcodec/x86/dsputil.asm|2 -- libavcodec/x86/fmtconvert.asm |3 --- libavutil/x86/x86util.asm | 22 ++

Re: [libav-devel] [PATCH 3/4] x86: ac3dsp: replace inline asm for in-decoder downmixing with yasm asm

2012-11-17 Thread Diego Biurrun
On Sat, Nov 17, 2012 at 10:33:27AM -0500, Justin Ruggles wrote: > > --- a/libavcodec/ac3dsp.c > +++ b/libavcodec/ac3dsp.c > @@ -24,6 +24,10 @@ > #include "ac3.h" > #include "ac3dsp.h" > > +#if ARCH_X86 > +#include "x86/ac3dsp.h" > +#endif > + > --- /dev/null > +++ b/libavcodec/x86/ac3dsp.h > @

[libav-devel] [PATCH 4/4] ac3dsp: cosmetics: indentation

2012-11-17 Thread Justin Ruggles
--- libavcodec/ac3dsp.c | 48 1 files changed, 24 insertions(+), 24 deletions(-) diff --git a/libavcodec/ac3dsp.c b/libavcodec/ac3dsp.c index aa17a90..3d6d50e 100644 --- a/libavcodec/ac3dsp.c +++ b/libavcodec/ac3dsp.c @@ -180,39 +180,39 @@ static

[libav-devel] [PATCH 3/4] x86: ac3dsp: replace inline asm for in-decoder downmixing with yasm asm

2012-11-17 Thread Justin Ruggles
Adds a wrapper function for downmixing which detects channel count changes and updates the selected downmix function accordingly. --- libavcodec/ac3dec.c |8 +- libavcodec/ac3dsp.c | 72 +--- libavcodec/ac3dsp.h |8 +- libavcodec/x86/Make

[libav-devel] [PATCH 2/4] ac3dsp: add some special-case handling for the C downmix function

2012-11-17 Thread Justin Ruggles
This is about 200% faster for in-decoder downmixing of 5.0 and 5.1 content. --- libavcodec/ac3dsp.c | 39 ++- 1 files changed, 38 insertions(+), 1 deletions(-) diff --git a/libavcodec/ac3dsp.c b/libavcodec/ac3dsp.c index 6a30561..5b7e2e7 100644 --- a/libavcod

[libav-devel] [PATCH 1/4] ac3dsp: reverse matrix in/out order in downmix()

2012-11-17 Thread Justin Ruggles
Also use (float **) instead of (float (*)[2]). This matches the matrix layout in libavresample so we can reuse some assembly code between the two. --- libavcodec/ac3dec.c | 40 +++- libavcodec/ac3dec.h |2 +- libavcodec/ac3dsp.c

[libav-devel] [PATCH 0/4] x86: replace ac3 downmixing inline asm with yasm asm

2012-11-17 Thread Justin Ruggles
These patches depend on Mans' ac3dec planar decoding patch set, which he seems reluctant to push due to the inline asm changes. This removes the inline asm and replaces it with a simplified version of the x86 downmixing asm from libavresample. Justin Ruggles (4): ac3dsp: reverse matrix in/out o

Re: [libav-devel] [PATCH 3/4] avcodec: split avpicture from imgconvert

2012-11-17 Thread Justin Ruggles
On 11/12/2012 08:11 PM, Luca Barbato wrote: > On 11/12/2012 02:13 AM, Kieran Kunhya wrote: >> On Mon, Nov 12, 2012 at 1:09 AM, Kieran Kunhya >> wrote: >>> On Mon, Nov 12, 2012 at 12:25 AM, Luca Barbato wrote: All the non deprecated functions are in avpicture.c now. --- libavcodec

Re: [libav-devel] [PATCH 3/4] avcodec: split avpicture from imgconvert

2012-11-17 Thread Luca Barbato
On 11/12/2012 01:25 AM, Luca Barbato wrote: > All the non deprecated functions are in avpicture.c now. > --- > libavcodec/Makefile | 1 + > libavcodec/avpicture.c | 123 > > libavcodec/imgconvert.c | 85 - >

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

2012-11-17 Thread Loren Merritt
On Fri, 16 Nov 2012, Ronald S. Bultje wrote: > So ... I originally added code to not support changing bitdepth at all, > basically to prevent this and alike issues. E.g., what if the reference is > 8bit but the next frame is 10bit? The standard allows SPS to change only at IDR-frames, i.e. where