Re: [libav-devel] [PATCH 04/11] lavfi: clarify the context of a comment in avfilter_default_get_video_buffer()

2011-09-20 Thread Anton Khirnov
On Sun, 18 Sep 2011 09:43:27 -0700, Luca Barbato wrote: > On 9/18/11 7:02 AM, Anton Khirnov wrote: > > From: Stefano Sabatini > > > > The comment is meant to be about the align parameter. > > > > Signed-off-by: Anton Khirnov > > --- > > libavfilter/defaults.c |2 +- > > 1 files changed, 1

Re: [libav-devel] [PATCH] swfdec: Add support for sample_rate_code 0 (5512 Hz)

2011-09-20 Thread Anton Khirnov
On Tue, 20 Sep 2011 18:42:51 -0700, Alex Converse wrote: > --- > libavformat/swfdec.c |5 +++-- > 1 files changed, 3 insertions(+), 2 deletions(-) > > diff --git a/libavformat/swfdec.c b/libavformat/swfdec.c > index 59977a9..502647f 100644 > --- a/libavformat/swfdec.c > +++ b/libavformat/s

Re: [libav-devel] [PATCH 0/3] Support for encrypted OMA files

2011-09-20 Thread Anton Khirnov
On Fri, 2 Sep 2011 08:20:57 +0200, David Goldwich wrote: > This patch series adds support for encrypted OMA files. Patch #1 > modifies the DES library to support CBC-MAC calculation more easily. > Patch #2 adds support for binary or mixed-type tags in the ID3v2 > parser. Patch #3 adds the decry

[libav-devel] [PATCH] swfdec: Add support for sample_rate_code 0 (5512 Hz)

2011-09-20 Thread Alex Converse
--- libavformat/swfdec.c |5 +++-- 1 files changed, 3 insertions(+), 2 deletions(-) diff --git a/libavformat/swfdec.c b/libavformat/swfdec.c index 59977a9..502647f 100644 --- a/libavformat/swfdec.c +++ b/libavformat/swfdec.c @@ -136,8 +136,9 @@ static int swf_read_packet(AVFormatContext *s, A

[libav-devel] Concurrent ProRes decoder

2011-09-20 Thread Maxim
Hi crews, Yes, it's no mistake! What you see is just another code implementing ProRes decoder. Now I'm going to explain that confusion. Writing a ProRes compatible decoder began as team project in 2010. During this period, I created an internal documentation on the coding algorithm (entropy codin

Re: [libav-devel] [PATCH 7/8] Fix js_vlc_bits value validation when joint stereo is used in cook decoder.

2011-09-20 Thread Laurent Aimar
On Mon, Sep 19, 2011 at 07:50:09PM +0200, Laurent Aimar wrote: > On Sat, Sep 17, 2011 at 04:56:35PM +0200, fen...@elivagar.org wrote: > > From: Laurent Aimar > > > > --- > > libavcodec/cook.c |5 +++-- > > 1 files changed, 3 insertions(+), 2 deletions(-) > > > > diff --git a/libavcodec/cook

Re: [libav-devel] [PATCH] fate.sh: add support for building arbitrary targets

2011-09-20 Thread Måns Rullgård
Diego Biurrun writes: > On Tue, Sep 20, 2011 at 05:23:46PM +0100, Måns Rullgård wrote: >> Diego Biurrun writes: >> >> > Sometimes it can be useful to build not just the default target, but some >> > different target or a list of targets in addition to the default one. >> > If the make_targets v

Re: [libav-devel] [PATCH] fate.sh: add support for building arbitrary targets

2011-09-20 Thread Diego Biurrun
On Tue, Sep 20, 2011 at 05:23:46PM +0100, Måns Rullgård wrote: > Diego Biurrun writes: > > > Sometimes it can be useful to build not just the default target, but some > > different target or a list of targets in addition to the default one. > > If the make_targets variable is set, that list of ta

Re: [libav-devel] [PATCH] fate.sh: add support for building arbitrary targets

2011-09-20 Thread Måns Rullgård
Diego Biurrun writes: > Sometimes it can be useful to build not just the default target, but some > different target or a list of targets in addition to the default one. > If the make_targets variable is set, that list of targets is built, > otherwise just the default target is built as usual. >

Re: [libav-devel] [PATCH 1/5] fate.sh: whitespace cosmetics

2011-09-20 Thread Diego Biurrun
On Tue, Sep 20, 2011 at 04:35:53PM +0100, Måns Rullgård wrote: > Diego Biurrun writes: > > > On Tue, Sep 20, 2011 at 04:17:44PM +0100, Måns Rullgård wrote: > >> Måns Rullgård writes: > >> > >> > Diego Biurrun writes: > >> > > >> >> --- a/tests/fate.sh > >> >> +++ b/tests/fate.sh > >> >> @@ -21

[libav-devel] [PATCH] fate.sh: add support for building arbitrary targets

2011-09-20 Thread Diego Biurrun
Sometimes it can be useful to build not just the default target, but some different target or a list of targets in addition to the default one. If the make_targets variable is set, that list of targets is built, otherwise just the default target is built as usual. --- tests/fate.sh |2 +- 1 fi

Re: [libav-devel] [PATCH 1/5] fate.sh: whitespace cosmetics

2011-09-20 Thread Måns Rullgård
Diego Biurrun writes: > On Tue, Sep 20, 2011 at 04:17:44PM +0100, Måns Rullgård wrote: >> Måns Rullgård writes: >> >> > Diego Biurrun writes: >> > >> >> --- a/tests/fate.sh >> >> +++ b/tests/fate.sh >> >> @@ -21,7 +21,7 @@ test -d "$samples" || die "samples location not >> >> specified" >> >>

Re: [libav-devel] [PATCH 5/5] fate.sh: add support for build-only FATE instances

2011-09-20 Thread Diego Biurrun
On Tue, Sep 20, 2011 at 05:27:50PM +0200, Diego Biurrun wrote: > On Tue, Sep 20, 2011 at 04:14:29PM +0100, Måns Rullgård wrote: > > Diego Biurrun writes: > > > > > If the "build_only" variable is set in the configuration file, the FATE > > > client will skip running tests and just compile all tar

Re: [libav-devel] [PATCH 1/5] fate.sh: whitespace cosmetics

2011-09-20 Thread Diego Biurrun
On Tue, Sep 20, 2011 at 04:17:44PM +0100, Måns Rullgård wrote: > Måns Rullgård writes: > > > Diego Biurrun writes: > > > >> --- a/tests/fate.sh > >> +++ b/tests/fate.sh > >> @@ -21,7 +21,7 @@ test -d "$samples" || die "samples location not > >> specified" > >> > >> lock(){ > >> lock=$1/fa

Re: [libav-devel] [PATCH 5/5] fate.sh: add support for build-only FATE instances

2011-09-20 Thread Diego Biurrun
On Tue, Sep 20, 2011 at 04:14:29PM +0100, Måns Rullgård wrote: > Diego Biurrun writes: > > > If the "build_only" variable is set in the configuration file, the FATE > > client will skip running tests and just compile all targets. > > > > --- a/tests/fate.sh > > +++ b/tests/fate.sh > > @@ -65,6 +6

Re: [libav-devel] [PATCH 1/5] fate.sh: whitespace cosmetics

2011-09-20 Thread Måns Rullgård
Måns Rullgård writes: > Diego Biurrun writes: > >> --- >> tests/fate.sh | 16 >> 1 files changed, 8 insertions(+), 8 deletions(-) >> >> diff --git a/tests/fate.sh b/tests/fate.sh >> index 9fd117c..6015155 100755 >> --- a/tests/fate.sh >> +++ b/tests/fate.sh >> @@ -21,7 +21,7

Re: [libav-devel] [PATCH 5/5] fate.sh: add support for build-only FATE instances

2011-09-20 Thread Måns Rullgård
Diego Biurrun writes: > If the "build_only" variable is set in the configuration file, the FATE > client will skip running tests and just compile all targets. > --- > tests/fate.sh |1 + > 1 files changed, 1 insertions(+), 0 deletions(-) > > diff --git a/tests/fate.sh b/tests/fate.sh > index

Re: [libav-devel] [PATCH 3/5] fate.sh: Ignore errors from rm command during cleanup.

2011-09-20 Thread Måns Rullgård
Diego Biurrun writes: > The install directory being deleted might not be present if the build failed. > This can lead to annoying error output if the FATE client is run as a cronjob. > --- > tests/fate.sh |2 +- > 1 files changed, 1 insertions(+), 1 deletions(-) > > diff --git a/tests/fate.s

Re: [libav-devel] [PATCH 1/5] fate.sh: whitespace cosmetics

2011-09-20 Thread Måns Rullgård
Diego Biurrun writes: > --- > tests/fate.sh | 16 > 1 files changed, 8 insertions(+), 8 deletions(-) > > diff --git a/tests/fate.sh b/tests/fate.sh > index 9fd117c..6015155 100755 > --- a/tests/fate.sh > +++ b/tests/fate.sh > @@ -21,7 +21,7 @@ test -d "$samples" || die "sample

[libav-devel] [PATCH 5/5] fate.sh: add support for build-only FATE instances

2011-09-20 Thread Diego Biurrun
If the "build_only" variable is set in the configuration file, the FATE client will skip running tests and just compile all targets. --- tests/fate.sh |1 + 1 files changed, 1 insertions(+), 0 deletions(-) diff --git a/tests/fate.sh b/tests/fate.sh index b0df4df..7b1823c 100755 --- a/tests/fa

[libav-devel] [PATCH 2/5] fate.sh: Run git-pull in quiet mode to avoid console spam.

2011-09-20 Thread Diego Biurrun
Since fate.sh can be run from cron, silent commands are preferrable. --- tests/fate.sh |2 +- 1 files changed, 1 insertions(+), 1 deletions(-) diff --git a/tests/fate.sh b/tests/fate.sh index 6015155..df53192 100755 --- a/tests/fate.sh +++ b/tests/fate.sh @@ -35,7 +35,7 @@ checkout(){ update

[libav-devel] [PATCH 4/5] fate.sh: Ignore errors from non-existing FATE report files.

2011-09-20 Thread Diego Biurrun
In case of build failures accessing these files will generate error messages that are annoying when running the FATE client through cron and add no value since the build failure implies that these files will not get created. --- tests/fate.sh |3 ++- 1 files changed, 2 insertions(+), 1 deletio

[libav-devel] [PATCH 3/5] fate.sh: Ignore errors from rm command during cleanup.

2011-09-20 Thread Diego Biurrun
The install directory being deleted might not be present if the build failed. This can lead to annoying error output if the FATE client is run as a cronjob. --- tests/fate.sh |2 +- 1 files changed, 1 insertions(+), 1 deletions(-) diff --git a/tests/fate.sh b/tests/fate.sh index df53192..7519

[libav-devel] [PATCH 1/5] fate.sh: whitespace cosmetics

2011-09-20 Thread Diego Biurrun
--- tests/fate.sh | 16 1 files changed, 8 insertions(+), 8 deletions(-) diff --git a/tests/fate.sh b/tests/fate.sh index 9fd117c..6015155 100755 --- a/tests/fate.sh +++ b/tests/fate.sh @@ -21,7 +21,7 @@ test -d "$samples" || die "samples location not specified" lock(){

Re: [libav-devel] [PATCH] avplay: flush audio decoder with empty packets at EOF if the decoder has CODEC_CAP_DELAY set.

2011-09-20 Thread Anton Khirnov
On Fri, 16 Sep 2011 22:52:28 -0400, Justin Ruggles wrote: > --- > Well, I hate to do this... but I found a weird situation where my > previous patch did not work. Some audio decoders (e.g. shorten) buffer > input data internally. When that happens they will consume all input > data but someti

Re: [libav-devel] [PATCH 1/2] rtmp: Handle FLV packets written in more than one write call

2011-09-20 Thread Martin Storsjö
On Tue, 20 Sep 2011, Kostya Shishkov wrote: On Tue, Sep 20, 2011 at 04:09:41PM +0300, Martin Storsjö wrote: From: Chiranjeevi Melam If the FLV packet is larger than the AVIO buffer, a partial FLV packet will be flushed to the RTMP protocol. This commit handles the most common cases of FLV pa

Re: [libav-devel] [PATCH 2/2] rtmp: Don't blindly skip the 4 trailer bytes from the FLV packets

2011-09-20 Thread Kostya Shishkov
On Tue, Sep 20, 2011 at 04:09:42PM +0300, Martin Storsjö wrote: > If not enough bytes are available, keep track of them and skip > them on next call. > > In practice, if these trailer bytes are written in a separate > call, there is no other data written in this call, making it > fall into the "FL

Re: [libav-devel] [PATCH 1/2] rtmp: Handle FLV packets written in more than one write call

2011-09-20 Thread Kostya Shishkov
On Tue, Sep 20, 2011 at 04:09:41PM +0300, Martin Storsjö wrote: > From: Chiranjeevi Melam > > If the FLV packet is larger than the AVIO buffer, a partial > FLV packet will be flushed to the RTMP protocol. > > This commit handles the most common cases of FLV packets > being written in more than o

[libav-devel] [PATCH 2/2] rtmp: Don't blindly skip the 4 trailer bytes from the FLV packets

2011-09-20 Thread Martin Storsjö
If not enough bytes are available, keep track of them and skip them on next call. In practice, if these trailer bytes are written in a separate call, there is no other data written in this call, making it fall into the "FLV packet too small" case currently - working, but not as intended. This pat

[libav-devel] [PATCH 1/2] rtmp: Handle FLV packets written in more than one write call

2011-09-20 Thread Martin Storsjö
From: Chiranjeevi Melam If the FLV packet is larger than the AVIO buffer, a partial FLV packet will be flushed to the RTMP protocol. This commit handles the most common cases of FLV packets being written in more than one call. --- Compared to other versions of the same patch, this one adjusts s

Re: [libav-devel] [PATCH 2/3] id3v2: add support for non-text and GEOB type meta tags

2011-09-20 Thread Anton Khirnov
On Tue, 20 Sep 2011 08:18:24 +0200, David Goldwich wrote: > From 4666e1916daa0ba09ac63b2d293eb8e21a25cc99 Mon Sep 17 00:00:00 2001 > From: David Goldwich > Date: Sun, 18 Sep 2011 18:04:34 +0200 > Subject: [PATCH] id3v2: add support for non-text and GEOB type tag frames > > This extends the ID3

Re: [libav-devel] [PATCH 2/2] Fix assert() calls that need updates after FF_COMMON_FRAME macro elimination.

2011-09-20 Thread Martin Storsjö
On Tue, 13 Sep 2011, Diego Biurrun wrote: --- libavcodec/h261dec.c |4 ++-- libavcodec/h264.c |2 +- libavcodec/h264_direct.c |4 ++-- libavcodec/mpegvideo.c |4 ++-- libavcodec/mpegvideo_enc.c | 10 +- 5 files changed, 12 insertions(+), 12 deletions(-)

Re: [libav-devel] [PATCH 2/2] Fix assert() calls that need updates after FF_COMMON_FRAME macro elimination.

2011-09-20 Thread Kostya Shishkov
On Tue, Sep 13, 2011 at 05:06:47PM +0200, Diego Biurrun wrote: > --- > libavcodec/h261dec.c |4 ++-- > libavcodec/h264.c |2 +- > libavcodec/h264_direct.c |4 ++-- > libavcodec/mpegvideo.c |4 ++-- > libavcodec/mpegvideo_enc.c | 10 +- > 5 files chang

Re: [libav-devel] [PATCH 1/2] Fix av_dlog invocations without logging context.

2011-09-20 Thread Martin Storsjö
On Tue, 13 Sep 2011, Diego Biurrun wrote: --- libavcodec/flashsvenc.c |2 +- libavformat/mpegts.c|2 +- 2 files changed, 2 insertions(+), 2 deletions(-) diff --git a/libavcodec/flashsvenc.c b/libavcodec/flashsvenc.c index 6dc33b9..81aa5b6 100644 --- a/libavcodec/flashsvenc.c +++ b/lib

Re: [libav-devel] [PATCH 1/2] Fix av_dlog invocations without logging context.

2011-09-20 Thread Kostya Shishkov
On Tue, Sep 13, 2011 at 05:06:46PM +0200, Diego Biurrun wrote: > --- > libavcodec/flashsvenc.c |2 +- > libavformat/mpegts.c|2 +- > 2 files changed, 2 insertions(+), 2 deletions(-) > > diff --git a/libavcodec/flashsvenc.c b/libavcodec/flashsvenc.c > index 6dc33b9..81aa5b6 100644 > --

Re: [libav-devel] [PATCH] Check for invalid slice offsets in real decoder.

2011-09-20 Thread Martin Storsjö
On Tue, 20 Sep 2011, Kostya Shishkov wrote: On Mon, Sep 19, 2011 at 10:48:53PM +0200, fen...@elivagar.org wrote: From: Laurent Aimar --- libavcodec/rv34.c |9 + 1 files changed, 5 insertions(+), 4 deletions(-) diff --git a/libavcodec/rv34.c b/libavcodec/rv34.c index 8223e92..a76

Re: [libav-devel] [PATCH] msmpeg4: remove leftover unused debug variable declaration

2011-09-20 Thread Kostya Shishkov
On Tue, Sep 20, 2011 at 09:24:01AM +0200, Diego Biurrun wrote: > --- > libavcodec/msmpeg4.c |4 > 1 files changed, 0 insertions(+), 4 deletions(-) > > diff --git a/libavcodec/msmpeg4.c b/libavcodec/msmpeg4.c > index d773c38..eca2add 100644 > --- a/libavcodec/msmpeg4.c > +++ b/libavcodec/

[libav-devel] [PATCH] msmpeg4: remove leftover unused debug variable declaration

2011-09-20 Thread Diego Biurrun
--- libavcodec/msmpeg4.c |4 1 files changed, 0 insertions(+), 4 deletions(-) diff --git a/libavcodec/msmpeg4.c b/libavcodec/msmpeg4.c index d773c38..eca2add 100644 --- a/libavcodec/msmpeg4.c +++ b/libavcodec/msmpeg4.c @@ -62,10 +62,6 @@ static uint32_t v2_dc_chroma_table[512][2]; /* vc