[libav-devel] [PATCH 12/12] build: Fine-grained link-time dependency settings

2017-01-24 Thread Diego Biurrun
Previously, all link-time dependencies were added for all libraries, resulting in bogus link-time dependencies since not all dependencies are shared across libraries. Also, in some cases like libavutil, not all dependencies were taken into account, resulting in some cases of underlinking. To

[libav-devel] [PATCH 09/12] configure: Add proper weak dependency of drawtext filter on libfontconfig

2017-01-24 Thread Diego Biurrun
--- configure | 1 + 1 file changed, 1 insertion(+) diff --git a/configure b/configure index 90bc8b2..9ff6ad0 100755 --- a/configure +++ b/configure @@ -2471,6 +2471,7 @@ deinterlace_qsv_filter_deps="libmfx" deinterlace_vaapi_filter_deps="vaapi" delogo_filter_deps="gpl"

[libav-devel] [v3] fine-grained library dependencies for all components

2017-01-24 Thread Diego Biurrun
.. and along comes another iteration of my patch along with another host of preliminaries .. This addresses the main concern of the last iteration that each external library dependency added another extralibs line in configure, resulting in a lot of boilerplate duplication. This is taken care of

[libav-devel] [PATCH 04/12] configure: Simplify libxcb check

2017-01-24 Thread Diego Biurrun
--- configure | 18 -- 1 file changed, 4 insertions(+), 14 deletions(-) diff --git a/configure b/configure index c543c05..4ee4dab 100755 --- a/configure +++ b/configure @@ -2397,6 +2397,7 @@ sndio_outdev_deps="sndio" v4l2_indev_deps_any="linux_videodev2_h sys_videoio_h"

[libav-devel] [PATCH 01/12] configure: Use cppflags check helper functions where appropriate

2017-01-24 Thread Diego Biurrun
--- configure | 14 -- 1 file changed, 4 insertions(+), 10 deletions(-) diff --git a/configure b/configure index 61d1807..2c8b57e 100755 --- a/configure +++ b/configure @@ -3746,18 +3746,12 @@ add_cppflags -D_ISOC99_SOURCE # some compilers silently accept -std=c11, so we also need

[libav-devel] [PATCH 05/12] configure: Add require_header() convenience function

2017-01-24 Thread Diego Biurrun
Simplifies checking for external library headers and aborting if the external library support was requested, but is not available. --- configure | 15 +++ 1 file changed, 11 insertions(+), 4 deletions(-) diff --git a/configure b/configure index 4ee4dab..a5d8274 100755 --- a/configure

[libav-devel] [PATCH 02/12] configure: Merge compiler/libc/os hacks sections

2017-01-24 Thread Diego Biurrun
--- configure | 27 +-- 1 file changed, 13 insertions(+), 14 deletions(-) diff --git a/configure b/configure index 2c8b57e..f7436c5 100755 --- a/configure +++ b/configure @@ -4097,26 +4097,25 @@ test -n "$libc_type" && enable libc_$libc_type probe_libc host_ test -n

[libav-devel] [PATCH 11/12] configure: Simplify dlopen check

2017-01-24 Thread Diego Biurrun
--- configure | 23 --- 1 file changed, 8 insertions(+), 15 deletions(-) diff --git a/configure b/configure index 04f3beb..e07dd0d 100755 --- a/configure +++ b/configure @@ -2220,10 +2220,8 @@ wmv3_vaapi_hwaccel_select="vc1_vaapi_hwaccel"

[libav-devel] [PATCH 10/12] configure: Drop weak dependencies on external libraries for webm muxer

2017-01-24 Thread Diego Biurrun
Weak dependencies on external libraries do not obviate having to explicitly enable these libraries, so the weak dependency does not simplify the configure command line nor have any real effect. --- configure | 1 - 1 file changed, 1 deletion(-) diff --git a/configure b/configure index

[libav-devel] [PATCH 08/12] configure: Rework dependency handling for conflicting components

2017-01-24 Thread Diego Biurrun
This makes the feature more visible and obvious. --- configure | 22 +- 1 file changed, 13 insertions(+), 9 deletions(-) diff --git a/configure b/configure index d3fd489..90bc8b2 100755 --- a/configure +++ b/configure @@ -620,19 +620,21 @@ do_check_deps(){ eval

[libav-devel] [PATCH 03/12] configure: Use inline asm check helper function where appropriate

2017-01-24 Thread Diego Biurrun
--- configure | 4 +--- 1 file changed, 1 insertion(+), 3 deletions(-) diff --git a/configure b/configure index f7436c5..c543c05 100755 --- a/configure +++ b/configure @@ -4165,9 +4165,7 @@ EOF sym=$($nm $TMPO | awk '/ff_extern/{ print substr($0, match($0, /[^ \t]*ff_extern/)) }')

[libav-devel] [PATCH 06/12] configure: Add require_cpp_condition() convenience function

2017-01-24 Thread Diego Biurrun
Simplifies checking for conditions in external library headers and aborting if said conditions are not met. --- configure | 17 +++-- 1 file changed, 11 insertions(+), 6 deletions(-) diff --git a/configure b/configure index a5d8274..747ce17 100755 --- a/configure +++ b/configure @@

[libav-devel] [PATCH 07/12] configure: Add name parameter to require_pkg_config() helper function

2017-01-24 Thread Diego Biurrun
This allows distinguishing between the internal variable name for external libraries and the pkg-config package name. Having both names available avoids special-casing outside the helper function when the two identifiers do not match. --- configure | 59

Re: [libav-devel] [PATCH 09/11] wma: Convert to the new bitstream reader

2017-01-24 Thread Vittorio Giovara
On Wed, Jan 11, 2017 at 6:10 PM, Diego Biurrun wrote: > From: Alexandra Hájková > > --- > libavcodec/wma.c| 41 +++ > libavcodec/wma.h| 8 +- > libavcodec/wmadec.c | 64 +-- > libavcodec/wmalosslessdec.c |

Re: [libav-devel] [PATCH 1/3] build: Map -Wall compiler flag to -W3 for MSVC and -Wextra to -W4

2017-01-24 Thread Luca Barbato
On 19/01/2017 11:43, Diego Biurrun wrote: > These are more appropriate warning level equivalents. > --- > configure | 4 +++- > 1 file changed, 3 insertions(+), 1 deletion(-) > Not against. ___ libav-devel mailing list libav-devel@libav.org

Re: [libav-devel] [PATCH 3/3] configure: Use proper compiler-specific speed flags for hostcc

2017-01-24 Thread Luca Barbato
On 19/01/2017 11:43, Diego Biurrun wrote: > --- > configure | 3 ++- > 1 file changed, 2 insertions(+), 1 deletion(-) > > diff --git a/configure b/configure > index 7d78ac0..09d2e06 100755 > --- a/configure > +++ b/configure > @@ -3375,6 +3375,7 @@ set_ccvars CC > > probe_cc hostcc "$host_cc"

Re: [libav-devel] [PATCH 2/3] configure: Move optflags checks to a more sensible place

2017-01-24 Thread Luca Barbato
On 19/01/2017 11:43, Diego Biurrun wrote: > --- > configure | 7 +++ > 1 file changed, 3 insertions(+), 4 deletions(-) > > diff --git a/configure b/configure > index 461d47c..7d78ac0 100755 > --- a/configure > +++ b/configure > @@ -4929,6 +4929,9 @@ check_optflags(){ > enabled lto &&

Re: [libav-devel] [PATCH 06/11] tak: Convert to the new bitstream reader

2017-01-24 Thread Vittorio Giovara
On Wed, Jan 11, 2017 at 6:10 PM, Diego Biurrun wrote: > From: Alexandra Hájková > > --- > libavcodec/tak.c| 49 +++ > libavcodec/tak.h| 10 +-- > libavcodec/tak_parser.c | 15 ++--- > libavcodec/takdec.c | 158 >

Re: [libav-devel] [PATCH 2/2] truemotion2rt: Convert to the new bitstream reader

2017-01-24 Thread Vittorio Giovara
On Wed, Jan 11, 2017 at 10:44 PM, Diego Biurrun wrote: > --- > > This one is too old and fringe to need benchmarking. > > libavcodec/truemotion2rt.c | 16 > 1 file changed, 8 insertions(+), 8 deletions(-) sure -- Vittorio

Re: [libav-devel] [PATCH 1/2] magicyuv: Convert to the new bitstream reader

2017-01-24 Thread Vittorio Giovara
On Wed, Jan 11, 2017 at 10:44 PM, Diego Biurrun wrote: > --- > > Another fringe codec that should need no benchmarks. > > libavcodec/magicyuv.c | 33 + > 1 file changed, 17 insertions(+), 16 deletions(-) > > diff --git a/libavcodec/magicyuv.c

Re: [libav-devel] [PATCH] swscale-test: const correctness for pointer variable

2017-01-24 Thread Vittorio Giovara
On Tue, Jan 24, 2017 at 11:33 AM, Diego Biurrun wrote: > On Thu, Jan 19, 2017 at 12:02:43PM +0100, Diego Biurrun wrote: >> libswscale/swscale-test.c:369:20: warning: passing argument 2 of ‘sws_scale’ >> from incompatible pointer type [-Wincompatible-pointer-types] >>

Re: [libav-devel] [PATCH] des-test: Pass the proper types to av_des_*() functions

2017-01-24 Thread Diego Biurrun
On Thu, Jan 19, 2017 at 12:10:41PM +0100, Diego Biurrun wrote: > Fixes a number of incompatible pointer type warnings. > --- > > Better names for the union or alternative suggestions welcome. > > libavutil/tests/des.c | 35 +++ > 1 file changed, 19 insertions(+),

Re: [libav-devel] [PATCH] swscale-test: const correctness for pointer variable

2017-01-24 Thread Diego Biurrun
On Thu, Jan 19, 2017 at 12:02:43PM +0100, Diego Biurrun wrote: > libswscale/swscale-test.c:369:20: warning: passing argument 2 of ‘sws_scale’ > from incompatible pointer type [-Wincompatible-pointer-types] > libswscale/swscale.h:207:5: note: expected ‘const uint8_t * const* {aka const > unsigned

Re: [libav-devel] [PATCH 1/3] build: Map -Wall compiler flag to -W3 for MSVC and -Wextra to -W4

2017-01-24 Thread Diego Biurrun
On Thu, Jan 19, 2017 at 11:43:38AM +0100, Diego Biurrun wrote: > These are more appropriate warning level equivalents. > --- > configure | 4 +++- > 1 file changed, 3 insertions(+), 1 deletion(-) ping the set Diego ___ libav-devel mailing list

Re: [libav-devel] [PATCH 05/11] mpc: Convert to the new bitstream reader

2017-01-24 Thread Luca Barbato
On 11/01/2017 18:10, Diego Biurrun wrote: > From: Alexandra Hájková > Sounds Ok. lu ___ libav-devel mailing list libav-devel@libav.org https://lists.libav.org/mailman/listinfo/libav-devel

Re: [libav-devel] [PATCH 07/11] wavpack: Convert to the new bitstream reader

2017-01-24 Thread Luca Barbato
On 11/01/2017 18:10, Diego Biurrun wrote: > From: Alexandra Hájková > > --- > libavcodec/wavpack.c | 92 > ++-- > 1 file changed, 46 insertions(+), 46 deletions(-) > Seems still ok.

Re: [libav-devel] [PATCH 1/4] vaapi_encode: Pass framerate parameters to driver

2017-01-24 Thread Anton Khirnov
Quoting Mark Thompson (2017-01-21 23:10:54) > On 13/01/17 11:07, Anton Khirnov wrote: > > Quoting Mark Thompson (2017-01-10 00:47:55) > >> --- > >> The i965 driver has been updated so that this works consistently. > >> > >> libavcodec/vaapi_encode.c | 14 ++ > >>