Re: [libav-devel] [PATCH] PGS subtitles: Exposed forced flag

2012-11-01 Thread Luca Barbato
On 10/20/12 6:59 PM, John Stebbins wrote: On 10/15/2012 09:36 AM, Diego Biurrun wrote: On Mon, Oct 15, 2012 at 06:01:17PM +0200, John Stebbins wrote: On 10/15/2012 05:38 PM, Diego Biurrun wrote: On Sun, Oct 14, 2012 at 05:41:03PM +0200, John Stebbins wrote: On 10/14/2012 05:14 PM, Luca Barbat

Re: [libav-devel] [PATCH 4/5] lavf: clarify the lifetime of demuxed packets.

2012-11-01 Thread Anton Khirnov
On Fri, 2 Nov 2012 00:00:56 +0200 (EET), Martin Storsjö wrote: > On Wed, 31 Oct 2012, Anton Khirnov wrote: > > > --- > > libavformat/avformat.h | 26 -- > > 1 file changed, 16 insertions(+), 10 deletions(-) > > > > diff --git a/libavformat/avformat.h b/libavformat/avfor

Re: [libav-devel] [PATCH 2/5] avconv: do not free muxed packet on streamcopy.

2012-11-01 Thread Anton Khirnov
On Fri, 2 Nov 2012 00:07:49 +0200 (EET), Martin Storsjö wrote: > On Wed, 31 Oct 2012, Anton Khirnov wrote: > > > The packet belongs to lavf, the caller must not free it or access it in > > any other way. > > No change in practice, since destruct is set to NULL anyway. > > --- > > avconv.c |

Re: [libav-devel] [PATCH 5/5] rtpdec_vp8: remove write-only variables.

2012-11-01 Thread Anton Khirnov
On Fri, 2 Nov 2012 00:18:10 +0200 (EET), Martin Storsjö wrote: > On Thu, 1 Nov 2012, Anton Khirnov wrote: > > > --- > > libavformat/rtpdec_vp8.c |8 +--- > > 1 file changed, 1 insertion(+), 7 deletions(-) > > http://patches.libav.org/patch/28492/ - this does pretty much the same but >

Re: [libav-devel] [PATCH] rtpdec_vp8: Don't parse fields that aren't used

2012-11-01 Thread Anton Khirnov
On Tue, 9 Oct 2012 01:32:37 +0300, Martin Storsjö wrote: > This avoids warnings about unused variables. > --- > libavformat/rtpdec_vp8.c | 25 ++--- > 1 file changed, 6 insertions(+), 19 deletions(-) > > diff --git a/libavformat/rtpdec_vp8.c b/libavformat/rtpdec_vp8.c > i

Re: [libav-devel] [PATCH 2/5] matroskadec: do not use avpacket internals

2012-11-01 Thread Anton Khirnov
On Fri, 2 Nov 2012 00:13:23 +0200 (EET), Martin Storsjö wrote: > On Thu, 1 Nov 2012, Anton Khirnov wrote: > > > --- > > libavformat/matroskadec.c | 15 --- > > 1 file changed, 8 insertions(+), 7 deletions(-) > > > > diff --git a/libavformat/matroskadec.c b/libavformat/matroskadec.c

Re: [libav-devel] [PATCH 1/1] doc: add apidoc target for doxygen API documentation

2012-11-01 Thread Anton Khirnov
On Fri, 02 Nov 2012 01:06:33 +0100, Diego Biurrun wrote: > On Wed, Oct 31, 2012 at 09:26:10PM +0100, Anton Khirnov wrote: > > On Wed, 31 Oct 2012 17:33:35 +0100, Diego Biurrun wrote: > > > On Mon, Oct 29, 2012 at 03:45:43PM +0100, Diego Biurrun wrote: > > > > On Mon, Oct 29, 2012 at 03:19:27PM +

Re: [libav-devel] [PATCH 1/1] doc: add apidoc target for doxygen API documentation

2012-11-01 Thread Diego Biurrun
On Wed, Oct 31, 2012 at 09:26:10PM +0100, Anton Khirnov wrote: > On Wed, 31 Oct 2012 17:33:35 +0100, Diego Biurrun wrote: > > On Mon, Oct 29, 2012 at 03:45:43PM +0100, Diego Biurrun wrote: > > > On Mon, Oct 29, 2012 at 03:19:27PM +0100, Janne Grunau wrote: > > > > On 2012-10-29 15:03:45 +0100, Die

Re: [libav-devel] [PATCH 4/5] audiointerleave: deobfuscate a function call.

2012-11-01 Thread Diego Biurrun
On Thu, Nov 01, 2012 at 09:46:26PM +0100, Anton Khirnov wrote: > right above there is if (pkt) {; pkt = NULL}, so pkt is just a > fancy name for NULL at this point. > --- > libavformat/audiointerleave.c |2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) OK Diego _

Re: [libav-devel] [PATCH 1/5] a64: remove interleaved mode.

2012-11-01 Thread Diego Biurrun
On Thu, Nov 01, 2012 at 09:46:23PM +0100, Anton Khirnov wrote: > It has been disabled since it was added two years ago. > --- > libavformat/a64.c | 116 > + > 1 file changed, 2 insertions(+), 114 deletions(-) Fine with me. Diego _

Re: [libav-devel] [PATCH 5/5] rtpdec_vp8: remove write-only variables.

2012-11-01 Thread Martin Storsjö
On Thu, 1 Nov 2012, Anton Khirnov wrote: --- libavformat/rtpdec_vp8.c |8 +--- 1 file changed, 1 insertion(+), 7 deletions(-) http://patches.libav.org/patch/28492/ - this does pretty much the same but simplifying things a little more. I might readd some of these things if I improve th

Re: [libav-devel] [PATCH 3/5] rtpdec: factorize identical code used in several handlers

2012-11-01 Thread Martin Storsjö
On Thu, 1 Nov 2012, Anton Khirnov wrote: --- libavformat/rtpdec.c | 11 +++ libavformat/rtpdec.h |5 + libavformat/rtpdec_h263_rfc2190.c | 12 +--- libavformat/rtpdec_jpeg.c | 13 +++-- libavformat/rtpdec_svq3.c |9 +

Re: [libav-devel] [PATCH 2/5] matroskadec: do not use avpacket internals

2012-11-01 Thread Martin Storsjö
On Thu, 1 Nov 2012, Anton Khirnov wrote: --- libavformat/matroskadec.c | 15 --- 1 file changed, 8 insertions(+), 7 deletions(-) diff --git a/libavformat/matroskadec.c b/libavformat/matroskadec.c index 26f8707..bf67253 100644 --- a/libavformat/matroskadec.c +++ b/libavformat/matros

Re: [libav-devel] [PATCH 2/5] avconv: do not free muxed packet on streamcopy.

2012-11-01 Thread Martin Storsjö
On Wed, 31 Oct 2012, Anton Khirnov wrote: The packet belongs to lavf, the caller must not free it or access it in any other way. No change in practice, since destruct is set to NULL anyway. --- avconv.c |1 - 1 file changed, 1 deletion(-) diff --git a/avconv.c b/avconv.c index 5d050dd..421ae

Re: [libav-devel] [PATCH 5/5] lavf: fix av_interleaved_write_frame() doxy.

2012-11-01 Thread Martin Storsjö
On Wed, 31 Oct 2012, Anton Khirnov wrote: A sentence was mistakenly inserted in the middle of another sentence. --- libavformat/avformat.h |4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/libavformat/avformat.h b/libavformat/avformat.h index 0961308..3e10496 100644 --- a

Re: [libav-devel] [PATCH 4/5] lavf: clarify the lifetime of demuxed packets.

2012-11-01 Thread Martin Storsjö
On Wed, 31 Oct 2012, Anton Khirnov wrote: --- libavformat/avformat.h | 26 -- 1 file changed, 16 insertions(+), 10 deletions(-) diff --git a/libavformat/avformat.h b/libavformat/avformat.h index 8af007e..0961308 100644 --- a/libavformat/avformat.h +++ b/libavformat/avfo

Re: [libav-devel] [PATCH 3/5] lavc: add some AVPacket doxy.

2012-11-01 Thread Martin Storsjö
On Wed, 31 Oct 2012, Anton Khirnov wrote: --- libavcodec/avcodec.h | 22 ++ 1 file changed, 22 insertions(+) diff --git a/libavcodec/avcodec.h b/libavcodec/avcodec.h index d6a4e4d..73321b2 100644 --- a/libavcodec/avcodec.h +++ b/libavcodec/avcodec.h @@ -872,6 +872,28 @@ enu

[libav-devel] [PATCH 1/5] a64: remove interleaved mode.

2012-11-01 Thread Anton Khirnov
It has been disabled since it was added two years ago. --- libavformat/a64.c | 116 + 1 file changed, 2 insertions(+), 114 deletions(-) diff --git a/libavformat/a64.c b/libavformat/a64.c index 56b9a44..15a0475 100644 --- a/libavformat/a64.c +++

Re: [libav-devel] [PATCH] decode_audio3: initialize AVFrame

2012-11-01 Thread Anton Khirnov
On Wed, 31 Oct 2012 11:24:36 +0200, il...@porogrammer.fi wrote: > From: Ilkka Ollakka > > Same fix and issue than in a25d912dca9cd553440167e0476c47581359c0fc > --- Pushed. Thanks for the patch. -- Anton Khirnov ___ libav-devel mailing list libav-de

[libav-devel] [PATCH 2/5] matroskadec: do not use avpacket internals

2012-11-01 Thread Anton Khirnov
--- libavformat/matroskadec.c | 15 --- 1 file changed, 8 insertions(+), 7 deletions(-) diff --git a/libavformat/matroskadec.c b/libavformat/matroskadec.c index 26f8707..bf67253 100644 --- a/libavformat/matroskadec.c +++ b/libavformat/matroskadec.c @@ -1137,13 +1137,14 @@ static voi

[libav-devel] [PATCH 5/5] rtpdec_vp8: remove write-only variables.

2012-11-01 Thread Anton Khirnov
--- libavformat/rtpdec_vp8.c |8 +--- 1 file changed, 1 insertion(+), 7 deletions(-) diff --git a/libavformat/rtpdec_vp8.c b/libavformat/rtpdec_vp8.c index 541a1bc..0ad5e16 100644 --- a/libavformat/rtpdec_vp8.c +++ b/libavformat/rtpdec_vp8.c @@ -45,16 +45,14 @@ static int vp8_handle_packe

[libav-devel] [PATCH 3/5] rtpdec: factorize identical code used in several handlers

2012-11-01 Thread Anton Khirnov
--- libavformat/rtpdec.c | 11 +++ libavformat/rtpdec.h |5 + libavformat/rtpdec_h263_rfc2190.c | 12 +--- libavformat/rtpdec_jpeg.c | 13 +++-- libavformat/rtpdec_svq3.c |9 - libavformat/rtpdec_vp8.c

[libav-devel] [PATCH 4/5] audiointerleave: deobfuscate a function call.

2012-11-01 Thread Anton Khirnov
right above there is if (pkt) {; pkt = NULL}, so pkt is just a fancy name for NULL at this point. --- libavformat/audiointerleave.c |2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/libavformat/audiointerleave.c b/libavformat/audiointerleave.c index e48f826..5df0bb0 100644 ---

Re: [libav-devel] [PATCH 2/8] lavu: add av_ctz() for trailing zero bit count

2012-11-01 Thread Luca Barbato
On 11/1/12 5:35 PM, Justin Ruggles wrote: On 10/29/2012 10:30 AM, Justin Ruggles wrote: --- doc/APIchanges |3 ++ libavutil/Makefile |2 +- libavutil/{log2.c => intmath.c} |5 +++ libavutil/intmath.h | 55 ++

Re: [libav-devel] [PATCH] PGS subtitles: Exposed forced flag

2012-11-01 Thread John Stebbins
On 10/20/2012 09:59 AM, John Stebbins wrote: > On 10/15/2012 09:36 AM, Diego Biurrun wrote: >> On Mon, Oct 15, 2012 at 06:01:17PM +0200, John Stebbins wrote: >>> On 10/15/2012 05:38 PM, Diego Biurrun wrote: On Sun, Oct 14, 2012 at 05:41:03PM +0200, John Stebbins wrote: > On 10/14/2012 05:1

Re: [libav-devel] [PATCH 2/8] lavu: add av_ctz() for trailing zero bit count

2012-11-01 Thread Justin Ruggles
On 10/29/2012 10:30 AM, Justin Ruggles wrote: > --- > doc/APIchanges |3 ++ > libavutil/Makefile |2 +- > libavutil/{log2.c => intmath.c} |5 +++ > libavutil/intmath.h | 55 > +++ > libavutil/version.h

Re: [libav-devel] [PATCH] doc: Point to the new location of the c99-to-c89 tool

2012-11-01 Thread Derek Buitenhuis
On 01/11/2012 9:29 AM, Martin Storsjö wrote: > This location contains prebuilt binaries. > --- > doc/platform.texi |2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) OK. - Derek ___ libav-devel mailing list libav-devel@libav.org https://lists.l

[libav-devel] [PATCH] doc: Point to the new location of the c99-to-c89 tool

2012-11-01 Thread Martin Storsjö
This location contains prebuilt binaries. --- doc/platform.texi |2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/doc/platform.texi b/doc/platform.texi index 6bb7136..999a5f1 100644 --- a/doc/platform.texi +++ b/doc/platform.texi @@ -109,7 +109,7 @@ wrapper. You will need the