Re: [libav-devel] [PATCH 1/4] libavcodec: Prefer x264 over openh264 if both are available

2015-01-07 Thread Timothy Gu
On Wed Jan 07 2015 at 4:42:03 PM Luca Barbato wrote: > On 08/01/15 00:27, Vittorio Giovara wrote: > > Hadn't thought of that, yes twolame should definitely get priority > > > > Twolame fdk-aac and x264 should go in a group with higher priority, then > native, then external lower priority. > I co

Re: [libav-devel] [PATCH 2/4] doc: Add documentation about the OpenH264 encoder wrapper

2015-01-07 Thread Timothy Gu
Hi, On Wed Jan 07 2015 at 2:08:19 PM Martin Storsjö wrote: > --- > doc/general.texi | 10 +- > 1 file changed, 9 insertions(+), 1 deletion(-) > Any chance of a full documentation in doc/encoders.texi? > > diff --git a/doc/general.texi b/doc/general.texi > index ae59941..31972ee 10064

Re: [libav-devel] [PATCH 1/4] libavcodec: Prefer x264 over openh264 if both are available

2015-01-07 Thread Luca Barbato
On 08/01/15 00:27, Vittorio Giovara wrote: > Hadn't thought of that, yes twolame should definitely get priority > Twolame fdk-aac and x264 should go in a group with higher priority, then native, then external lower priority. Does it make sense to you? lu ___

Re: [libav-devel] [PATCH 1/4] libavcodec: Prefer x264 over openh264 if both are available

2015-01-07 Thread Vittorio Giovara
On Thu, Jan 8, 2015 at 12:18 AM, Martin Storsjö wrote: > On Thu, 8 Jan 2015, Vittorio Giovara wrote: > >> On Wed, Jan 7, 2015 at 11:41 PM, Luca Barbato wrote: >>> >>> On 07/01/15 23:08, Martin Storsjö wrote: From: Michael Niedermayer --- libavcodec/allcodecs.c | 5 -

Re: [libav-devel] [PATCH 3/4] libopenh264enc: Fix a typo and some nitpicks

2015-01-07 Thread Vittorio Giovara
On Wed, Jan 7, 2015 at 11:08 PM, Martin Storsjö wrote: > Also move the .long_name entry to below the .name entry. > --- > configure | 2 +- > libavcodec/libopenh264enc.c | 4 ++-- > 2 files changed, 3 insertions(+), 3 deletions(-) > > diff --git a/configure b/configure > index 0

Re: [libav-devel] [PATCH 2/4] doc: Add documentation about the OpenH264 encoder wrapper

2015-01-07 Thread Vittorio Giovara
On Wed, Jan 7, 2015 at 11:08 PM, Martin Storsjö wrote: > --- > doc/general.texi | 10 +- > 1 file changed, 9 insertions(+), 1 deletion(-) > > diff --git a/doc/general.texi b/doc/general.texi > index ae59941..31972ee 100644 > --- a/doc/general.texi > +++ b/doc/general.texi > @@ -95,6 +95,1

Re: [libav-devel] [PATCH 1/4] libavcodec: Prefer x264 over openh264 if both are available

2015-01-07 Thread Martin Storsjö
On Thu, 8 Jan 2015, Vittorio Giovara wrote: On Wed, Jan 7, 2015 at 11:41 PM, Luca Barbato wrote: On 07/01/15 23:08, Martin Storsjö wrote: From: Michael Niedermayer --- libavcodec/allcodecs.c | 5 - 1 file changed, 4 insertions(+), 1 deletion(-) diff --git a/libavcodec/allcodecs.c b/li

Re: [libav-devel] [PATCH 1/4] libavcodec: Prefer x264 over openh264 if both are available

2015-01-07 Thread Vittorio Giovara
On Wed, Jan 7, 2015 at 11:41 PM, Luca Barbato wrote: > On 07/01/15 23:08, Martin Storsjö wrote: >> From: Michael Niedermayer >> >> --- >> libavcodec/allcodecs.c | 5 - >> 1 file changed, 4 insertions(+), 1 deletion(-) >> >> diff --git a/libavcodec/allcodecs.c b/libavcodec/allcodecs.c >> inde

Re: [libav-devel] [PATCH 1/4] libavcodec: Prefer x264 over openh264 if both are available

2015-01-07 Thread Martin Storsjö
On Wed, 7 Jan 2015, Luca Barbato wrote: On 07/01/15 23:08, Martin Storsjö wrote: From: Michael Niedermayer --- libavcodec/allcodecs.c | 5 - 1 file changed, 4 insertions(+), 1 deletion(-) diff --git a/libavcodec/allcodecs.c b/libavcodec/allcodecs.c index 0d46afe..cb5d846 100644 --- a/li

Re: [libav-devel] [PATCH 1/4] libavcodec: Prefer x264 over openh264 if both are available

2015-01-07 Thread Luca Barbato
On 07/01/15 23:08, Martin Storsjö wrote: > From: Michael Niedermayer > > --- > libavcodec/allcodecs.c | 5 - > 1 file changed, 4 insertions(+), 1 deletion(-) > > diff --git a/libavcodec/allcodecs.c b/libavcodec/allcodecs.c > index 0d46afe..cb5d846 100644 > --- a/libavcodec/allcodecs.c > +++

[libav-devel] [PATCH 2/4] doc: Add documentation about the OpenH264 encoder wrapper

2015-01-07 Thread Martin Storsjö
--- doc/general.texi | 10 +- 1 file changed, 9 insertions(+), 1 deletion(-) diff --git a/doc/general.texi b/doc/general.texi index ae59941..31972ee 100644 --- a/doc/general.texi +++ b/doc/general.texi @@ -95,6 +95,14 @@ Go to @url{http://www.wavpack.com/} and follow the instructions for

[libav-devel] [PATCH 1/4] libavcodec: Prefer x264 over openh264 if both are available

2015-01-07 Thread Martin Storsjö
From: Michael Niedermayer --- libavcodec/allcodecs.c | 5 - 1 file changed, 4 insertions(+), 1 deletion(-) diff --git a/libavcodec/allcodecs.c b/libavcodec/allcodecs.c index 0d46afe..cb5d846 100644 --- a/libavcodec/allcodecs.c +++ b/libavcodec/allcodecs.c @@ -442,7 +442,6 @@ void avcodec_re

[libav-devel] [PATCH 4/4] libopenh264enc: Move a declaration of a variable into an ifdef

2015-01-07 Thread Martin Storsjö
This avoids needing an attribute for silencing warnings about it being unused. --- libavcodec/libopenh264enc.c | 5 ++--- 1 file changed, 2 insertions(+), 3 deletions(-) diff --git a/libavcodec/libopenh264enc.c b/libavcodec/libopenh264enc.c index 4110426..6c80f41 100644 --- a/libavcodec/libopenh2

[libav-devel] [PATCH 3/4] libopenh264enc: Fix a typo and some nitpicks

2015-01-07 Thread Martin Storsjö
Also move the .long_name entry to below the .name entry. --- configure | 2 +- libavcodec/libopenh264enc.c | 4 ++-- 2 files changed, 3 insertions(+), 3 deletions(-) diff --git a/configure b/configure index 09cb30b..5085333 100755 --- a/configure +++ b/configure @@ -193,7 +193,7

Re: [libav-devel] [PATCH] libfdk-aacdec: Make sure decoding doesn't add any extra delay in the latest version of fdk-aac

2015-01-07 Thread Luca Barbato
On 07/01/15 22:53, Martin Storsjö wrote: > The latest version added support for a new option for enabling > a signal level limiter, which adds some extra delay. In fdk-aac, this > is enabled by default, but disable it by default here since we'd rather > have zero-delay decoding. > --- > Later, one

[libav-devel] [PATCH] libfdk-aacdec: Make sure decoding doesn't add any extra delay in the latest version of fdk-aac

2015-01-07 Thread Martin Storsjö
The latest version added support for a new option for enabling a signal level limiter, which adds some extra delay. In fdk-aac, this is enabled by default, but disable it by default here since we'd rather have zero-delay decoding. --- Later, one could also check info->outputDelay in get_stream_info

Re: [libav-devel] [PATCH 1/3] lavc: Add DSS_SP decoder

2015-01-07 Thread Diego Biurrun
On Fri, Dec 19, 2014 at 12:59:15AM +0100, Vittorio Giovara wrote: > > Changelog | 1 + > libavcodec/Makefile | 1 + > libavcodec/allcodecs.c | 1 + > libavcodec/avcodec.h| 1 + > libavcodec/codec_desc.c | 7 + > libavcodec/dss_sp.c | 791 > +++

Re: [libav-devel] [PATCH 2/3] lavf: Add DSS demuxer

2015-01-07 Thread Diego Biurrun
On Fri, Dec 19, 2014 at 12:59:16AM +0100, Vittorio Giovara wrote: > --- > This version fixes the stack corruption when parsing the date string. > Vittorio > > Changelog| 2 +- > libavformat/Makefile | 1 + > libavformat/allformats.c | 1 + > libavformat/dss.c| 34

Re: [libav-devel] [PATCH] configure: Remap -L to -libpath for msvc

2015-01-07 Thread Diego Biurrun
On Sat, Jan 03, 2015 at 12:57:55AM +0200, Martin Storsjö wrote: > This allows using libraries that are detected via pkg-config with > msvc. (The libraries themselves may have to be built with MSVC > though.) > --- > configure | 1 + > 1 file changed, 1 insertion(+) OK I'd uppercase MSVC everywhe

Re: [libav-devel] [PATCH] libavcodec: Add an OpenH264 encoder wrapper

2015-01-07 Thread Diego Biurrun
On Sun, Jan 04, 2015 at 08:53:02PM +0200, Martin Storsjö wrote: > Compared to existing, common opensource H264 encoders, this can be open source H.264 > --- > Changelog | 1 + > configure | 4 + > libavcodec/Makefile | 1 + > libavcodec/allcodecs

Re: [libav-devel] [PATCH] arm: NEON optimized FIR audio resampling

2015-01-07 Thread Diego Biurrun
On Sun, Dec 28, 2014 at 04:50:02PM +0100, Peter Meerwald wrote: > > diff --git a/libavresample/arm/Makefile b/libavresample/arm/Makefile > index 60f3f6d..a7c2d22 100644 > --- a/libavresample/arm/Makefile > +++ b/libavresample/arm/Makefile > @@ -1,5 +1,7 @@ > -OBJS += arm/audio_convert_init.o

Re: [libav-devel] [PATCH] xsub: Support DXSA subtitles

2015-01-07 Thread Vittorio Giovara
On Wed, Jan 7, 2015 at 12:35 PM, Luca Barbato wrote: > From: Alexandre Colucci > > These have a DXSA tag and contain alpha in addition to > color values for palette. > > Signed-off-by: Jean-Baptiste Kempf > Signed-off-by: Luca Barbato > --- > > The bound check should be updated (otherwise you c

Re: [libav-devel] [PATCH] xsub: Support DXSA subtitles

2015-01-07 Thread Jean-Baptiste Kempf
On 07 Jan, Luca Barbato wrote : > On 07/01/15 12:43, Jean-Baptiste Kempf wrote: > >On 07 Jan, Luca Barbato wrote : > >>The bound check should be updated (otherwise you can craft evil > >>packets that read over the buffer). > > > >Sure. > > > >>Having branches in loops isn't that nice. > > > >Tbh, I

Re: [libav-devel] [PATCH] xsub: Support DXSA subtitles

2015-01-07 Thread Luca Barbato
On 07/01/15 12:43, Jean-Baptiste Kempf wrote: On 07 Jan, Luca Barbato wrote : The bound check should be updated (otherwise you can craft evil packets that read over the buffer). Sure. Having branches in loops isn't that nice. Tbh, I don't see the point. For me the code is less readable wit

Re: [libav-devel] [PATCH] xsub: Support DXSA subtitles

2015-01-07 Thread Jean-Baptiste Kempf
On 07 Jan, Luca Barbato wrote : > The bound check should be updated (otherwise you can craft evil > packets that read over the buffer). Sure. > Having branches in loops isn't that nice. Tbh, I don't see the point. For me the code is less readable with this split. > For the rest looks nice to me

[libav-devel] [PATCH] xsub: Support DXSA subtitles

2015-01-07 Thread Luca Barbato
From: Alexandre Colucci These have a DXSA tag and contain alpha in addition to color values for palette. Signed-off-by: Jean-Baptiste Kempf Signed-off-by: Luca Barbato --- The bound check should be updated (otherwise you can craft evil packets that read over the buffer). Having branches in l

[libav-devel] [PATCH] Fix decoding of DXSA subtitles.

2015-01-07 Thread Jean-Baptiste Kempf
From: Alexandre Colucci These have a DXSA tag and contain alpha in addition to color values for palette. Signed-off-by: Jean-Baptiste Kempf --- libavcodec/xsubdec.c | 5 +++-- 1 file changed, 3 insertions(+), 2 deletions(-) diff --git a/libavcodec/xsubdec.c b/libavcodec/xsubdec.c index 3d8597

Re: [libav-devel] [PATCH 3/9] Increase the line-height of the page header caption

2015-01-07 Thread Luca Barbato
On 07/01/15 03:29, Timothy Gu wrote: This makes the page a lot more comfortable to look at. --- htdocs/css/bootstrap-theme.css | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/htdocs/css/bootstrap-theme.css b/htdocs/css/bootstrap-theme.css index 8ed8b74..a08f304 100644 --- a

Re: [libav-devel] [PATCH 1/9] Remove sourceMappingURL in CSS

2015-01-07 Thread Luca Barbato
On 07/01/15 03:29, Timothy Gu wrote: We are not using minified CSS so this is not needed. Plus they generate noise on JavaScript console. --- htdocs/css/bootstrap-theme.css | 2 -- htdocs/css/bootstrap.css | 1 - 2 files changed, 3 deletions(-) diff --git a/htdocs/css/bootstrap-theme.c

Re: [libav-devel] [PATCH 2/9] Remove width setting in .footer

2015-01-07 Thread Luca Barbato
On 07/01/15 03:29, Timothy Gu wrote: This is causing the page to have a x-scrollbar, and the page looks fine without it. --- htdocs/css/bootstrap-theme.css | 2 -- 1 file changed, 2 deletions(-) diff --git a/htdocs/css/bootstrap-theme.css b/htdocs/css/bootstrap-theme.css index db3ebfa..8ed8b7

Re: [libav-devel] [PATCH 4/9] Remove default.css

2015-01-07 Thread Luca Barbato
On 07/01/15 03:29, Timothy Gu wrote: It's not used any more. --- htdocs/default.css | 173 - 1 file changed, 173 deletions(-) delete mode 100644 htdocs/default.css Sure (again depends if root deploys the new website within the week or no

Re: [libav-devel] [PATCH 6/9] Fix footer badge alignment

2015-01-07 Thread Luca Barbato
On 07/01/15 03:29, Timothy Gu wrote: Also convert to unordered list which is better fit for this purpose. Might be a better fit. I'd roll it in the new website though. lu ___ libav-devel mailing list libav-devel@libav.org https://lists.libav.org/ma

Re: [libav-devel] [PATCH 5/9] Remove obsolete `xmlns` attribute

2015-01-07 Thread Luca Barbato
On 07/01/15 03:29, Timothy Gu wrote: This is HTML5, not XHTML. --- src/template_head1 | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/src/template_head1 b/src/template_head1 index 1e5c8a8..adc03d5 100644 --- a/src/template_head1 +++ b/src/template_head1 @@ -1,5 +1,5 @@

Re: [libav-devel] [PATCH 7/9] Use for navigation bar

2015-01-07 Thread Luca Barbato
On 07/01/15 03:29, Timothy Gu wrote: The navigation role is not allowed in . --- src/template_head2 | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/src/template_head2 b/src/template_head2 index fa85166..3195d86 100644 --- a/src/template_head2 +++ b/src/template_head2 @@

Re: [libav-devel] [PATCH 8/9] Use unordered list for the latest release panel

2015-01-07 Thread Luca Barbato
On 07/01/15 03:29, Timothy Gu wrote: That's what unordered list is for, and how the Bootstrap example is like. Also removes clearly unintentional/copy/pasted " We are about to migrate the website so this change might be undone or replaced. I'm not 100% sure an unordered list is the best for st

Re: [libav-devel] [PATCH 9/9] Remove accidentally added ``

2015-01-07 Thread Luca Barbato
On 07/01/15 03:29, Timothy Gu wrote: --- src/news | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/src/news b/src/news index 86dadfb..753e516 100644 --- a/src/news +++ b/src/news @@ -14,7 +14,7 @@ and share their patches against our release branches. See the -http://gi