[libav-devel] [PATCH] build: Disable VDA if an unsupported compiler is used

2016-01-19 Thread Vittorio Giovara
--- Alternatively we could make configure fail. Opinions? Vittorio configure | 9 - 1 file changed, 8 insertions(+), 1 deletion(-) diff --git a/configure b/configure index 2d0573a..b7658fe 100755 --- a/configure +++ b/configure @@ -2430,7 +2430,7 @@ enable swscale_alpha enable valgrind_

Re: [libav-devel] [PATCH] eatqi: Remove mpegvideo dependency

2016-01-19 Thread Luca Barbato
On 20/01/16 01:45, Vittorio Giovara wrote: > Signed-off-by: Vittorio Giovara > --- > configure | 2 +- > libavcodec/Makefile| 2 +- > libavcodec/eatqi.c | 179 > +++-- > libavcodec/mpeg12.h| 1 - > libavcodec/mpeg12dec.c

Re: [libav-devel] [PATCH] imgconvert: Drop outdated comment block

2016-01-19 Thread Luca Barbato
On 19/01/16 22:11, Vittorio Giovara wrote: > --- > libavcodec/imgconvert.c | 6 -- > 1 file changed, 6 deletions(-) > > diff --git a/libavcodec/imgconvert.c b/libavcodec/imgconvert.c > index f8e6f6c..4667a4f 100644 > --- a/libavcodec/imgconvert.c > +++ b/libavcodec/imgconvert.c > @@ -24,12 +2

Re: [libav-devel] [PATCH 3/3] imgconvert: Add the proper API guard to a deprecated function

2016-01-19 Thread Luca Barbato
On 19/01/16 22:09, Vittorio Giovara wrote: > --- > libavcodec/imgconvert.c | 2 ++ > libavcodec/version.h| 3 +++ > 2 files changed, 5 insertions(+) Sure! lu ___ libav-devel mailing list libav-devel@libav.org https://lists.libav.org/mailman/listin

Re: [libav-devel] [PATCH 2/3] imgconvert: Move AVPicture-related static function to the deprecated section

2016-01-19 Thread Luca Barbato
On 19/01/16 22:09, Vittorio Giovara wrote: > --- > libavcodec/imgconvert.c | 5 ++--- > 1 file changed, 2 insertions(+), 3 deletions(-) > > diff --git a/libavcodec/imgconvert.c b/libavcodec/imgconvert.c > index 4c707f3..7589828 100644 > --- a/libavcodec/imgconvert.c > +++ b/libavcodec/imgconvert.

Re: [libav-devel] [PATCH 1/3] imgconvert: Move the shrink functions only where needed

2016-01-19 Thread Luca Barbato
On 19/01/16 22:09, Vittorio Giovara wrote: > --- > libavcodec/imgconvert.c | 87 > libavcodec/imgconvert.h | 31 --- > libavcodec/mpegvideoencdsp.c | 94 > ++-- > 3 files changed, 90 insertions(

Re: [libav-devel] [PATCH] configure: Improve setting specific features

2016-01-19 Thread Luca Barbato
On 19/01/16 23:11, Hendrik Leppkes wrote: > This should probably error then on account of an explicit request that > cannot be fulfilled. It is in my plans, but I wanted to reduce it to the smallest changeset first and then build from it. To make it fail I need to change so instead of having 3 st

[libav-devel] [PATCH] eatqi: Remove mpegvideo dependency

2016-01-19 Thread Vittorio Giovara
Signed-off-by: Vittorio Giovara --- configure | 2 +- libavcodec/Makefile| 2 +- libavcodec/eatqi.c | 179 +++-- libavcodec/mpeg12.h| 1 - libavcodec/mpeg12dec.c | 5 -- 5 files changed, 145 insertions(+), 44 deletions(-

Re: [libav-devel] [PATCH] yuv2rgb: Document the color space coefficients

2016-01-19 Thread Kieran Kunhya
On Tue, 19 Jan 2016 at 22:11 Vittorio Giovara wrote: > Based on the original documentation found in mpeg2dec (1995). > > Signed-off-by: Vittorio Giovara > --- > Based off an archeological expedition with Derek. > Vittorio > > libswscale/yuv2rgb.c | 11 +++ > 1 file changed, 11 insertion

Re: [libav-devel] [PATCH] configure: Improve setting specific features

2016-01-19 Thread Hendrik Leppkes
On Tue, Jan 19, 2016 at 10:56 PM, Vittorio Giovara wrote: > On Tue, Jan 19, 2016 at 4:50 PM, Luca Barbato wrote: >> Before this patch trying to enable only https by issuing >> >> --disable-protocols --enable-protocol=https --enable-gnutls >> >> does not enable https, and >> >> --disable-a

[libav-devel] [PATCH] yuv2rgb: Document the color space coefficients

2016-01-19 Thread Vittorio Giovara
Based on the original documentation found in mpeg2dec (1995). Signed-off-by: Vittorio Giovara --- Based off an archeological expedition with Derek. Vittorio libswscale/yuv2rgb.c | 11 +++ 1 file changed, 11 insertions(+) diff --git a/libswscale/yuv2rgb.c b/libswscale/yuv2rgb.c index a4

Re: [libav-devel] [PATCH] configure: Improve setting specific features

2016-01-19 Thread Vittorio Giovara
On Tue, Jan 19, 2016 at 4:50 PM, Luca Barbato wrote: > Before this patch trying to enable only https by issuing > > --disable-protocols --enable-protocol=https --enable-gnutls > > does not enable https, and > > --disable-all --enable-protocols > > does not enable libavformat and the protoc

[libav-devel] [PATCH] configure: Improve setting specific features

2016-01-19 Thread Luca Barbato
Before this patch trying to enable only https by issuing --disable-protocols --enable-protocol=https --enable-gnutls does not enable https, and --disable-all --enable-protocols does not enable libavformat and the protocols component. Now the default-enabled options are set after the ex

[libav-devel] [PATCH] imgconvert: Drop outdated comment block

2016-01-19 Thread Vittorio Giovara
--- libavcodec/imgconvert.c | 6 -- 1 file changed, 6 deletions(-) diff --git a/libavcodec/imgconvert.c b/libavcodec/imgconvert.c index f8e6f6c..4667a4f 100644 --- a/libavcodec/imgconvert.c +++ b/libavcodec/imgconvert.c @@ -24,12 +24,6 @@ * misc image conversion routines */ -/* TODO: -

[libav-devel] [PATCH 3/3] imgconvert: Add the proper API guard to a deprecated function

2016-01-19 Thread Vittorio Giovara
--- libavcodec/imgconvert.c | 2 ++ libavcodec/version.h| 3 +++ 2 files changed, 5 insertions(+) diff --git a/libavcodec/imgconvert.c b/libavcodec/imgconvert.c index 7589828..f8e6f6c 100644 --- a/libavcodec/imgconvert.c +++ b/libavcodec/imgconvert.c @@ -39,12 +39,14 @@ #include "libavutil/i

[libav-devel] [PATCH 2/3] imgconvert: Move AVPicture-related static function to the deprecated section

2016-01-19 Thread Vittorio Giovara
--- libavcodec/imgconvert.c | 5 ++--- 1 file changed, 2 insertions(+), 3 deletions(-) diff --git a/libavcodec/imgconvert.c b/libavcodec/imgconvert.c index 4c707f3..7589828 100644 --- a/libavcodec/imgconvert.c +++ b/libavcodec/imgconvert.c @@ -159,6 +159,8 @@ enum AVPixelFormat avcodec_find_best_

[libav-devel] [PATCH 1/3] imgconvert: Move the shrink functions only where needed

2016-01-19 Thread Vittorio Giovara
--- libavcodec/imgconvert.c | 87 libavcodec/imgconvert.h | 31 --- libavcodec/mpegvideoencdsp.c | 94 ++-- 3 files changed, 90 insertions(+), 122 deletions(-) delete mode 100644 libavcodec/imgc

[libav-devel] [PATCH] configure: Improve setting specific features

2016-01-19 Thread Luca Barbato
Before this patch trying to enable only https by issuing --disable-protocols --enable-protocol=https --enable-gnutls does not enable https, and --disable-all --enable-protocols does not enable libavformat and the protocols component. Now the default-enabled options are set after the ex

Re: [libav-devel] [PATCH 2/2] configure: Make https depend on http

2016-01-19 Thread Luca Barbato
On 19/01/16 19:48, Martin Storsjö wrote: > On Sun, 17 Jan 2016, Luca Barbato wrote: > >> --- >> configure | 2 +- >> 1 file changed, 1 insertion(+), 1 deletion(-) >> >> diff --git a/configure b/configure >> index e611602..196be51 100755 >> --- a/configure >> +++ b/configure >> @@ -2305,7 +2305,7 @@

[libav-devel] [PATCH] configure: Improve setting specific features

2016-01-19 Thread Luca Barbato
Before this patch trying to enable only https by issuing --disable-protocols --enable-protocol=https --enable-gnutls does not enable https, and --disable-all --enable-protocols does not enable libavformat and the protocols component. Now a `select_any` keyword is provided to describe a

Re: [libav-devel] [PATCH 1/2] lavc: Make sure that the effective timebase would not overflow

2016-01-19 Thread Vittorio Giovara
On Tue, Jan 19, 2016 at 2:06 PM, Luca Barbato wrote: > From: Arttu Ylä-Outinen > > In the unlikely situation the user decides to set ticks_per_frame > and timebase to a value large enough to overflow. > > Signed-off-by: Luca Barbato > --- > > The same check would be needed by other encoders, not

Re: [libav-devel] [PATCH/RFC] lavc: configure: Factor out celp module

2016-01-19 Thread Vittorio Giovara
On Tue, Jan 19, 2016 at 1:36 PM, Diego Biurrun wrote: > Leave these out or create proper fine-grained dependencies. Bloating > the build for the other codecs is not acceptable. I have a very similar > patch in a branch I need to revisit some day, waiting for me to address > the same issues. ok

[libav-devel] [PATCH 1/2] lavc: Make sure that the effective timebase would not overflow

2016-01-19 Thread Luca Barbato
From: Arttu Ylä-Outinen In the unlikely situation the user decides to set ticks_per_frame and timebase to a value large enough to overflow. Signed-off-by: Luca Barbato --- The same check would be needed by other encoders, not just libkvazaar. libavcodec/utils.c | 10 ++ 1 file change

[libav-devel] [PATCH 2/2] libkvazaar: Set frame rate as a rational number

2016-01-19 Thread Luca Barbato
From: Arttu Ylä-Outinen Updates libkvazaar to pass the exact frame rate to Kvazaar by setting the numerator and denominator separately instead of a single floating point number. The exact frame rate is needed for writing timing info to the bitstream. Requires Kvazaar version 0.8.1. Signed-off-b

Re: [libav-devel] [PATCH 2/2] configure: Make https depend on http

2016-01-19 Thread Martin Storsjö
On Sun, 17 Jan 2016, Luca Barbato wrote: --- configure | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/configure b/configure index e611602..196be51 100755 --- a/configure +++ b/configure @@ -2305,7 +2305,7 @@ ffrtmphttp_protocol_select="http_protocol" gopher_protocol_select="n

Re: [libav-devel] [PATCH/RFC] lavc: configure: Factor out celp module

2016-01-19 Thread Diego Biurrun
build: Factor out CELP module It's not just configure. On Tue, Jan 19, 2016 at 01:12:11PM -0500, Vittorio Giovara wrote: > --- a/configure > +++ b/configure > @@ -1646,6 +1646,7 @@ CONFIG_EXTRA=" > blockdsp > bswapdsp > cabac > +celp_math > dirac_parse > dvprofile >

[libav-devel] [PATCH/RFC] lavc: configure: Factor out celp module

2016-01-19 Thread Vittorio Giovara
--- This might compile a couple of unnecessary files, but they are relatively small and it will simplify more entries in this way. Vittorio configure | 13 - libavcodec/Makefile | 32 ++-- 2 files changed, 18 insertions(+), 27 deletions(-) diff --

Re: [libav-devel] [PATCH 1/2] configure: Improve setting specific features

2016-01-19 Thread Vittorio Giovara
On Sun, Jan 17, 2016 at 2:55 AM, Luca Barbato wrote: > Before this patch trying to enable only https by issuing > > --disable-protocols --enable-protocol=https > > does not work as expected, nor does > > --disable-all --enable-protocols > > Now a `select_any` keyword is provided to describ

Re: [libav-devel] [PATCH 4/4] avcodec/v210: document the requirement for sample_factor

2016-01-19 Thread James Darnley
On 2016-01-19 17:24, Vittorio Giovara wrote: > On Fri, Jan 15, 2016 at 2:35 PM, James Darnley > wrote: >> The sample factor must be the same for both 8- and 10-bit functions chosen >> otherwise the output will be incorrect. > > Thanks for the set, can you explain in more detail why this > requir

Re: [libav-devel] [PATCH 00/14] codec private options

2016-01-19 Thread Vittorio Giovara
On Mon, Jan 11, 2016 at 4:14 PM, Luca Barbato wrote: > On 11/01/16 21:46, Vittorio Giovara wrote: >> On Mon, Jan 4, 2016 at 8:34 AM, Vittorio Giovara >> wrote: >>> AVCodecContext holds a lot of fields that are used seldom within >>> libavcodec, in most cases only to set options that apply to a si

Re: [libav-devel] [PATCH] qsvenc: Add private option to replace coder_type

2016-01-19 Thread Vittorio Giovara
On Mon, Jan 18, 2016 at 3:52 PM, Anton Khirnov wrote: > Quoting Vittorio Giovara (2016-01-13 21:28:11) >> Missing from be00ec832c519427cd92218abac77dafdc1d5487. >> --- >> libavcodec/qsvenc.c | 10 -- >> libavcodec/qsvenc.h | 2 ++ >> 2 files changed, 10 insertions(+), 2 deletions(-) >> >

Re: [libav-devel] [PATCH] libkvazaar: Set frame rate as a rational number

2016-01-19 Thread Vittorio Giovara
On Mon, Jan 18, 2016 at 2:59 PM, Luca Barbato wrote: > On 18/01/16 15:05, Arttu Ylä-Outinen wrote: >> On 2016-01-18 15:22, Luca Barbato wrote: >>> On 18/01/16 08:23, Arttu Ylä-Outinen wrote: I sent this to FFmpeg as well and they noticed that the multiplication might overflow. I'll send

Re: [libav-devel] [PATCH 4/4] avcodec/v210: document the requirement for sample_factor

2016-01-19 Thread Vittorio Giovara
On Fri, Jan 15, 2016 at 2:35 PM, James Darnley wrote: > The sample factor must be the same for both 8- and 10-bit functions chosen > otherwise the output will be incorrect. Thanks for the set, can you explain in more detail why this requirement is necessary? -- Vittorio _

Re: [libav-devel] [PATCH 2/2] configure: Make https depend on http

2016-01-19 Thread Diego Biurrun
On Sun, Jan 17, 2016 at 08:55:12AM +0100, Luca Barbato wrote: > --- a/configure > +++ b/configure > @@ -2305,7 +2305,7 @@ ffrtmphttp_protocol_select="http_protocol" > http_protocol_select="tcp_protocol" > httpproxy_protocol_select="tcp_protocol" > -https_protocol_select="tls_protocol" > +https_pr

[libav-devel] [PATCH] x86: Add SSSE3_SLOW CPU flag and related convenience macros

2016-01-19 Thread Diego Biurrun
--- Fixed FAST_SLOW typo. doc/APIchanges | 3 +++ libavutil/cpu.h | 1 + libavutil/version.h | 2 +- libavutil/x86/cpu.h | 6 ++ 4 files changed, 11 insertions(+), 1 deletion(-) diff --git a/doc/APIchanges b/doc/APIchanges index ffa270e..f2e6bea 100644 --- a/doc/APIchanges +++ b/do

Re: [libav-devel] [PATCH] x86: Add SSSE3_SLOW CPU flag and related convenience macros

2016-01-19 Thread Diego Biurrun
On Tue, Jan 19, 2016 at 01:53:02PM +0200, Martin Storsjö wrote: > On Tue, 19 Jan 2016, Diego Biurrun wrote: > >--- a/libavutil/x86/cpu.h > >+++ b/libavutil/x86/cpu.h > >@@ -38,6 +38,8 @@ > >#define X86_SSE3_FAST(flags)CPUEXT_FAST(flags, SSE3) > >#define X86_SSE3_SLOW(flags)CPUEXT_SL

Re: [libav-devel] [PATCH] flvdec: perform duration search just once

2016-01-19 Thread Luca Barbato
On 19/01/16 09:02, Martin Storsjö wrote: > On Mon, 18 Jan 2016, Luca Barbato wrote: > >> On 18/01/16 15:06, Martin Storsjö wrote: >>> From: Michael Niedermayer >>> >>> When loading a truncated flv file over http, it would previously try to >>> do a seek to the end for every packet read. >>> --- >

Re: [libav-devel] [PATCH] flvdec: Add sanity checking of the last packet size

2016-01-19 Thread Luca Barbato
On 19/01/16 09:07, Martin Storsjö wrote: > For http, this avoids spurious warnings about failed requests (e.g. > HTTP error 416 Requested Range Not Satisfiable), if the last packet > is truncated and the size read is bogus. > --- > libavformat/flvdec.c | 16 +--- > 1 file changed, 9 in

Re: [libav-devel] [PATCH] x86: Add SSSE3_SLOW CPU flag and related convenience macros

2016-01-19 Thread Martin Storsjö
On Tue, 19 Jan 2016, Diego Biurrun wrote: --- Added APIchanges entry, version bump and use a non-conflicting flag value. doc/APIchanges | 3 +++ libavutil/cpu.h | 1 + libavutil/version.h | 2 +- libavutil/x86/cpu.h | 6 ++ 4 files changed, 11 insertions(+), 1 deletion(-) diff --git

[libav-devel] [PATCH] x86: Add SSSE3_SLOW CPU flag and related convenience macros

2016-01-19 Thread Diego Biurrun
--- Added APIchanges entry, version bump and use a non-conflicting flag value. doc/APIchanges | 3 +++ libavutil/cpu.h | 1 + libavutil/version.h | 2 +- libavutil/x86/cpu.h | 6 ++ 4 files changed, 11 insertions(+), 1 deletion(-) diff --git a/doc/APIchanges b/doc/APIchanges index

Re: [libav-devel] [PATCH 03/10] x86: hpeldsp: Split off VP3-specific bits into a separate file

2016-01-19 Thread Diego Biurrun
On Mon, Jan 18, 2016 at 04:50:18PM +0100, Hendrik Leppkes wrote: > On Mon, Jan 18, 2016 at 4:38 PM, Diego Biurrun wrote: > > --- /dev/null > > +++ b/libavcodec/x86/hpeldsp_vp3_init.c > > @@ -0,0 +1,54 @@ > > + > > +#include "libavutil/attributes.h" > > +#include "libavutil/cpu.h" > > +#include "li

Re: [libav-devel] [PATCH 07/10] x86: Add SSSE3_SLOW CPU flag and related convenience macros

2016-01-19 Thread Diego Biurrun
On Mon, Jan 18, 2016 at 11:24:38PM -0300, James Almer wrote: > On 1/18/2016 12:38 PM, Diego Biurrun wrote: > > --- > > libavutil/cpu.h | 1 + > > libavutil/x86/cpu.h | 6 ++ > > Missing minor version bump. > > > --- a/libavutil/cpu.h > > +++ b/libavutil/cpu.h > > @@ -38,6 +38,7 @@ > > #d

[libav-devel] [PATCH] flvdec: Add sanity checking of the last packet size

2016-01-19 Thread Martin Storsjö
For http, this avoids spurious warnings about failed requests (e.g. HTTP error 416 Requested Range Not Satisfiable), if the last packet is truncated and the size read is bogus. --- libavformat/flvdec.c | 16 +--- 1 file changed, 9 insertions(+), 7 deletions(-) diff --git a/libavformat

Re: [libav-devel] [PATCH] flvdec: perform duration search just once

2016-01-19 Thread Martin Storsjö
On Mon, 18 Jan 2016, Luca Barbato wrote: On 18/01/16 15:06, Martin Storsjö wrote: From: Michael Niedermayer When loading a truncated flv file over http, it would previously try to do a seek to the end for every packet read. --- libavformat/flvdec.c | 5 - 1 file changed, 4 insertions(+),