Re: [libav-devel] [RFC] [PATCH] lavc: Drop exporting 2-pass encoding stats

2015-12-02 Thread Luca Barbato
On 03/12/15 01:53, Vittorio Giovara wrote: > I'm hesitant about this, this removal, and the the changes from global > to codec private (i have more in a branch) might pollute a little too > much the file. Maybe we could do a single entry that list all the > fields that got removed before release? >

Re: [libav-devel] [RFC] [PATCH] lavc: Drop exporting 2-pass encoding stats

2015-12-02 Thread Vittorio Giovara
On Wed, Dec 2, 2015 at 1:53 PM, Andreas Cadhalpun wrote: > On 30.11.2015 18:19, Vittorio Giovara wrote: >> These variables leaked from mpegvideoenc where are supposedly used for >> statistics. However they might very well be private, and, due to their >> absolute lack of documentation, they are ha

Re: [libav-devel] [PATCH] sgienc: Make sure to initialize skipped header portions

2015-12-02 Thread Luca Barbato
On 03/12/15 01:10, Vittorio Giovara wrote: > Fix fate tests with asan. Introduced during bytestream2 porting > (in revision 62cc8f4d79dad119e8efeaae080a58a8dcb1e89d). > > Signed-off-by: Vittorio Giovara > --- > Got suggested to use plain _byte functions. > Vittorio > > libavcodec/sgienc.c | 8 +

[libav-devel] [PATCH] sgienc: Make sure to initialize skipped header portions

2015-12-02 Thread Vittorio Giovara
Fix fate tests with asan. Introduced during bytestream2 porting (in revision 62cc8f4d79dad119e8efeaae080a58a8dcb1e89d). Signed-off-by: Vittorio Giovara --- Got suggested to use plain _byte functions. Vittorio libavcodec/sgienc.c | 8 +--- 1 file changed, 5 insertions(+), 3 deletions(-) dif

Re: [libav-devel] [PATCH] g723: Add missing header

2015-12-02 Thread Vittorio Giovara
On Wed, Dec 2, 2015 at 6:45 PM, Luca Barbato wrote: > Unbreak make check. > --- > > libavcodec/g723_1.h | 2 ++ > 1 file changed, 2 insertions(+) > > diff --git a/libavcodec/g723_1.h b/libavcodec/g723_1.h > index 3fd52df..166d897 100644 > --- a/libavcodec/g723_1.h > +++ b/libavcodec/g723_1.h > @@

[libav-devel] [PATCH] sgienc: Make sure to initialize skipped header portions

2015-12-02 Thread Vittorio Giovara
Fix fate tests with asan. Introduced during bytestream2 porting (in revision 62cc8f4d79dad119e8efeaae080a58a8dcb1e89d). Signed-off-by: Vittorio Giovara --- libavcodec/sgienc.c | 8 +--- 1 file changed, 5 insertions(+), 3 deletions(-) diff --git a/libavcodec/sgienc.c b/libavcodec/sgienc.c in

[libav-devel] [PATCH] g723: Add missing header

2015-12-02 Thread Luca Barbato
Unbreak make check. --- libavcodec/g723_1.h | 2 ++ 1 file changed, 2 insertions(+) diff --git a/libavcodec/g723_1.h b/libavcodec/g723_1.h index 3fd52df..166d897 100644 --- a/libavcodec/g723_1.h +++ b/libavcodec/g723_1.h @@ -30,6 +30,8 @@ #include +#include "libavutil/log.h" + #define SUBFR

[libav-devel] [PATCH] swfdec: support compressed swf

2015-12-02 Thread Vittorio Giovara
From: Clément Bœsch Signed-off-by: Vittorio Giovara --- configure| 1 + libavformat/swf.h| 13 libavformat/swfdec.c | 86 +--- 3 files changed, 96 insertions(+), 4 deletions(-) diff --git a/configure b/configure index 61

Re: [libav-devel] [PATCH] matroskadec: Fix sample_aspect_ratio for stereo matroska content

2015-12-02 Thread Luca Barbato
On 03/12/15 00:13, Vittorio Giovara wrote: > From: Aaron Colwell > > matroskaenc applies divisors to the display width/height when generating > stereo content. This patch adds the corresponding multipliers to matroskadec > so that the original sample aspect ratio can be recovered. > > Signed-off

[libav-devel] [PATCH] matroskadec: Fix sample_aspect_ratio for stereo matroska content

2015-12-02 Thread Vittorio Giovara
From: Aaron Colwell matroskaenc applies divisors to the display width/height when generating stereo content. This patch adds the corresponding multipliers to matroskadec so that the original sample aspect ratio can be recovered. Signed-off-by: wm4 Signed-off-by: Vittorio Giovara --- libavform

Re: [libav-devel] [PATCH] configure: Support MSYS2 mingw-w64 64bit

2015-12-02 Thread Hendrik Leppkes
On Wed, Dec 2, 2015 at 5:48 PM, Luca Barbato wrote: > --- > configure | 4 ++-- > 1 file changed, 2 insertions(+), 2 deletions(-) > > diff --git a/configure b/configure > index 53de905..942abfe 100755 > --- a/configure > +++ b/configure > @@ -2756,7 +2756,7 @@ fi > > exesuf() { > case $1 in

Re: [libav-devel] [RFC] [PATCH] lavc: Drop exporting 2-pass encoding stats

2015-12-02 Thread Andreas Cadhalpun
On 30.11.2015 18:19, Vittorio Giovara wrote: > These variables leaked from mpegvideoenc where are supposedly used for > statistics. However they might very well be private, and, due to their > absolute lack of documentation, they are hardly used in the wild. Despite > being write-only there are opt

Re: [libav-devel] [PATCH] lavc: Deprecate coder_type and its symbols

2015-12-02 Thread Andreas Cadhalpun
On 30.11.2015 18:17, Vittorio Giovara wrote: > Most option values are simply unused or ignored and in practice the > majory of codecs only need to check whether to enable rle or not. > > Add appropriate codec private options which better expose the allowed > features. > > Signed-off-by: Vittorio

[libav-devel] [PATCH] configure: Support MSYS2 mingw-w64 64bit

2015-12-02 Thread Luca Barbato
--- configure | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/configure b/configure index 53de905..942abfe 100755 --- a/configure +++ b/configure @@ -2756,7 +2756,7 @@ fi exesuf() { case $1 in -mingw32*|win32|win64|cygwin*|*-dos|freedos|opendos|os/2*|symbian

[libav-devel] [PATCH] configure: Replace `pr` since it is not provided by busybox

2015-12-02 Thread Luca Barbato
From: Kylie McClain While pr is a valid POSIX.1 command, its usage in configure is a little borderline and is possible to replace it with printf. Bug-Id: 913 Signed-off-by: Luca Barbato --- Now using more of the expected 80cols and with a space after ; configure | 8 ++-- 1 file changed

Re: [libav-devel] [PATCH] configure: Replace `pr` since it is not provided by busybox

2015-12-02 Thread Luca Barbato
On 02/12/15 13:47, Luca Barbato wrote: > On 02/12/15 12:33, Diego Biurrun wrote: >> On Thu, Nov 26, 2015 at 11:41:26AM +0100, Luca Barbato wrote: >>> From: Kylie McClain >>> >>> While pr is a valid POSIX.1 command, its usage in configure >>> is a little borderline and is possible to replace it wit

Re: [libav-devel] [PATCH] configure: Replace `pr` since it is not provided by busybox

2015-12-02 Thread Ganesh Ajjanagadde
On Wed, Dec 2, 2015 at 7:47 AM, Luca Barbato wrote: > On 02/12/15 12:33, Diego Biurrun wrote: >> On Thu, Nov 26, 2015 at 11:41:26AM +0100, Luca Barbato wrote: >>> From: Kylie McClain >>> >>> While pr is a valid POSIX.1 command, its usage in configure >>> is a little borderline and is possible to

Re: [libav-devel] [PATCH] configure: Replace `pr` since it is not provided by busybox

2015-12-02 Thread Luca Barbato
On 02/12/15 12:33, Diego Biurrun wrote: > On Thu, Nov 26, 2015 at 11:41:26AM +0100, Luca Barbato wrote: >> From: Kylie McClain >> >> While pr is a valid POSIX.1 command, its usage in configure >> is a little borderline and is possible to replace it with >> printf. >> >> --- >> >> The time spent by

Re: [libav-devel] [PATCH] configure: Replace `pr` since it is not provided by busybox

2015-12-02 Thread Luca Barbato
On 02/12/15 12:37, Diego Biurrun wrote: > On Wed, Dec 02, 2015 at 12:33:27PM +0100, Diego Biurrun wrote: >> On Thu, Nov 26, 2015 at 11:41:26AM +0100, Luca Barbato wrote: >>> --- a/configure >>> +++ b/configure >>> @@ -343,7 +343,11 @@ log(){ >>> >>> log_file(){ >>> log BEGIN $1 >>> -pr -n

Re: [libav-devel] [PATCH] configure: Replace `pr` since it is not provided by busybox

2015-12-02 Thread Diego Biurrun
On Wed, Dec 02, 2015 at 12:33:27PM +0100, Diego Biurrun wrote: > On Thu, Nov 26, 2015 at 11:41:26AM +0100, Luca Barbato wrote: > > --- a/configure > > +++ b/configure > > @@ -343,7 +343,11 @@ log(){ > > > > log_file(){ > > log BEGIN $1 > > -pr -n -t $1 >> $logfile > > +i=1 > > +w

Re: [libav-devel] [PATCH] configure: Replace `pr` since it is not provided by busybox

2015-12-02 Thread Diego Biurrun
On Thu, Nov 26, 2015 at 11:41:26AM +0100, Luca Barbato wrote: > From: Kylie McClain > > While pr is a valid POSIX.1 command, its usage in configure > is a little borderline and is possible to replace it with > printf. > > --- > > The time spent by configure is pretty much the same or less, much