Re: [libav-devel] [PATCH] fix an issue on multichannel ADTS AAC with non-zero channel_config + PCE

2015-06-02 Thread Luca Barbato
On 03/06/15 07:40, nu774 wrote: > Let me explain on this patch. > > Issue: > -- > AAC decoder incorectly decode this sample (taken from Japanese DVB): > https://dl.dropboxusercontent.com/u/76042064/5.1ch_PCE_issue.aac > Every once in 0.5 sec or so, glitches are generated. It is audible, and >

Re: [libav-devel] [PATCH] lavf: add a format flag for separate carriers, and an event for detecting carrier presence.

2015-06-02 Thread Luca Barbato
On 25/05/15 10:30, John Högberg wrote: > AVFMT_SEPCARRIERS is a new flag denoting that the format has a separate > carrier > for each stream, e.g. a separate file, transport stream, or connection per > stream. > > AVSTREAM_EVENT_FLAG_CARRIER_PRESENT is a new event indicating the presence of > a g

Re: [libav-devel] [PATCH] fix an issue on multichannel ADTS AAC with non-zero channel_config + PCE

2015-06-02 Thread nu774
Let me explain on this patch. Issue: -- AAC decoder incorectly decode this sample (taken from Japanese DVB): https://dl.dropboxusercontent.com/u/76042064/5.1ch_PCE_issue.aac Every once in 0.5 sec or so, glitches are generated. It is audible, and also you can easily spot them on spectrogram o

Re: [libav-devel] [PATCH] lavf: add a format flag for separate carriers, and an event for detecting carrier presence.

2015-06-02 Thread John Högberg
John Högberg wrote: > Hi, > > libavformat currently doesn't provide any mechanism for detecting when a > substream is no longer present but the stream as a whole is otherwise OK. Your > best guess is to check when you last got any frames on it, which tends to work > for video and audio but not for

[libav-devel] [PATCH] fix an issue on multichannel ADTS AAC with non-zero channel_config + PCE

2015-06-02 Thread nu774
--- libavcodec/aacdec.c | 13 ++--- 1 file changed, 10 insertions(+), 3 deletions(-) diff --git a/libavcodec/aacdec.c b/libavcodec/aacdec.c index 7b306c9..e453372 100644 --- a/libavcodec/aacdec.c +++ b/libavcodec/aacdec.c @@ -443,12 +443,18 @@ static int output_configure(AACContext *ac,

[libav-devel] [PATCH] bink: Factorize bink put_pixel

2015-06-02 Thread Luca Barbato
And make sure to check INTER_BLOCK as had been fixed by Michael Niedermayer. Reported-By: Andreas Cadhalpun CC: libav-sta...@libav.org --- I'll stab bink.c a little more later once I'm awake =p libavcodec/bink.c | 55 +++ 1 file changed, 31 in

Re: [libav-devel] [PATCH] configure: Don't run "export $e" for an empty string

2015-06-02 Thread Luca Barbato
On 02/06/15 21:56, Martin Storsjö wrote: > This fixes the side effect of printing the whole environment, > if no --env parameter was provided. > --- > configure | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/configure b/configure > index 2458adb..1ed8956 100755 > --- a/c

[libav-devel] [PATCH] configure: Skip empty $env

2015-06-02 Thread Luca Barbato
--- Shame on me for not noticing before pushing. configure | 8 +--- 1 file changed, 5 insertions(+), 3 deletions(-) diff --git a/configure b/configure index 30d6f18..768d6a5 100755 --- a/configure +++ b/configure @@ -2542,9 +2542,11 @@ for opt do esac done -for e in "$env"; do -

Re: [libav-devel] [PATCH] configure: don't enable tls protocols if network is disabled

2015-06-02 Thread Martin Storsjö
On Tue, 2 Jun 2015, James Almer wrote: This was a regression introduced with d8ffb2055f0e0fcb5d025bab72eb19c2a886c125. Signed-off-by: James Almer --- configure | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/configure b/configure index 30d6f18..2458adb 100755 --- a/configu

[libav-devel] [PATCH] configure: Don't run "export $e" for an empty string

2015-06-02 Thread Martin Storsjö
This fixes the side effect of printing the whole environment, if no --env parameter was provided. --- configure | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/configure b/configure index 2458adb..1ed8956 100755 --- a/configure +++ b/configure @@ -2543,7 +2543,7 @@ for opt do

Re: [libav-devel] [RFC][WIP] Conversion of FFV1 specification to Markdown

2015-06-02 Thread Luca Barbato
On 02/06/15 18:36, Dave Rice wrote: > Hi all, > > As there is ongoing work on the FFV1 specification in support of a > standardization process via the IETF, Ashley Blewer and I (as part of > the PreForma project [1]) have drafted a translation of the current > FFV1 specification which is formatted

[libav-devel] [RFC][WIP] Conversion of FFV1 specification to Markdown

2015-06-02 Thread Dave Rice
Hi all, As there is ongoing work on the FFV1 specification in support of a standardization process via the IETF, Ashley Blewer and I (as part of the PreForma project [1]) have drafted a translation of the current FFV1 specification which is formatted in lyx [2] to markdown. Such a transition wa

[libav-devel] [PATCH] configure: don't enable tls protocols if network is disabled

2015-06-02 Thread James Almer
This was a regression introduced with d8ffb2055f0e0fcb5d025bab72eb19c2a886c125. Signed-off-by: James Almer --- configure | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/configure b/configure index 30d6f18..2458adb 100755 --- a/configure +++ b/configure @@ -2215,9 +2215,10 @

Re: [libav-devel] [PATCH 2/4] configure: Require LPDIRECT3DSURFACE9 for dxva2

2015-06-02 Thread James Almer
On 02/06/15 4:27 AM, Martin Storsjö wrote: > On Mon, 1 Jun 2015, James Almer wrote: > >> On 01/06/15 7:54 AM, Martin Storsjö wrote: >>> This fixes dxva2 detection (i.e. correctly realizes that it isn't >>> available) for WinRT, where dxva2api.h does exist, but these definitions >>> are omitted (wh

[libav-devel] [PATCH] Hap decoder and encoder

2015-06-02 Thread Vittorio Giovara
Signed-off-by: Vittorio Giovara --- In this version I add the size check and rename the option (from profile to format) to avoid aliasing the main profile option. Cheers, Vittorio Changelog | 1 + configure | 7 ++ doc/general.texi| 1 + libavcodec/M

[libav-devel] [PATCH] Introduce a TextureDSP module

2015-06-02 Thread Vittorio Giovara
This module implements generic texture decompression from different families (DXTC, RGTC, BCn) and texture compression DXTC 1, 3, and 5. Signed-off-by: Vittorio Giovara --- Updated with Justin's and James' comments (thanks). Vittorio configure | 2 + libavcodec/Makefile

Re: [libav-devel] [PATCH] configure: don't enable tls protocols if network is disabled

2015-06-02 Thread Vittorio Giovara
On Mon, Jun 1, 2015 at 9:14 PM, James Almer wrote: > This was a regression introduced with > d8ffb2055f0e0fcb5d025bab72eb19c2a886c125. > > Signed-off-by: James Almer > --- > configure | 2 ++ > 1 file changed, 2 insertions(+) > > diff --git a/configure b/configure > index cdc5a8d..a29cd38 10075

Re: [libav-devel] [PATCH 1/2] configure: Remove unnecessary tests for d3d11va

2015-06-02 Thread Hendrik Leppkes
On Tue, Jun 2, 2015 at 9:48 AM, Martin Storsjö wrote: > These are only necessary once/if avconv gets support for this hwaccel. > While that obviously is desireable, we don't have it yet, and they > currently only are a distraction. Technically the cobj macros are also used in the hwaccel code the

[libav-devel] [PATCH 1/2] configure: Remove unnecessary tests for d3d11va

2015-06-02 Thread Martin Storsjö
These are only necessary once/if avconv gets support for this hwaccel. While that obviously is desireable, we don't have it yet, and they currently only are a distraction. --- configure | 15 --- 1 file changed, 15 deletions(-) diff --git a/configure b/configure index af9d7a5..e8cd1f1

[libav-devel] [PATCH 2/2] configure: Check for DXVA2_ConfigPictureDecode instead of LPDIRECT3DSURFACE9

2015-06-02 Thread Martin Storsjö
Checking this struct is a bit more natural for this dependency. --- configure | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/configure b/configure index e8cd1f1..96a97df 100755 --- a/configure +++ b/configure @@ -1981,7 +1981,7 @@ zmbv_encoder_deps="zlib" # hardware acc

Re: [libav-devel] [PATCH] configure: don't enable tls protocols if network is disabled

2015-06-02 Thread Martin Storsjö
On Mon, 1 Jun 2015, James Almer wrote: This was a regression introduced with d8ffb2055f0e0fcb5d025bab72eb19c2a886c125. Signed-off-by: James Almer --- configure | 2 ++ 1 file changed, 2 insertions(+) diff --git a/configure b/configure index cdc5a8d..a29cd38 100755 --- a/configure +++ b/configu

Re: [libav-devel] [PATCH 2/4] configure: Require LPDIRECT3DSURFACE9 for dxva2

2015-06-02 Thread Martin Storsjö
On Mon, 1 Jun 2015, James Almer wrote: On 01/06/15 7:54 AM, Martin Storsjö wrote: This fixes dxva2 detection (i.e. correctly realizes that it isn't available) for WinRT, where dxva2api.h does exist, but these definitions are omitted (when targeting the API subsets). Ideally we should rather ch