Re: [libav-devel] [PATCH] dca: Move the downmix request check outside the loop

2017-04-24 Thread Vittorio Giovara
On Mon, Apr 24, 2017 at 4:41 PM, Luca Barbato wrote: > From: Anton Khirnov > > --- > > An easy part of the whole ch_layout set we are shaping up. > > libavcodec/dcadec.c | 11 +++ > 1 file changed, 7 insertions(+), 4 deletions(-) > > diff --git a/libavcodec/dcadec.c b/libavcodec/dcadec.c

[libav-devel] [PATCH] dca: Move the downmix request check outside the loop

2017-04-24 Thread Luca Barbato
From: Anton Khirnov --- An easy part of the whole ch_layout set we are shaping up. libavcodec/dcadec.c | 11 +++ 1 file changed, 7 insertions(+), 4 deletions(-) diff --git a/libavcodec/dcadec.c b/libavcodec/dcadec.c index 3fe46cd..9c1f878 100644 --- a/libavcodec/dcadec.c +++ b/libavco

Re: [libav-devel] [RFC] Getting rid of non-native endian pixfmts

2017-04-24 Thread Luca Barbato
On 4/24/17 7:19 PM, wm4 wrote: > On Mon, 24 Apr 2017 18:34:41 +0200 > Luca Barbato wrote: > >> On 4/24/17 5:37 PM, wm4 wrote: >>> I propose that we remove non-native endian pixfmt variations. Pixel >>> data would always be in native endianness (for example little endian on >>> a little endian sys

Re: [libav-devel] [RFC] Getting rid of non-native endian pixfmts

2017-04-24 Thread Vittorio Giovara
On Mon, Apr 24, 2017 at 1:32 PM, wm4 wrote: > On Mon, 24 Apr 2017 20:27:28 +0300 > Rémi Denis-Courmont wrote: > >> Le maanantaina 24. huhtikuuta 2017, 17.37.00 EEST wm4 a écrit : >> > I propose that we remove non-native endian pixfmt variations. Pixel >> > data would always be in native endiannes

Re: [libav-devel] [RFC] Getting rid of non-native endian pixfmts

2017-04-24 Thread Vittorio Giovara
On Mon, Apr 24, 2017 at 11:37 AM, wm4 wrote: > I propose that we remove non-native endian pixfmt variations. Pixel > data would always be in native endianness (for example little endian on > a little endian system). All LE/BE formats would be dropped, and only > those without suffix would be left

Re: [libav-devel] [RFC] Getting rid of non-native endian pixfmts

2017-04-24 Thread Jean-Baptiste Kempf
Hello, On Mon, 24 Apr 2017, at 08:37, wm4 wrote: > I propose that we remove non-native endian pixfmt variations. Pixel > data would always be in native endianness (for example little endian on > a little endian system). All LE/BE formats would be dropped, and only > those without suffix would be l

Re: [libav-devel] [RFC] Getting rid of non-native endian pixfmts

2017-04-24 Thread wm4
On Mon, 24 Apr 2017 20:27:28 +0300 Rémi Denis-Courmont wrote: > Le maanantaina 24. huhtikuuta 2017, 17.37.00 EEST wm4 a écrit : > > I propose that we remove non-native endian pixfmt variations. Pixel > > data would always be in native endianness (for example little endian on > > a little endian s

Re: [libav-devel] [RFC] Getting rid of non-native endian pixfmts

2017-04-24 Thread wm4
On Mon, 24 Apr 2017 18:34:41 +0200 Luca Barbato wrote: > On 4/24/17 5:37 PM, wm4 wrote: > > I propose that we remove non-native endian pixfmt variations. Pixel > > data would always be in native endianness (for example little endian on > > a little endian system). All LE/BE formats would be dropp

Re: [libav-devel] [RFC] Getting rid of non-native endian pixfmts

2017-04-24 Thread Luca Barbato
On 4/24/17 5:37 PM, wm4 wrote: > I propose that we remove non-native endian pixfmt variations. Pixel > data would always be in native endianness (for example little endian on > a little endian system). All LE/BE formats would be dropped, and only > those without suffix would be left (mostly this im

Re: [libav-devel] [RFC] [PATCH] stereo3d: Update the naming API to be more consistent

2017-04-24 Thread Vittorio Giovara
On Mon, Apr 24, 2017 at 11:24 AM, James Almer wrote: > On 4/24/2017 11:55 AM, Vittorio Giovara wrote: >> On Sun, Apr 23, 2017 at 11:31 PM, James Almer wrote: >>> On 4/20/2017 12:34 PM, Vittorio Giovara wrote: Change input type of the type->str function and return a proper error cod

[libav-devel] [RFC] Getting rid of non-native endian pixfmts

2017-04-24 Thread wm4
I propose that we remove non-native endian pixfmt variations. Pixel data would always be in native endianness (for example little endian on a little endian system). All LE/BE formats would be dropped, and only those without suffix would be left (mostly this implies moving the current native-endian

Re: [libav-devel] [PATCH] configure: Escape elements being filtered

2017-04-24 Thread Diego Biurrun
On Thu, Apr 06, 2017 at 06:14:29PM +0200, Luca Barbato wrote: > The function is used to filter out ldflags arguments that may contain $. > Update filter() as well to be consistent. configure: Properly escape arguments in filter/filter_out helper functions The arguments may contain '$', which

Re: [libav-devel] [RFC] [PATCH] stereo3d: Update the naming API to be more consistent

2017-04-24 Thread James Almer
On 4/24/2017 11:55 AM, Vittorio Giovara wrote: > On Sun, Apr 23, 2017 at 11:31 PM, James Almer wrote: >> On 4/20/2017 12:34 PM, Vittorio Giovara wrote: >>> >>> Change input type of the type->str function and return a proper >>> error code for the str->type function. >>> --- >>> Similar reasoning t

Re: [libav-devel] [PATCH 2/2] utvideodec: Fix gradient prediction when stride does not match width

2017-04-24 Thread Vittorio Giovara
On Sat, Apr 22, 2017 at 6:33 AM, Luca Barbato wrote: > From: Paul B Mahol > > Signed-off-by: Paul B Mahol > Signed-off-by: Luca Barbato > --- > libavcodec/utvideodec.c | 12 ++-- > 1 file changed, 10 insertions(+), 2 deletions(-) > > diff --git a/libavcodec/utvideodec.c b/libavcodec/ut

Re: [libav-devel] [PATCH 1/2] utvideodec: Fix decoding odd sizes with interlaced video with some formats

2017-04-24 Thread Vittorio Giovara
On Sat, Apr 22, 2017 at 6:33 AM, Luca Barbato wrote: > From: Paul B Mahol > > Signed-off-by: Paul B Mahol > Signed-off-by: Luca Barbato > --- > libavcodec/utvideodec.c | 12 +++- > 1 file changed, 11 insertions(+), 1 deletion(-) > > diff --git a/libavcodec/utvideodec.c b/libavcodec/utv

Re: [libav-devel] [PATCH] avprobe: Handle invalid values for the color description

2017-04-24 Thread Vittorio Giovara
On Sun, Apr 23, 2017 at 9:23 AM, Luca Barbato wrote: > On 4/18/17 7:11 PM, Luca Barbato wrote: >> On 17/04/2017 18:04, Vittorio Giovara wrote: >>> On Mon, Apr 17, 2017 at 9:48 AM, Luca Barbato wrote: print_str() cannot print NULL. Bug-Id: 1040 CC: libav-sta...@libav.org -

Re: [libav-devel] [RFC] [PATCH] pixdesc: Change color property name APIs return type

2017-04-24 Thread Vittorio Giovara
On Thu, Apr 20, 2017 at 11:26 AM, Vittorio Giovara wrote: > This should make these APIs simpler to use, and less error prone > in case the caller does not check they are valid, and makes them > more similar to other naming APIs. > --- > This should help in the bug in avprobe found by Luca. > Sendi

Re: [libav-devel] [RFC] [PATCH] stereo3d: Update the naming API to be more consistent

2017-04-24 Thread Vittorio Giovara
On Sun, Apr 23, 2017 at 11:31 PM, James Almer wrote: > On 4/20/2017 12:34 PM, Vittorio Giovara wrote: >> >> Change input type of the type->str function and return a proper >> error code for the str->type function. >> --- >> Similar reasoning to the previous patch. >> Vittorio >> >> libavutil/ste

Re: [libav-devel] [PATCH] configure: Fix sem_timedwait probe

2017-04-24 Thread Diego Biurrun
On Sun, Apr 23, 2017 at 02:14:32PM +, Luca Barbato wrote: > It requires pthreads. configure: Add missing pthreads extralibs to sem_timedwait check > --- a/configure > +++ b/configure > @@ -4693,7 +4693,7 @@ if ! disabled pthreads && ! enabled w32threads; then > > enabled pthreads && > -

Re: [libav-devel] [PATCH] Add ClearVideo decoder

2017-04-24 Thread James Almer
On 4/24/2017 11:13 AM, Diego Biurrun wrote: > On Sat, Apr 22, 2017 at 01:05:01PM +0200, Diego Biurrun wrote: >> --- /dev/null >> +++ b/libavcodec/clearvideo.c >> @@ -0,0 +1,387 @@ >> +static int clv_decode_frame(AVCodecContext *avctx, void *data, >> +int *got_frame, AVPa

Re: [libav-devel] [PATCH] Add ClearVideo decoder

2017-04-24 Thread Diego Biurrun
On Sat, Apr 22, 2017 at 01:05:01PM +0200, Diego Biurrun wrote: > --- /dev/null > +++ b/libavcodec/clearvideo.c > @@ -0,0 +1,387 @@ > +static int clv_decode_frame(AVCodecContext *avctx, void *data, > +int *got_frame, AVPacket *avpkt) > +{ > +if (frame_type & 0x2) { >