Re: [libav-devel] [PATCH] tiff: add support to TIFF_SAMPLES_PER_PIXEL case in tiff_decode_tag()

2011-05-10 Thread Kostya
On Mon, May 09, 2011 at 12:51:24PM +0200, Stefano Sabatini wrote: > Also add support for bits per component storage. > > Fix decoding of file 11.tiff, trac issue number #167. > > Based on a patch by Kostya Shishkov . > --- > libavcodec/tiff.c | 138 > ---

Re: [libav-devel] [PATCH] ffmpeg: Simplify decode loop condition to not use next_pts

2011-05-10 Thread Alexander Strange
On May 10, 2011, at 6:57 AM, Ronald S. Bultje wrote: > Hi, > > On Tue, May 10, 2011 at 1:53 AM, Alexander Strange > wrote: >> --- >> ffmpeg.c | 15 --- >> 1 files changed, 8 insertions(+), 7 deletions(-) > > I've tested this for a while locally already, and this improves output

Re: [libav-devel] [PATCH 2/2] Allow resampling with no channel count change for up to 8 channels.

2011-05-10 Thread Justin Ruggles
On 05/10/2011 09:31 PM, Aℓex Converse wrote: > On Tue, May 10, 2011 at 6:15 PM, Justin Ruggles > wrote: >> > On 05/10/2011 08:00 PM, Aℓex Converse wrote: >> > > [...] > >>> >> @@ -291,12 +290,9 @@ int audio_resample(ReSampleContext *s, short >>> >> *output, short *input, int nb_sampl >>> >>

Re: [libav-devel] [PATCH 2/2] Port x86 10-bit H.264 deblock asm from x264

2011-05-10 Thread Ronald S. Bultje
Hi, On Tue, May 10, 2011 at 12:14 PM, Jason Garrett-Glaser wrote: > --- >  libavcodec/x86/dsputil_mmx.c          |    1 + >  libavcodec/x86/h264_deblock.asm       |   34 +- >  libavcodec/x86/h264_deblock_10bit.asm |  804 > + >  libavcodec/x86/h264dsp_mmx.c        

Re: [libav-devel] [PATCH 3/2] cosmetics: Fix crazy formatting in resample.

2011-05-10 Thread Aℓex Converse
On Tue, May 10, 2011 at 6:05 PM, Justin Ruggles wrote: > On 05/10/2011 08:04 PM, Aℓex Converse wrote: > >> I used indent. This is not perfect but it is good enough. I just need >> something that doesn't make my eyes bleed. If anyone feels more >> strongly about these formatting issues, I'm glad to

Re: [libav-devel] [PATCH 1/2] Update H.264 deblock asm

2011-05-10 Thread Ronald S. Bultje
Hi, On Tue, May 10, 2011 at 12:14 PM, Jason Garrett-Glaser wrote: > Includes AVX versions from x264. > --- >  libavcodec/x86/h264_deblock.asm |  395 > ++- >  libavcodec/x86/h264dsp_mmx.c    |   61 --- >  libavcodec/x86/x86util.asm      |   19 ++- >  3 file

Re: [libav-devel] [PATCH 2/2] Allow resampling with no channel count change for up to 8 channels.

2011-05-10 Thread Aℓex Converse
On Tue, May 10, 2011 at 6:15 PM, Justin Ruggles wrote: > On 05/10/2011 08:00 PM, Aℓex Converse wrote: > [...] >> From 5bcc724fd3a3b69c37d0419626c7b17913a09c9d Mon Sep 17 00:00:00 2001 >> From: Alex Converse >> Date: Tue, 10 May 2011 14:24:05 -0700 >> Subject: [PATCH 2/3] Allow resampling with n

Re: [libav-devel] [PATCH 1/2] Don't allow unsupported resampling configurations.

2011-05-10 Thread Justin Ruggles
On 05/10/2011 09:27 PM, Aℓex Converse wrote: > On Tue, May 10, 2011 at 4:01 PM, Alex Converse wrote: >> --- >> libavcodec/resample.c |5 + >> 1 files changed, 5 insertions(+), 0 deletions(-) >> > > Updated based on comments from [patch 2/2] ok. -Justin ___

Re: [libav-devel] [PATCH 1/2] Don't allow unsupported resampling configurations.

2011-05-10 Thread Aℓex Converse
On Tue, May 10, 2011 at 4:01 PM, Alex Converse wrote: > --- >  libavcodec/resample.c |    5 + >  1 files changed, 5 insertions(+), 0 deletions(-) > Updated based on comments from [patch 2/2] From d1309ce9ffd71d74b3e59efdd50375f8612f2262 Mon Sep 17 00:00:00 2001 From: Alex Converse Date: Tue,

Re: [libav-devel] [PATCH 2/2] Allow resampling with no channel count change for up to 8 channels.

2011-05-10 Thread Justin Ruggles
On 05/10/2011 08:00 PM, Aℓex Converse wrote: > On Tue, May 10, 2011 at 4:01 PM, Alex Converse wrote: >> > --- >> > libavcodec/resample.c | 83 >> > - >> > 1 files changed, 41 insertions(+), 42 deletions(-) >> > > Updated to drop an obsoleted com

Re: [libav-devel] [PATCH 3/2] cosmetics: Fix crazy formatting in resample.

2011-05-10 Thread Justin Ruggles
On 05/10/2011 08:04 PM, Aℓex Converse wrote: > I used indent. This is not perfect but it is good enough. I just need > something that doesn't make my eyes bleed. If anyone feels more > strongly about these formatting issues, I'm glad to turn this over to > you but you must act quickly. looks goo

Re: [libav-devel] [PATCH] fix fate failures for 10bit H264 on some systems

2011-05-10 Thread Ronald S. Bultje
Hi, On Tue, May 10, 2011 at 8:08 PM, Ronald S. Bultje wrote: > 2011/5/10 Måns Rullgård : >> "Ronald S. Bultje" writes: >>> @@ -703,9 +718,11 @@ static void FUNCC(pred8x8l_vertical)(uint8_t *_src, >>> int has_topleft, int has_top >>>      src[5] = t5; >>>      src[6] = t6; >>>      src[7] = t7;

Re: [libav-devel] [PATCH] fix fate failures for 10bit H264 on some systems

2011-05-10 Thread Ronald S. Bultje
Hi, 2011/5/10 Måns Rullgård : > "Ronald S. Bultje" writes: >>  #define PREDICT_16x16_DC(v)\ >>      for(i=0; i<16; i++){\ >> -        AV_WN4P(src+ 0, v);\ >> -        AV_WN4P(src+ 4, v);\ >> -        AV_WN4P(src+ 8, v);\ >> -        AV_WN4P(src+12, v);\ >> +        AV_WN4PA(src+ 0, v);\ >> +    

[libav-devel] [PATCH 3/2] cosmetics: Fix crazy formatting in resample.

2011-05-10 Thread Aℓex Converse
I used indent. This is not perfect but it is good enough. I just need something that doesn't make my eyes bleed. If anyone feels more strongly about these formatting issues, I'm glad to turn this over to you but you must act quickly. Regards, Alex From 74e8ec80bb50f3ecaa1918155c3cb83b1783ea51 Mon

Re: [libav-devel] [PATCH 2/2] Allow resampling with no channel count change for up to 8 channels.

2011-05-10 Thread Aℓex Converse
On Tue, May 10, 2011 at 4:01 PM, Alex Converse wrote: > --- >  libavcodec/resample.c |   83 > - >  1 files changed, 41 insertions(+), 42 deletions(-) > Updated to drop an obsoleted comment. From 5bcc724fd3a3b69c37d0419626c7b17913a09c9d Mon Sep 17 0

Re: [libav-devel] [PATCH 14/18] ffmpeg: improve reporting if size/pixel format changes

2011-05-10 Thread Stefano Sabatini
On date Tuesday 2011-05-10 18:25:24 -0400, Ronald S. Bultje encoded: > Hi, > > On Mon, May 9, 2011 at 1:49 PM, Anton Khirnov wrote: > > From: Stefano Sabatini > > > > Use av_log() rather than fprintf(stderr, ...), and show information > > related to the previous size/pixel format configuration.

Re: [libav-devel] [PATCH] fix fate failures for 10bit H264 on some systems

2011-05-10 Thread Måns Rullgård
"Ronald S. Bultje" writes: > Hi, > > as you can see, our fate is yellow since I committed H264/10bit. While > debugging, I found that h264pred appears to have some aliasing > problems, in the form of: > (pixel4=uint64_t) > > ((pixel4 *) dst)[x] = ((pixel4 *) src)[x]; > > ... and this is partially

Re: [libav-devel] [PATCH 3/4] bitstream: Properly promote av_reverse values before shifting.

2011-05-10 Thread Måns Rullgård
Aℓex Converse writes: > 2011/5/1 Måns Rullgård : >> Alex Converse writes: >> >>> On Thu, Apr 28, 2011 at 5:08 AM, Måns Rullgård wrote: Aℓex Converse writes: > From ae62aa2fca3b2cf45a98dc2f9777065bcb452797 Mon Sep 17 00:00:00 2001 > From: Alex Converse > Date: Tue, 26 Apr

[libav-devel] [PATCH 2/2] Allow resampling with no channel count change for up to 8 channels.

2011-05-10 Thread Alex Converse
--- libavcodec/resample.c | 83 - 1 files changed, 41 insertions(+), 42 deletions(-) diff --git a/libavcodec/resample.c b/libavcodec/resample.c index 8800098..06cbe94 100644 --- a/libavcodec/resample.c +++ b/libavcodec/resample.c @@ -29,6 +29,8 @@

[libav-devel] [PATCH 1/2] Don't allow unsupported resampling configurations.

2011-05-10 Thread Alex Converse
--- libavcodec/resample.c |5 + 1 files changed, 5 insertions(+), 0 deletions(-) diff --git a/libavcodec/resample.c b/libavcodec/resample.c index 2185a11..8800098 100644 --- a/libavcodec/resample.c +++ b/libavcodec/resample.c @@ -156,6 +156,11 @@ ReSampleContext *av_audio_resample_init(in

Re: [libav-devel] [FFmpeg-devel] [Announcement] FFmtech board of directors election

2011-05-10 Thread Thilo Borgmann
Am 09.05.11 23:50, schrieb Benjamin Larsson: > Hi, it's time to elect a new board for the FFmtech foundation. If you > want to candidate or participate in electing the new board then please > respond to this email. ... I want to vote. -Thilo ___ libav-

[libav-devel] [PATCH 2/2] Port x86 10-bit H.264 deblock asm from x264

2011-05-10 Thread Jason Garrett-Glaser
--- libavcodec/x86/dsputil_mmx.c |1 + libavcodec/x86/h264_deblock.asm | 34 +- libavcodec/x86/h264_deblock_10bit.asm | 804 + libavcodec/x86/h264dsp_mmx.c | 129 -- libavcodec/x86/x86util.asm|5 + 5 files changed,

[libav-devel] [PATCH 1/2] Update H.264 deblock asm

2011-05-10 Thread Jason Garrett-Glaser
Includes AVX versions from x264. --- libavcodec/x86/h264_deblock.asm | 395 ++- libavcodec/x86/h264dsp_mmx.c| 61 --- libavcodec/x86/x86util.asm | 19 ++- 3 files changed, 276 insertions(+), 199 deletions(-) diff --git a/libavcodec/x86/h264_de

Re: [libav-devel] [FFmpeg-devel] [PATCH] tiff: add support to TIFF_SAMPLES_PER_PIXEL case in tiff_decode_tag()

2011-05-10 Thread Stefano Sabatini
On date Monday 2011-05-09 22:27:44 +0200, Stefano Sabatini encoded: > Also add support for bits per component storage. > > Fix decoding of file 11.tiff, trac issue number #167. > > Based on a patch by Kostya Shiskov . Updated, CC-ed to libav-devel. >From f264eeb4d74f399f04b6a70536a5e9131bfd5283

Re: [libav-devel] [PATCH 06/18] ffmpeg: make 'bits_per_raw_sample' option more useful

2011-05-10 Thread Ronald S. Bultje
Hi, On Mon, May 9, 2011 at 1:49 PM, Anton Khirnov wrote: > From: Peter Ross > > Currently bits_per_raw_sample is exposed as an AVCodecContext option. > The option is not very useful, because ffmpeg 1) overwrites it with > a value from the upstream codec, or 2) it resets the value whenever > the

Re: [libav-devel] [PATCH 14/18] ffmpeg: improve reporting if size/pixel format changes

2011-05-10 Thread Ronald S. Bultje
Hi, On Mon, May 9, 2011 at 1:49 PM, Anton Khirnov wrote: > From: Stefano Sabatini > > Use av_log() rather than fprintf(stderr, ...), and show information > related to the previous size/pixel format configuration. > > Consistent with the corresponding message issued in case of audio > configurati

Re: [libav-devel] [PATCH 04/18] ffmpeg: warns the user when the selected pixel format is ignored

2011-05-10 Thread Ronald S. Bultje
Hi, On Mon, May 9, 2011 at 1:49 PM, Anton Khirnov wrote: > From: Stefano Sabatini > > Signed-off-by: Anton Khirnov > --- >  ffmpeg.c |    9 - >  1 files changed, 8 insertions(+), 1 deletions(-) > > diff --git a/ffmpeg.c b/ffmpeg.c > index bf80667..4671bd0 100644 > --- a/ffmpeg.c > +++ b

Re: [libav-devel] [PATCH 17/18] ffmpeg: reformat resample condition code in transcode()

2011-05-10 Thread Ronald S. Bultje
Hi, On Mon, May 9, 2011 at 1:49 PM, Anton Khirnov wrote: > From: Stefano Sabatini > > Signed-off-by: Stefano Sabatini > Signed-off-by: Anton Khirnov > --- >  ffmpeg.c |    6 +++--- >  1 files changed, 3 insertions(+), 3 deletions(-) > > diff --git a/ffmpeg.c b/ffmpeg.c > index 5e00e49..8d49b01

Re: [libav-devel] [PATCH 18/18] ffmpeg: call pre_process_video_frame() only if decoding is needed

2011-05-10 Thread Ronald S. Bultje
Hi, On Mon, May 9, 2011 at 1:49 PM, Anton Khirnov wrote: > From: Stefano Sabatini > > In output_packet(), move the pre_process_video_frame() call inside the > if (ist->decoding_needed) { } block. This way > pre_process_video_frame() is not called when stream-copy has been > selected. > > Also si

Re: [libav-devel] [PATCH 16/18] ffmpeg: factorize resampling condition check in do_video_out()

2011-05-10 Thread Ronald S. Bultje
Hi, On Mon, May 9, 2011 at 1:49 PM, Anton Khirnov wrote: > From: Stefano Sabatini > > Simplify and improve readability. > > Signed-off-by: Stefano Sabatini > Signed-off-by: Anton Khirnov > --- >  ffmpeg.c |   15 +++ >  1 files changed, 7 insertions(+), 8 deletions(-) OK. Ronald _

Re: [libav-devel] [PATCH 15/18] ffmpeg: prefer "dec" over "ist->st->codec" in do_video_out() snippet

2011-05-10 Thread Ronald S. Bultje
Hi, On Mon, May 9, 2011 at 1:49 PM, Anton Khirnov wrote: > From: Stefano Sabatini > > Simplify, ease readability. > > Signed-off-by: Stefano Sabatini > Signed-off-by: Anton Khirnov > --- >  ffmpeg.c |    4 ++-- >  1 files changed, 2 insertions(+), 2 deletions(-) > > diff --git a/ffmpeg.c b/ffm

[libav-devel] Libav-soc commits list created

2011-05-10 Thread Alex Converse
Greetings everyone, A new mailing list has been created to track Libav and FFmpeg soc commits. libav-...@libav.org https://lists.libav.org/mailman/listinfo/libav-soc Students: Please configure your hooks to email this list on each push. Then subscribe to the list yourself. (In Github this is und

[libav-devel] [PATCH] fix fate failures for 10bit H264 on some systems

2011-05-10 Thread Ronald S. Bultje
Hi, as you can see, our fate is yellow since I committed H264/10bit. While debugging, I found that h264pred appears to have some aliasing problems, in the form of: (pixel4=uint64_t) ((pixel4 *) dst)[x] = ((pixel4 *) src)[x]; ... and this is partially responsible for the fate failures. Basically

Re: [libav-devel] [PATCH 4/6] Fix libx264 fastfirstpass, move apply_fastfirstpass after user customized options. (cherry picked from commit 37c0a443f9aa01fcf33215b3cde06961f1e9df5c)

2011-05-10 Thread Aℓex Converse
On Tue, May 10, 2011 at 11:34 AM, Anton Khirnov wrote: > On Wed, 20 Apr 2011 15:56:16 -0700, Aℓex Converse > wrote: >> On Wed, Apr 20, 2011 at 3:40 PM, Alex Converse wrote: >> > --- >> >  libavcodec/libx264.c |   14 +++--- >> >  1 files changed, 7 insertions(+), 7 deletions(-) >> > >> >

Re: [libav-devel] [PATCH 4/6] Fix libx264 fastfirstpass, move apply_fastfirstpass after user customized options. (cherry picked from commit 37c0a443f9aa01fcf33215b3cde06961f1e9df5c)

2011-05-10 Thread Anton Khirnov
On Wed, 20 Apr 2011 15:56:16 -0700, Aℓex Converse wrote: > On Wed, Apr 20, 2011 at 3:40 PM, Alex Converse wrote: > > --- > >  libavcodec/libx264.c |   14 +++--- > >  1 files changed, 7 insertions(+), 7 deletions(-) > > > > > From abb44bdc8e25a75c3beaf47fd23bc9ca48490ccc Mon Sep 17 00:00:0

Re: [libav-devel] [PATCH 2/2] preliminary Flash Screen Video v2 decoder from Kostya

2011-05-10 Thread Daniel Kang
On Wed, May 4, 2011 at 1:00 PM, Ronald S. Bultje wrote: > > +if (s->ver == 2) { > > +skip_bits(&gb, 6); > > +if (get_bits1(&gb)) { > > +av_log(avctx, AV_LOG_ERROR, "FIXME iframe not implemented > yet\n"); > > +return -1; > > +} > > +if (

Re: [libav-devel] [PATCH 3/4] bitstream: Properly promote av_reverse values before shifting.

2011-05-10 Thread Aℓex Converse
2011/5/1 Måns Rullgård : > Alex Converse writes: > >> On Thu, Apr 28, 2011 at 5:08 AM, Måns Rullgård wrote: >>> Aℓex Converse writes: >>> From ae62aa2fca3b2cf45a98dc2f9777065bcb452797 Mon Sep 17 00:00:00 2001 From: Alex Converse Date: Tue, 26 Apr 2011 09:08:26 -0700 Subject:

Re: [libav-devel] [PATCH] AVOptions: make default_val a union, as proposed in AVOption2.

2011-05-10 Thread Reinhard Tartler
On Tue, May 10, 2011 at 19:46:08 (CEST), Ronald S. Bultje wrote: > Hi, > > On Tue, May 3, 2011 at 6:37 AM, Anton Khirnov wrote: >> This breaks API and ABI. >> --- >>  libavcodec/ac3enc.c      |   70 +++--- >>  libavcodec/dnxhdenc.c    |    2 +- >>  libavcodec/libvorbis.c   |    2 +- >>  libavcode

Re: [libav-devel] [PATCH] AVOptions: make default_val a union, as proposed in AVOption2.

2011-05-10 Thread Ronald S. Bultje
Hi, On Tue, May 3, 2011 at 6:37 AM, Anton Khirnov wrote: > This breaks API and ABI. > --- >  libavcodec/ac3enc.c      |   70 +++--- >  libavcodec/dnxhdenc.c    |    2 +- >  libavcodec/libvorbis.c   |    2 +- >  libavcodec/options.c     |  764 > +++--- >  l

[libav-devel] [PATCH 1/2] mdec.c: fix overread.

2011-05-10 Thread Ronald S. Bultje
--- libavcodec/mdec.c |3 ++- 1 files changed, 2 insertions(+), 1 deletions(-) diff --git a/libavcodec/mdec.c b/libavcodec/mdec.c index 545b919..9b6e6b6 100644 --- a/libavcodec/mdec.c +++ b/libavcodec/mdec.c @@ -125,7 +125,8 @@ static inline int decode_mb(MDECContext *a, DCTELEM block[6][64]

[libav-devel] [PATCH 2/2] mdec: enable frame-level multithreading.

2011-05-10 Thread Ronald S. Bultje
From: Alexander Strange Signed-off-by: Ronald S. Bultje --- libavcodec/mdec.c | 20 +--- 1 files changed, 17 insertions(+), 3 deletions(-) diff --git a/libavcodec/mdec.c b/libavcodec/mdec.c index 9b6e6b6..17671cf 100644 --- a/libavcodec/mdec.c +++ b/libavcodec/mdec.c @@ -31,6

Re: [libav-devel] x264: allow setting open/glosed GOP

2011-05-10 Thread Jindřich Makovička
2011/5/10 Jindřich Makovička : > Hi, > > the attached patch adds support for setting of the b_open_gop > parameter in libx264 via "cgop" codec flag. It also updates the > presets to keep encoding with closed GOP by default. Updated for current libx264. -- Jindrich Makovicka 0001-libx264-handle

[libav-devel] [RFC] Fix asfdec "regression"

2011-05-10 Thread Vladimir Pantelic
Uoti Urpala wrote: Attached patch fixes an assertion failure that's fairly easy to trigger with corrupt files. The asf seek regression triggered by the preroll changes (caused by binary seek fallback) is still there, anyone going to fix that? yes, the preroll changes now expose that asf bug.

Re: [libav-devel] [PATCH] arm: attempt to fix compilation.

2011-05-10 Thread Kostya
On Tue, May 10, 2011 at 08:42:10AM -0400, Ronald S. Bultje wrote: > --- > libavcodec/arm/h264pred_init_arm.c |2 +- > 1 files changed, 1 insertions(+), 1 deletions(-) > > diff --git a/libavcodec/arm/h264pred_init_arm.c > b/libavcodec/arm/h264pred_init_arm.c > index cae32d7..e96f339 100644 >

[libav-devel] [PATCH] arm: attempt to fix compilation.

2011-05-10 Thread Ronald S. Bultje
--- libavcodec/arm/h264pred_init_arm.c |2 +- 1 files changed, 1 insertions(+), 1 deletions(-) diff --git a/libavcodec/arm/h264pred_init_arm.c b/libavcodec/arm/h264pred_init_arm.c index cae32d7..e96f339 100644 --- a/libavcodec/arm/h264pred_init_arm.c +++ b/libavcodec/arm/h264pred_init_arm.c

Re: [libav-devel] [PATCH] h264dsp_mmx: place bracket outside #if/#endif block.

2011-05-10 Thread Kostya
On Tue, May 10, 2011 at 08:39:49AM -0400, Ronald S. Bultje wrote: > Should fix compile on systems missing yasm/nasm. > --- > libavcodec/x86/h264dsp_mmx.c |2 +- > 1 files changed, 1 insertions(+), 1 deletions(-) > > diff --git a/libavcodec/x86/h264dsp_mmx.c b/libavcodec/x86/h264dsp_mmx.c > in

[libav-devel] [PATCH] h264dsp_mmx: place bracket outside #if/#endif block.

2011-05-10 Thread Ronald S. Bultje
Should fix compile on systems missing yasm/nasm. --- libavcodec/x86/h264dsp_mmx.c |2 +- 1 files changed, 1 insertions(+), 1 deletions(-) diff --git a/libavcodec/x86/h264dsp_mmx.c b/libavcodec/x86/h264dsp_mmx.c index b4936a6..3a783a3 100644 --- a/libavcodec/x86/h264dsp_mmx.c +++ b/libavcodec/

[libav-devel] x264: allow setting open/glosed GOP

2011-05-10 Thread Jindřich Makovička
Hi, the attached patch adds support for setting of the b_open_gop parameter in libx264 via "cgop" codec flag. It also updates the presets to keep encoding with closed GOP by default. Regards, -- Jindrich Makovicka 0001-libx264-handle-closed-GOP-codec-flag.patch Description: Binary data ___

Re: [libav-devel] [PATCH] Fix behavior introduced by commit 96573c0d7605672d69b42ae1dcf18764ce47c71a.

2011-05-10 Thread Ronald S. Bultje
Hi, On Sun, May 8, 2011 at 4:17 PM, Ronald S. Bultje wrote: > On Sun, May 8, 2011 at 3:29 PM, Vitor Sessak wrote: >> On 05/08/2011 09:22 PM, Måns Rullgård wrote: >>> Vitor Sessak  writes: On 05/08/2011 09:03 PM, Luca Barbato wrote: > On 5/8/11 3:35 PM, Vitor Sessak wrote: >> On 05/0

Re: [libav-devel] [PATCH 04/18] Preparatory patch for high bit depth h264 decoding support.

2011-05-10 Thread Kostya
On Mon, May 09, 2011 at 12:52:46PM -0400, Ronald S. Bultje wrote: > From: Oskar Arvidsson > > Signed-off-by: Ronald S. Bultje > --- > libavcodec/h264dsp.c | 259 +- > libavcodec/h264dsp_template.c | 285 > +

Re: [libav-devel] [PATCH 15/18] Adds 8-, 9- and 10-bit versions of some of the functions used by the h264 decoder.

2011-05-10 Thread Kostya
On Mon, May 09, 2011 at 04:36:25PM -0400, Ronald S. Bultje wrote: > Hi, > > On Mon, May 9, 2011 at 12:52 PM, Ronald S. Bultje wrote: > > From: Oskar Arvidsson > > > > This patch lets e.g. dsputil_init chose dsp functions with respect to > > the bit depth to decode. The naming scheme of bit depth

Re: [libav-devel] [PATCH 08/18] Preparatory patch for high bit depth h264 decoding support.

2011-05-10 Thread Kostya
On Mon, May 09, 2011 at 12:52:50PM -0400, Ronald S. Bultje wrote: > From: Oskar Arvidsson > > Signed-off-by: Ronald S. Bultje > --- > libavcodec/h264dsp_template.c | 119 > ++--- > 1 files changed, 75 insertions(+), 44 deletions(-) lgtm ___

Re: [libav-devel] [PATCH 11/18] Template dsputil_template.c with respect to pixel size, etc.

2011-05-10 Thread Kostya
On Mon, May 09, 2011 at 12:52:53PM -0400, Ronald S. Bultje wrote: > From: Oskar Arvidsson > > Preparatory patch for high bit depth h264 decoding support. > > Signed-off-by: Ronald S. Bultje > --- > libavcodec/dsputil_template.c | 704 > +++-- > 1 files cha

Re: [libav-devel] [PATCH 10/18] Template h264idct_template.c with respect to pixel size, etc.

2011-05-10 Thread Kostya
On Mon, May 09, 2011 at 12:52:52PM -0400, Ronald S. Bultje wrote: > From: Oskar Arvidsson > > Preparatory patch for high bit depth h264 decoding support. > > Signed-off-by: Ronald S. Bultje > --- > libavcodec/h264idct_template.c | 109 +++ > 1 files changed

Re: [libav-devel] [PATCH 09/18] Template h264pred_template.c with respect to bit depth, etc.

2011-05-10 Thread Kostya
On Mon, May 09, 2011 at 04:35:37PM -0400, Ronald S. Bultje wrote: > Hi, > > On Mon, May 9, 2011 at 12:52 PM, Ronald S. Bultje wrote: > > From: Oskar Arvidsson > > > > Preparatory patch for high bit depth h264 decoding support. > > > > Signed-off-by: Ronald S. Bultje > > --- > >  libavcodec/h264

Re: [libav-devel] [PATCH] ffmpeg: Simplify decode loop condition to not use next_pts

2011-05-10 Thread Ronald S. Bultje
Hi, On Tue, May 10, 2011 at 1:53 AM, Alexander Strange wrote: > --- >  ffmpeg.c |   15 --- >  1 files changed, 8 insertions(+), 7 deletions(-) I've tested this for a while locally already, and this improves output of terminal frames with MT enabled. Definitely OK for me. Ronald

Re: [libav-devel] [PATCH] Add a dummy option -i to ffplay so that it is easy to switch between ffmpeg -i "file" and ffplay -i "file".

2011-05-10 Thread Ronald S. Bultje
Hi, On Tue, May 10, 2011 at 6:30 AM, Ronald S. Bultje wrote: > On Tue, May 10, 2011 at 6:24 AM, Anton Khirnov wrote: >> From: Benjamin Larsson >> >> Signed-off-by: Anton Khirnov >> --- >>  cmdutils.c |    3 ++- >>  ffplay.c   |    1 + >>  2 files changed, 3 insertions(+), 1 deletions(-) >> >>

Re: [libav-devel] [PATCH] Add a dummy option -i to ffplay so that it is easy to switch between ffmpeg -i "file" and ffplay -i "file".

2011-05-10 Thread Ronald S. Bultje
Hi, On Tue, May 10, 2011 at 6:24 AM, Anton Khirnov wrote: > From: Benjamin Larsson > > Signed-off-by: Anton Khirnov > --- >  cmdutils.c |    3 ++- >  ffplay.c   |    1 + >  2 files changed, 3 insertions(+), 1 deletions(-) > > diff --git a/cmdutils.c b/cmdutils.c > index f957788..fbfe317 100644

[libav-devel] [PATCH] Add a dummy option -i to ffplay so that it is easy to switch between ffmpeg -i "file" and ffplay -i "file".

2011-05-10 Thread Anton Khirnov
From: Benjamin Larsson Signed-off-by: Anton Khirnov --- cmdutils.c |3 ++- ffplay.c |1 + 2 files changed, 3 insertions(+), 1 deletions(-) diff --git a/cmdutils.c b/cmdutils.c index f957788..fbfe317 100644 --- a/cmdutils.c +++ b/cmdutils.c @@ -282,7 +282,8 @@ unknown_opt:

Re: [libav-devel] [PATCH 10/18] ffmpeg: use parse_number_and_die() when it makes sense

2011-05-10 Thread Stefano Sabatini
On date Monday 2011-05-09 20:25:31 -0400, Ronald S. Bultje encoded: > Hi, > > On Mon, May 9, 2011 at 7:09 PM, Stefano Sabatini > wrote: > > On date Monday 2011-05-09 18:06:51 -0400, Ronald S. Bultje encoded: > >> Hi, > >> > >> On Mon, May 9, 2011 at 1:49 PM, Anton Khirnov wrote: > >> > -static v