Re: [libav-devel] [PATCH] fifo: add av_fifo_peek2()

2011-08-26 Thread Anton Khirnov
On Wed, 10 Aug 2011 17:41:03 +0200, Stefano Sabatini wrote: > On date Thursday 2011-08-04 18:48:09 +0200, Stefano Sabatini encoded: > > On date Thursday 2011-07-21 19:45:15 +0200, Stefano Sabatini encoded: > > > On date Saturday 2011-07-16 17:15:27 +0200, Stefano Sabatini encoded: > [...] > > >

[libav-devel] [PATCH] libx264: make options compatible with x264

2011-08-26 Thread Anton Khirnov
Replace '_' with '-', merge psy-trellis into psy-rd and rename cqp->qp --- libavcodec/libx264.c | 22 ++ 1 files changed, 10 insertions(+), 12 deletions(-) diff --git a/libavcodec/libx264.c b/libavcodec/libx264.c index 9a9e6d0..8bc6255 100644 --- a/libavcodec/libx264.c +++ b

Re: [libav-devel] [PATCH] mpegaudio_parser: be less picky on the start position

2011-08-26 Thread Alex Converse
2011/8/26 Måns Rullgård : [...] > > What effect does this change have and what, if anything, does it fix? > It fixes duration/bitrate estimation for me: before: built on Aug 26 2011 16:33:59 with clang 2.9 (tags/RELEASE_29/final) configuration: --enable-gpl --enable-libvorbis --enable-libmp3

Re: [libav-devel] [PATCH] mpegaudio_parser: be less picky on the start position

2011-08-26 Thread Måns Rullgård
Alex Converse writes: > From: Michael Niedermayer > > Signed-off-by: Michael Niedermayer > (cherry picked from commit b37dfe181cc34ea76155cf449d168c385f32f061) > > Signed-off-by: Alex Converse > --- > libavcodec/mpegaudio_parser.c |3 ++- > 1 files changed, 2 insertions(+), 1 deletions(-)

[libav-devel] [PATCH] mpegaudio_parser: be less picky on the start position

2011-08-26 Thread Alex Converse
From: Michael Niedermayer Signed-off-by: Michael Niedermayer (cherry picked from commit b37dfe181cc34ea76155cf449d168c385f32f061) Signed-off-by: Alex Converse --- libavcodec/mpegaudio_parser.c |3 ++- 1 files changed, 2 insertions(+), 1 deletions(-) diff --git a/libavcodec/mpegaudio_pars

Re: [libav-devel] [PATCH] make parsers' repeat_pict consistent with decoders

2011-08-26 Thread John Stebbins
Hi, On 08/26/2011 12:55 PM, Vladimir Pantelic wrote: > On 08/26/2011 05:43 PM, John Stebbins wrote: >> Hi, >> >> On 08/26/2011 05:48 AM, Vladimir Pantelic wrote: > >>> int ticks = st->st->codec->ticks_per_frame + (ist->st->parser ? >>> ist->st->parser->repeat_pict : 0); >>> >> >> Well, after all

Re: [libav-devel] [PATCH] make parsers' repeat_pict consistent with decoders

2011-08-26 Thread Vladimir Pantelic
On 08/26/2011 05:43 PM, John Stebbins wrote: Hi, On 08/26/2011 05:48 AM, Vladimir Pantelic wrote: int ticks = st->st->codec->ticks_per_frame + (ist->st->parser ? ist->st->parser->repeat_pict : 0); Well, after all that, I did some more code reading and spelunking of mailing list archives

[libav-devel] [PATCH] ac3enc: fix encoding of stereo ac3 files when rematrixing is disabled.

2011-08-26 Thread Justin Ruggles
The number of rematrixing bands still needs to be calculated. --- libavcodec/ac3enc_template.c | 10 +- 1 files changed, 5 insertions(+), 5 deletions(-) diff --git a/libavcodec/ac3enc_template.c b/libavcodec/ac3enc_template.c index dd759a7..3db49de 100644 --- a/libavcodec/ac3enc_templat

Re: [libav-devel] [PATCH] postprocess.c: filter name needs to be double 0 terminated

2011-08-26 Thread Reinhard Tartler
On Fri, Aug 26, 2011 at 19:01:51 (CEST), Kostya wrote: > On Fri, Aug 26, 2011 at 06:43:03PM +0200, Anton Khirnov wrote: >> >> On Wed, 24 Aug 2011 16:12:15 +0200, Reinhard Tartler >> wrote: >> > From: Piotr Kaczuba >> > >> > --- >> > libpostproc/postprocess.c |5 +++-- >> > 1 files change

Re: [libav-devel] Race conditions in libavcodec/pthread.c

2011-08-26 Thread Vitor Sessak
On Fri, Aug 26, 2011 at 5:56 AM, Ronald S. Bultje wrote: > Hi Aaron, > > On Thu, Aug 25, 2011 at 5:01 PM, Aaron Colwell wrote: >> Some recent changes to Chromium unit tests have caused FFmpeg decoding to >> get regularly scrutinized by our ThreadSanitizer tool. This has led to the >> detection of

Re: [libav-devel] [PATCH] Use deinterleavers for demangling audio packets in RealMedia.

2011-08-26 Thread Kostya
On Tue, Aug 16, 2011 at 02:55:56PM -0700, Ronald S. Bultje wrote: > Hi, > > On Sat, Aug 13, 2011 at 12:02 PM, Kostya Shishkov > wrote: > > Unlike other containers RealMedia stores its audio packets in scrambled > > form, > > with interleaver ID preceeding audio codec ID. Currently deinterleaving

Re: [libav-devel] [PATCH] mpeg12: propagate chunk decode errors and fix conditional indentation

2011-08-26 Thread Anton Khirnov
On Tue, 16 Aug 2011 18:02:33 -0400, Dustin Brody wrote: > --- > libavcodec/mpeg12.c | 26 ++ > 1 files changed, 22 insertions(+), 4 deletions(-) > > diff --git a/libavcodec/mpeg12.c b/libavcodec/mpeg12.c > index 91eb3f5..200289a 100644 > --- a/libavcodec/mpeg12.c > +++

Re: [libav-devel] [PATCH] postprocess.c: filter name needs to be double 0 terminated

2011-08-26 Thread Kostya
On Fri, Aug 26, 2011 at 06:43:03PM +0200, Anton Khirnov wrote: > > On Wed, 24 Aug 2011 16:12:15 +0200, Reinhard Tartler > wrote: > > From: Piotr Kaczuba > > > > --- > > libpostproc/postprocess.c |5 +++-- > > 1 files changed, 3 insertions(+), 2 deletions(-) > > > > diff --git a/libpostpr

Re: [libav-devel] [PATCH] postprocess.c: filter name needs to be double 0 terminated

2011-08-26 Thread Anton Khirnov
On Wed, 24 Aug 2011 16:12:15 +0200, Reinhard Tartler wrote: > From: Piotr Kaczuba > > --- > libpostproc/postprocess.c |5 +++-- > 1 files changed, 3 insertions(+), 2 deletions(-) > > diff --git a/libpostproc/postprocess.c b/libpostproc/postprocess.c > index ca6d85f..fb51d94 100644 > ---

Re: [libav-devel] [PATCH 2/2] doxygen: drop Doxygen markup where standard comments are more appropriate

2011-08-26 Thread Diego Biurrun
On Tue, Aug 09, 2011 at 11:14:07PM +0200, Luca Barbato wrote: > On 08/08/2011 08:42 PM, Måns Rullgård wrote: > >> The idea is "have documentation open in browser, and look at actual > >> code in editor". Otherwise you need multiple code editors open. > > > > I do not want code from someone who can

Re: [libav-devel] [PATCH] mjpeg: treat external huffman table setup failure as codec init failure if external huffman table use requested

2011-08-26 Thread Anton Khirnov
On Thu, 25 Aug 2011 19:47:01 -0400, Dustin Brody wrote: > --- > libavcodec/mjpegdec.c |4 ++-- > libavcodec/mxpegdec.c |4 +--- > 2 files changed, 3 insertions(+), 5 deletions(-) > Looks fine to me. No point in trying to be smart when the caller explicitly asked for something different

Re: [libav-devel] [PATCH] doxygen: drop pointless star from pointer variable names

2011-08-26 Thread Anton Khirnov
On Mon, 08 Aug 2011 12:07:20 +0200, Diego Biurrun wrote: > --- > libavcodec/s3tc.h |8 > libavutil/fifo.h | 24 > 2 files changed, 16 insertions(+), 16 deletions(-) > Ok. -- Anton Khirnov ___ libav-devel mai

Re: [libav-devel] [PATCH] Replace deprecated av_find_stream_info() by avformat_find_stream_info().

2011-08-26 Thread Anton Khirnov
On Fri, 26 Aug 2011 17:43:04 +0200, Diego Biurrun wrote: > --- > avprobe.c |2 +- > avserver.c |4 ++-- > libavformat/seek-test.c |2 +- > tools/pktdumper.c |4 ++-- > 4 files changed, 6 insertions(+), 6 deletions(-) > Ok. -- Anton Khirnov __

Re: [libav-devel] [PATCH] make parsers' repeat_pict consistent with decoders

2011-08-26 Thread John Stebbins
Hi, On 08/26/2011 05:48 AM, Vladimir Pantelic wrote: > John Stebbins wrote: >> On 08/25/2011 07:45 PM, John Stebbins wrote: >>> As we discussed on IRC this morning, this makes the value of repeat_pict >>> returned >>> by the parsers the same as the value returned by the decoders. >>> >>> The p

[libav-devel] [PATCH] Replace deprecated av_find_stream_info() by avformat_find_stream_info().

2011-08-26 Thread Diego Biurrun
--- avprobe.c |2 +- avserver.c |4 ++-- libavformat/seek-test.c |2 +- tools/pktdumper.c |4 ++-- 4 files changed, 6 insertions(+), 6 deletions(-) diff --git a/avprobe.c b/avprobe.c index 3843af9..d502421 100644 --- a/avprobe.c +++ b/avprobe.c @@

Re: [libav-devel] [PATCH] configure: add --disable-binaries option.

2011-08-26 Thread Diego Biurrun
On Fri, Aug 26, 2011 at 02:56:55PM +0200, Clément Bœsch wrote: > On Fri, Aug 26, 2011 at 03:29:10AM +0200, Diego Biurrun wrote: > > On Thu, Aug 25, 2011 at 11:45:09PM +0200, Clément Bœsch wrote: > > > On Thu, Aug 25, 2011 at 10:06:41PM +0100, Måns Rullgård wrote: > > > [...] > > > > > > > > I stil

Re: [libav-devel] [PATCH] configure: add --disable-binaries option.

2011-08-26 Thread Diego Biurrun
On Fri, Aug 26, 2011 at 02:58:00PM +0200, Clément Bœsch wrote: > On Fri, Aug 26, 2011 at 03:26:32AM +0200, Diego Biurrun wrote: > > On Thu, Aug 25, 2011 at 10:57:12PM +0200, Clément Bœsch wrote: > > > On Thu, Aug 25, 2011 at 10:03:54PM +0200, Clément Bœsch wrote: > > > [...] > > > > From 017782fe40

Re: [libav-devel] [PATCH] configure: add --disable-binaries option.

2011-08-26 Thread Måns Rullgård
Clément Bœsch writes: > On Fri, Aug 26, 2011 at 03:29:10AM +0200, Diego Biurrun wrote: >> On Thu, Aug 25, 2011 at 11:45:09PM +0200, Clément Bœsch wrote: >> > On Thu, Aug 25, 2011 at 10:06:41PM +0100, Måns Rullgård wrote: >> > [...] >> > > >> > > I still don't like it. >> > >> > OK you won, I gi

Re: [libav-devel] [PATCH] configure: add --disable-binaries option.

2011-08-26 Thread Clément Bœsch
On Fri, Aug 26, 2011 at 03:29:10AM +0200, Diego Biurrun wrote: > On Thu, Aug 25, 2011 at 11:45:09PM +0200, Clément Bœsch wrote: > > On Thu, Aug 25, 2011 at 10:06:41PM +0100, Måns Rullgård wrote: > > [...] > > > > > > I still don't like it. > > > > OK you won, I give up. I really don't feel welcom

Re: [libav-devel] [PATCH] configure: add --disable-binaries option.

2011-08-26 Thread Clément Bœsch
On Fri, Aug 26, 2011 at 03:26:32AM +0200, Diego Biurrun wrote: > On Thu, Aug 25, 2011 at 10:57:12PM +0200, Clément Bœsch wrote: > > On Thu, Aug 25, 2011 at 10:03:54PM +0200, Clément Bœsch wrote: > > [...] > > > From 017782fe40695b0ff54842bc03ea852de2460e34 Mon Sep 17 00:00:00 2001 > > > From: =?UTF

Re: [libav-devel] [PATCH] make parsers' repeat_pict consistent with decoders

2011-08-26 Thread Vladimir Pantelic
John Stebbins wrote: On 08/25/2011 07:45 PM, John Stebbins wrote: As we discussed on IRC this morning, this makes the value of repeat_pict returned by the parsers the same as the value returned by the decoders. The parser was returning a value for repeat_pict that was the actual number of