Re: [libav-devel] [PATCH] avconv: Replace raw picture frame swapping hack.

2011-08-30 Thread Vladimir Pantelic
Alex Converse wrote: --- avconv.c |6 +++--- 1 files changed, 3 insertions(+), 3 deletions(-) diff --git a/avconv.c b/avconv.c index 6514269..93484b8 100644 --- a/avconv.c +++ b/avconv.c @@ -18,6 +18,7 @@ * License along with Libav; if not, write to the Free Software * Foundation,

[libav-devel] [PATCH] RV40: reuse some H.264 motion compensation functions in RV40.

2011-08-30 Thread Kostya Shishkov
Since RV40 is based on H.264, it uses the same interpolation filters and functions for halfpel interpolation in one direction can be reused. Unfortunately other cases are implemented differently and produce slightly different output because of rounding. --- libavcodec/rv40dsp.c | 24

[libav-devel] [PATCH] avconv: remove -threads option.

2011-08-30 Thread Anton Khirnov
It's only shadowing the AVOption with the same name. --- avconv.c | 18 -- 1 files changed, 0 insertions(+), 18 deletions(-) diff --git a/avconv.c b/avconv.c index 7caf00a..fbd24c9 100644 --- a/avconv.c +++ b/avconv.c @@ -168,7 +168,6 @@ static int audio_volume = 256; static

Re: [libav-devel] [PATCH] Revert h264: Properly set coded_{width, height} when parsing H.264.

2011-08-30 Thread Luca Barbato
On 8/30/11 6:41 AM, Uoti Urpala wrote: Note that it isn't this code (as in the reverted code) that sets it to a wrong value. The code setting it is in avcodec_open2(), which initially sets width to equal coded_width; this is probably currently needed for codecs like raw that do not themselves

Re: [libav-devel] [PATCH] avconv: remove -threads option.

2011-08-30 Thread Luca Barbato
On 8/30/11 11:43 AM, Anton Khirnov wrote: It's only shadowing the AVOption with the same name. ok ___ libav-devel mailing list libav-devel@libav.org https://lists.libav.org/mailman/listinfo/libav-devel

Re: [libav-devel] [PATCH] avcodec: remove misleading comment

2011-08-30 Thread Luca Barbato
On 8/29/11 10:55 PM, Ronald S. Bultje wrote: Hi, On Mon, Aug 29, 2011 at 1:48 PM, Luca Barbatolu_z...@gentoo.org wrote: coded_width is the width before lowres scaling, coded_height is the height before lowres scaling. --- libavcodec/avcodec.h |3 +-- 1 files changed, 1 insertions(+), 2

Re: [libav-devel] [PATCH] Revert h264: Properly set coded_{width, height} when parsing H.264.

2011-08-30 Thread Måns Rullgård
Ronald S. Bultje rsbul...@gmail.com writes: Hi, On Mon, Aug 29, 2011 at 8:16 PM, Uoti Urpala uoti.urp...@pp1.inet.fi wrote: On Mon, 2011-08-29 at 19:48 -0700, Ronald S. Bultje wrote: On Mon, Aug 29, 2011 at 7:06 PM, Uoti Urpala uoti.urp...@pp1.inet.fi wrote: The actual problem here was

Re: [libav-devel] [PATCH] RV40: reuse some H.264 motion compensation functions in RV40.

2011-08-30 Thread Måns Rullgård
Kostya Shishkov kostya.shish...@gmail.com writes: Since RV40 is based on H.264, it uses the same interpolation filters and functions for halfpel interpolation in one direction can be reused. Unfortunately other cases are implemented differently and produce slightly different output because of

Re: [libav-devel] [PATCH] RV40: reuse some H.264 motion compensation functions in RV40.

2011-08-30 Thread Kostya Shishkov
On Tue, Aug 30, 2011 at 12:04:23PM +0100, Måns Rullgård wrote: Kostya Shishkov kostya.shish...@gmail.com writes: Since RV40 is based on H.264, it uses the same interpolation filters and functions for halfpel interpolation in one direction can be reused. Unfortunately other cases are

[libav-devel] [PATCH] Fix VC-1 width/height handling.

2011-08-30 Thread Tomas Härdin
Attached patch cherry-picked from FFmpeg. It fixes issue2076. I uploaded a sample to http://titan.codemill.se/~tomhar/samples/double-free.vc1 in case the original went missing. /Tomas From bde3e96195421df37d879f1cc481a6b542e11b61 Mon Sep 17 00:00:00 2001 From:

Re: [libav-devel] [PATCH] Fix VC-1 width/height handling.

2011-08-30 Thread Kostya Shishkov
On Tue, Aug 30, 2011 at 04:03:16PM +0200, Tomas Härdin wrote: Attached patch cherry-picked from FFmpeg. It fixes issue2076. I uploaded a sample to http://titan.codemill.se/~tomhar/samples/double-free.vc1 in case the original went missing. Well, it's probably OK (damned if I understand those

Re: [libav-devel] [PATCH] Fix VC-1 width/height handling.

2011-08-30 Thread Luca Barbato
On 8/30/11 4:03 PM, Tomas Härdin wrote: Attached patch cherry-picked from FFmpeg. It fixes issue2076. I uploaded a sample to http://titan.codemill.se/~tomhar/samples/double-free.vc1 in case the original went missing. Thank you, the patch is yet another point to drop this whole coded_dimension

Re: [libav-devel] PATCH: LATM muxer

2011-08-30 Thread Kieran Kunhya
 seems ok Tidied up patch attached 0001-Add-LATM-muxer.patch Description: Binary data ___ libav-devel mailing list libav-devel@libav.org https://lists.libav.org/mailman/listinfo/libav-devel

Re: [libav-devel] [PATCH] avcodec: remove misleading comment

2011-08-30 Thread Ronald S. Bultje
Hi, On Tue, Aug 30, 2011 at 3:11 AM, Luca Barbato lu_z...@gentoo.org wrote: On 8/29/11 10:55 PM, Ronald S. Bultje wrote: Hi, On Mon, Aug 29, 2011 at 1:48 PM, Luca Barbatolu_z...@gentoo.org  wrote: coded_width is the width before lowres scaling, coded_height is the height before lowres

Re: [libav-devel] [PATCH] avconv: Replace raw picture frame swapping hack.

2011-08-30 Thread Alex Converse
On Mon, Aug 29, 2011 at 11:15 PM, Vladimir Pantelic vlado...@gmail.com wrote: Alex Converse wrote: ---  avconv.c |    6 +++---  1 files changed, 3 insertions(+), 3 deletions(-) diff --git a/avconv.c b/avconv.c index 6514269..93484b8 100644 --- a/avconv.c +++ b/avconv.c @@ -18,6 +18,7 @@

Re: [libav-devel] [PATCH] avconv: Replace raw picture frame swapping hack.

2011-08-30 Thread Alex Converse
On Tue, Aug 30, 2011 at 9:46 AM, Alex Converse alex.conve...@gmail.com wrote: On Mon, Aug 29, 2011 at 11:15 PM, Vladimir Pantelic vlado...@gmail.com wrote: Alex Converse wrote: ---  avconv.c |    6 +++---  1 files changed, 3 insertions(+), 3 deletions(-) diff --git a/avconv.c b/avconv.c

Re: [libav-devel] [PATCH] avconv: Replace raw picture frame swapping hack.

2011-08-30 Thread Anton Khirnov
On Tue, 30 Aug 2011 10:13:28 -0700, Alex Converse alex.conve...@gmail.com wrote: Non-text part: multipart/mixed On Tue, Aug 30, 2011 at 9:46 AM, Alex Converse alex.conve...@gmail.com wrote: On Mon, Aug 29, 2011 at 11:15 PM, Vladimir Pantelic vlado...@gmail.com wrote: Alex Converse

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

2011-08-30 Thread Alexander Strange
On Aug 25, 2011, at 9:15 PM, Sean McGovern wrote: On Thu, Aug 25, 2011 at 8:01 PM, Aaron Colwell acolw...@chromium.org wrote: Hi, I started a thread with the following message on the FFmpeg-devel list and someone suggested that I start a discussion on this list as well. Any help you could

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

2011-08-30 Thread Alexander Strange
On Aug 25, 2011, at 8:01 PM, Aaron Colwell wrote: Hi, I started a thread with the following message on the FFmpeg-devel list and someone suggested that I start a discussion on this list as well. Any help you could provide would be greatly appreciated. Thanks, Aaron --