Re: [libav-devel] [PATCH] lavfi: Add the alength audio length filter.

2012-12-10 Thread Justin Ruggles
On 12/10/2012 06:37 PM, Diego Biurrun wrote: > From: Alex Converse > > Signed-off-by: Diego Biurrun > --- > > - Added missing version bump. > - Corrected garbled filter description. > - style fixes > > Changelog|1 + > doc/filters.texi | 34 > libavfilte

Re: [libav-devel] [PATCH] Add a VideoDSPContext with core video DSP functions.

2012-12-10 Thread Luca Barbato
On 12/11/12 5:56 AM, Ronald S. Bultje wrote: From: "Ronald S. Bultje" Move some functions from dsputil. The idea is that videodsp contains functions that are useful for a large and varied set of video decoders. Currently, it contains emulated_edge_mc() and prefetch(). This allows vp8.c to be co

Re: [libav-devel] [PATCH] lavfi: Add the alength audio length filter.

2012-12-10 Thread Luca Barbato
On 12/11/12 12:37 AM, Diego Biurrun wrote: From: Alex Converse Signed-off-by: Diego Biurrun --- Looks ok to me. ___ libav-devel mailing list libav-devel@libav.org https://lists.libav.org/mailman/listinfo/libav-devel

[libav-devel] [PATCH] lavfi: Add the alength audio length filter.

2012-12-10 Thread Diego Biurrun
From: Alex Converse Signed-off-by: Diego Biurrun --- - Added missing version bump. - Corrected garbled filter description. - style fixes Changelog|1 + doc/filters.texi | 34 libavfilter/Makefile |1 + libavfilter/af_alength.c | 207

[libav-devel] [PATCH] lavfi: Add the alength audio length filter.

2012-12-10 Thread Alex Converse
--- Changelog|1 + doc/filters.texi | 33 +++ libavfilter/Makefile |1 + libavfilter/af_alength.c | 213 ++ libavfilter/allfilters.c |1 + 5 files changed, 249 insertions(+), 0 deletions(-) create mode 1006

Re: [libav-devel] [PATCH 1/2] swfdec: do better validation of tag length

2012-12-10 Thread Luca Barbato
On 12/10/12 7:58 PM, Justin Ruggles wrote: On 12/10/2012 01:27 PM, Luca Barbato wrote: On 12/10/2012 06:50 PM, Justin Ruggles wrote: Avoids trying to read a packet with 0 or negative size. Avoids a potential infinite loop due to seeking backwards. if (st->codec->codec_type

Re: [libav-devel] [PATCH] wmadec: if block_align is not set, treat the whole packet as a single block

2012-12-10 Thread Janne Grunau
On 2012-12-10 13:55:57 -0500, Justin Ruggles wrote: > On 12/10/2012 01:32 PM, Måns Rullgård wrote: > > Justin Ruggles writes: > > > >> On 12/10/2012 01:17 PM, Luca Barbato wrote: > >>> On 12/10/2012 05:49 PM, Justin Ruggles wrote: > From: Michael Niedermayer > > This avoids a pote

Re: [libav-devel] [PATCH] wmadec: if block_align is not set, treat the whole packet as a single block

2012-12-10 Thread Måns Rullgård
Justin Ruggles writes: > On 12/10/2012 01:32 PM, Måns Rullgård wrote: >> Justin Ruggles writes: >> >>> On 12/10/2012 01:17 PM, Luca Barbato wrote: On 12/10/2012 05:49 PM, Justin Ruggles wrote: > From: Michael Niedermayer > > This avoids a potential infinite loop due to not con

Re: [libav-devel] [PATCH] aacsbr: silence message for SBR extension "padding".

2012-12-10 Thread Alex Converse
On Mon, Dec 10, 2012 at 5:48 AM, Jindřich Makovička wrote: > Hi, > > this is a backport of a patch from FFMPEG, which silences error > messages for AAC streams padded with zeros. These messages are emitted > for each AAC packet and can easily make hundreds of megabytes per day > if logged. > > A s

Re: [libav-devel] [PATCH 1/2] swfdec: do better validation of tag length

2012-12-10 Thread Justin Ruggles
On 12/10/2012 01:27 PM, Luca Barbato wrote: > On 12/10/2012 06:50 PM, Justin Ruggles wrote: >> Avoids trying to read a packet with 0 or negative size. >> Avoids a potential infinite loop due to seeking backwards. > >> if (st->codec->codec_type == AVMEDIA_TYPE_VIDEO && st->id >> =

Re: [libav-devel] [PATCH] wmadec: if block_align is not set, treat the whole packet as a single block

2012-12-10 Thread Justin Ruggles
On 12/10/2012 01:32 PM, Måns Rullgård wrote: > Justin Ruggles writes: > >> On 12/10/2012 01:17 PM, Luca Barbato wrote: >>> On 12/10/2012 05:49 PM, Justin Ruggles wrote: From: Michael Niedermayer This avoids a potential infinite loop due to not consuming any packet data.

Re: [libav-devel] [PATCH] aacsbr: silence message for SBR extension "padding".

2012-12-10 Thread Luca Barbato
On 12/10/2012 04:28 PM, Jindřich Makovička wrote: > On Mon, Dec 10, 2012 at 3:07 PM, Diego Biurrun wrote: >> On Mon, Dec 10, 2012 at 02:48:28PM +0100, Jindřich Makovička wrote: >>> >>> Some files contain a few additional, all-0 bits. >>> Check for that case and don't print incorrect "not supported

Re: [libav-devel] [PATCH] wmadec: if block_align is not set, treat the whole packet as a single block

2012-12-10 Thread Måns Rullgård
Justin Ruggles writes: > On 12/10/2012 01:17 PM, Luca Barbato wrote: >> On 12/10/2012 05:49 PM, Justin Ruggles wrote: >>> From: Michael Niedermayer >>> >>> This avoids a potential infinite loop due to not consuming any packet data. >>> >>> Signed-off-by: Michael Niedermayer >>> Signed-off-by: J

Re: [libav-devel] [PATCH] wmadec: if block_align is not set, treat the whole packet as a single block

2012-12-10 Thread Luca Barbato
On 12/10/2012 07:24 PM, Justin Ruggles wrote: > On 12/10/2012 01:17 PM, Luca Barbato wrote: >> On 12/10/2012 05:49 PM, Justin Ruggles wrote: >>> From: Michael Niedermayer >>> >>> This avoids a potential infinite loop due to not consuming any packet data. >>> >>> Signed-off-by: Michael Niedermayer

Re: [libav-devel] [PATCH 1/2] swfdec: do better validation of tag length

2012-12-10 Thread Luca Barbato
On 12/10/2012 06:50 PM, Justin Ruggles wrote: > Avoids trying to read a packet with 0 or negative size. > Avoids a potential infinite loop due to seeking backwards. > if (st->codec->codec_type == AVMEDIA_TYPE_VIDEO && st->id == > ch_id) { > frame = avio_rl16(

Re: [libav-devel] [PATCH] wmadec: if block_align is not set, treat the whole packet as a single block

2012-12-10 Thread Justin Ruggles
On 12/10/2012 01:17 PM, Luca Barbato wrote: > On 12/10/2012 05:49 PM, Justin Ruggles wrote: >> From: Michael Niedermayer >> >> This avoids a potential infinite loop due to not consuming any packet data. >> >> Signed-off-by: Michael Niedermayer >> Signed-off-by: Justin Ruggles >> >> CC:libav-sta.

Re: [libav-devel] [PATCH 2/2] Make LCOAL_ALIGNED syntactically similar on all systems

2012-12-10 Thread Luca Barbato
On 12/10/2012 06:22 PM, Måns Rullgård wrote: > It's been a few days of silence... > > I've fixed the typo in the subject so no need to point that out. Could someone try on msvc just in case? I'm sure all the other compilers got tested. Beside that feel free to commit anytime. ___

Re: [libav-devel] [PATCH] wmadec: if block_align is not set, treat the whole packet as a single block

2012-12-10 Thread Luca Barbato
On 12/10/2012 05:49 PM, Justin Ruggles wrote: > From: Michael Niedermayer > > This avoids a potential infinite loop due to not consuming any packet data. > > Signed-off-by: Michael Niedermayer > Signed-off-by: Justin Ruggles > > CC:libav-sta...@libav.org > --- > libavcodec/wmadec.c |5 ++

Re: [libav-devel] [PATCH] aacsbr: silence message for SBR extension "padding".

2012-12-10 Thread Jindřich Makovička
On Mon, Dec 10, 2012 at 6:29 PM, Kieran Kunhya wrote: > Off-topic but that transport stream is incredibly screwed up. Or is it just > coincidentally cut so that it contains only part of a GOP? Yes, I captured only a few seconds and missed the key frame, but I didn't bother to take a new sample as

[libav-devel] [PATCH 2/2] swfdec: cosmetics: fix indentation

2012-12-10 Thread Justin Ruggles
--- libavformat/swfdec.c | 32 1 files changed, 16 insertions(+), 16 deletions(-) diff --git a/libavformat/swfdec.c b/libavformat/swfdec.c index d8a9852..9ddee9d 100644 --- a/libavformat/swfdec.c +++ b/libavformat/swfdec.c @@ -180,22 +180,22 @@ static int swf_re

[libav-devel] [PATCH 1/2] swfdec: do better validation of tag length

2012-12-10 Thread Justin Ruggles
Avoids trying to read a packet with 0 or negative size. Avoids a potential infinite loop due to seeking backwards. --- libavformat/swfdec.c | 22 +++--- 1 files changed, 19 insertions(+), 3 deletions(-) diff --git a/libavformat/swfdec.c b/libavformat/swfdec.c index f105c14..d8a9

Re: [libav-devel] [PATCH] aacsbr: silence message for SBR extension "padding".

2012-12-10 Thread Kieran Kunhya
Off-topic but that transport stream is incredibly screwed up. Or is it just coincidentally cut so that it contains only part of a GOP? ___ libav-devel mailing list libav-devel@libav.org https://lists.libav.org/mailman/listinfo/libav-devel

Re: [libav-devel] [PATCH 2/2] Make LCOAL_ALIGNED syntactically similar on all systems

2012-12-10 Thread Måns Rullgård
Janne Grunau writes: > On 2012-12-07 23:03:07 +, Måns Rullgård wrote: >> This changes the LOCAL_ALIGNED definition on systems where >> DECLARE_ALIGNED is used such it matches the manual alignment >> case, ensuring invalid use will not compile on x86 only to >> fail on everything else. >> >>

[libav-devel] [PATCH] wmadec: if block_align is not set, treat the whole packet as a single block

2012-12-10 Thread Justin Ruggles
From: Michael Niedermayer This avoids a potential infinite loop due to not consuming any packet data. Signed-off-by: Michael Niedermayer Signed-off-by: Justin Ruggles CC:libav-sta...@libav.org --- libavcodec/wmadec.c |5 +++-- 1 files changed, 3 insertions(+), 2 deletions(-) diff --git

Re: [libav-devel] Fwd: Bug#695593: libavcodec53: Transcoding wmav2 file leaves ffmpeg in infinite loop

2012-12-10 Thread Justin Ruggles
On 12/10/2012 10:44 AM, Reinhard Tartler wrote: > Hi, > > could someone please look at the attached patch? From a quick look on > gitweb, it looks like it would apply to current libav master. > If it does, please add a CC: libav-stable @ to the commit message. Thanks! > > Reinhard > > --

[libav-devel] Fwd: Bug#695593: libavcodec53: Transcoding wmav2 file leaves ffmpeg in infinite loop

2012-12-10 Thread Reinhard Tartler
Hi, could someone please look at the attached patch? From a quick look on gitweb, it looks like it would apply to current libav master. If it does, please add a CC: libav-stable @ to the commit message. Thanks! Reinhard -- Forwarded message -- From: Laurent Bigonville Date: Mon,

Re: [libav-devel] [PATCH] aacsbr: silence message for SBR extension "padding".

2012-12-10 Thread Måns Rullgård
Jindřich Makovička writes: > On Mon, Dec 10, 2012 at 4:18 PM, Måns Rullgård wrote: >> Jindřich Makovička writes: >> >>> On Mon, Dec 10, 2012 at 3:00 PM, Måns Rullgård wrote: Jindřich Makovička writes: > diff --git a/libavcodec/aacsbr.c b/libavcodec/aacsbr.c > index add9f18..

Re: [libav-devel] [PATCH] aacsbr: silence message for SBR extension "padding".

2012-12-10 Thread Jindřich Makovička
On Mon, Dec 10, 2012 at 4:18 PM, Måns Rullgård wrote: > Jindřich Makovička writes: > >> On Mon, Dec 10, 2012 at 3:00 PM, Måns Rullgård wrote: >>> Jindřich Makovička writes: >>> diff --git a/libavcodec/aacsbr.c b/libavcodec/aacsbr.c index add9f18..dac2372 100644 --- a/libavcodec/a

Re: [libav-devel] [PATCH] aacsbr: silence message for SBR extension "padding".

2012-12-10 Thread Jindřich Makovička
On Mon, Dec 10, 2012 at 3:07 PM, Diego Biurrun wrote: > On Mon, Dec 10, 2012 at 02:48:28PM +0100, Jindřich Makovička wrote: >> >> Some files contain a few additional, all-0 bits. >> Check for that case and don't print incorrect "not supported" >> message. >> Fixes trac issue #836. > > We don't use

Re: [libav-devel] [PATCH] aacsbr: silence message for SBR extension "padding".

2012-12-10 Thread Måns Rullgård
Jindřich Makovička writes: > On Mon, Dec 10, 2012 at 3:00 PM, Måns Rullgård wrote: >> Jindřich Makovička writes: >> >>> diff --git a/libavcodec/aacsbr.c b/libavcodec/aacsbr.c >>> index add9f18..dac2372 100644 >>> --- a/libavcodec/aacsbr.c >>> +++ b/libavcodec/aacsbr.c >>> @@ -926,7 +926,9 @@ st

Re: [libav-devel] [PATCH] aacsbr: silence message for SBR extension "padding".

2012-12-10 Thread Jindřich Makovička
On Mon, Dec 10, 2012 at 3:00 PM, Måns Rullgård wrote: > Jindřich Makovička writes: > >> diff --git a/libavcodec/aacsbr.c b/libavcodec/aacsbr.c >> index add9f18..dac2372 100644 >> --- a/libavcodec/aacsbr.c >> +++ b/libavcodec/aacsbr.c >> @@ -926,7 +926,9 @@ static void read_sbr_extension(AACContex

Re: [libav-devel] [PATCH] build: fix 'clean' target

2012-12-10 Thread Diego Biurrun
On Mon, Dec 10, 2012 at 02:56:08PM +, Mans Rullgard wrote: > This fixes removal of TOOLS as well as HOSTPROGS declared in the > top-level Makefile. The clean target in common.mak needs to be > eval'd since the variables used within are reset for each library. Thanks, I had put fixing this on

[libav-devel] [PATCH] build: fix 'clean' target

2012-12-10 Thread Mans Rullgard
This fixes removal of TOOLS as well as HOSTPROGS declared in the top-level Makefile. The clean target in common.mak needs to be eval'd since the variables used within are reset for each library. Signed-off-by: Mans Rullgard --- Makefile| 1 - common.mak | 6 ++ library.mak | 2 +- 3 fi

Re: [libav-devel] [PATCH] aacsbr: silence message for SBR extension "padding".

2012-12-10 Thread Diego Biurrun
On Mon, Dec 10, 2012 at 02:48:28PM +0100, Jindřich Makovička wrote: > > Some files contain a few additional, all-0 bits. > Check for that case and don't print incorrect "not supported" > message. > Fixes trac issue #836. We don't use trac. > --- a/libavcodec/aacsbr.c > +++ b/libavcodec/aacsbr.c

Re: [libav-devel] [PATCH] aacsbr: silence message for SBR extension "padding".

2012-12-10 Thread Måns Rullgård
Jindřich Makovička writes: > diff --git a/libavcodec/aacsbr.c b/libavcodec/aacsbr.c > index add9f18..dac2372 100644 > --- a/libavcodec/aacsbr.c > +++ b/libavcodec/aacsbr.c > @@ -926,7 +926,9 @@ static void read_sbr_extension(AACContext *ac, > SpectralBandReplication *sbr, > } >

[libav-devel] [PATCH] aacsbr: silence message for SBR extension "padding".

2012-12-10 Thread Jindřich Makovička
Hi, this is a backport of a patch from FFMPEG, which silences error messages for AAC streams padded with zeros. These messages are emitted for each AAC packet and can easily make hundreds of megabytes per day if logged. A sample stream produced by an encoder from Cisco is available at geraldine.f

Re: [libav-devel] duplicating frames in avconv

2012-12-10 Thread Anton Khirnov
On Mon, 10 Dec 2012 09:48:11 +0100, Luca Barbato wrote: > On 12/10/12 6:35 AM, Anton Khirnov wrote: > > > > On Tue, 04 Dec 2012 14:37:33 +0100, Arash Shafiei wrote: > >> Hi, > >> > >> When we use avconv sometimes it duplicates frames to achieve a specified > >> frame-rate. > >> > >> I would like

Re: [libav-devel] duplicating frames in avconv

2012-12-10 Thread Luca Barbato
On 12/10/12 6:35 AM, Anton Khirnov wrote: On Tue, 04 Dec 2012 14:37:33 +0100, Arash Shafiei wrote: Hi, When we use avconv sometimes it duplicates frames to achieve a specified frame-rate. I would like to know at which layer this is implemented. Is this internal to libavcodec which automatica