Re: [libav-devel] [PATCH] lagarith: add decode support for arith rgb24 mode

2011-12-28 Thread Kostya Shishkov
On Tue, Dec 27, 2011 at 11:45:55PM -0600, Nathan Caldwell wrote: > --- > libavcodec/lagarith.c | 32 ++-- > 1 files changed, 22 insertions(+), 10 deletions(-) LGTM, just don't forget to change author's name or mention it's just slightly modified Thad's patch. ___

Re: [libav-devel] [PATCH 02/11] rawdec: Support decoding of 1 bpp and 2bpp rawvideo in avi.

2011-12-28 Thread Kostya Shishkov
On Wed, Dec 28, 2011 at 12:50:26AM -0500, Derek Buitenhuis wrote: > From: ami_stuff > > --- > libavcodec/rawdec.c |2 ++ > 1 files changed, 2 insertions(+), 0 deletions(-) > > diff --git a/libavcodec/rawdec.c b/libavcodec/rawdec.c > index dcbe23c..dfa5c37 100644 > --- a/libavcodec/rawdec.c

Re: [libav-devel] [PATCH 05/11] rawdec: Fix linesize for some rawvideo samples with odd width.

2011-12-28 Thread Kostya Shishkov
On Wed, Dec 28, 2011 at 12:50:29AM -0500, Derek Buitenhuis wrote: > From: Carl Eugen Hoyos > > --- > libavcodec/rawdec.c |8 +++- > 1 files changed, 7 insertions(+), 1 deletions(-) > > diff --git a/libavcodec/rawdec.c b/libavcodec/rawdec.c > index ddd8583..340278b 100644 > --- a/libavco

Re: [libav-devel] [PATCH 11/11] rawdec: use a default sample rate if none is specified. Fixes "ffmpeg -f s16le -i /dev/zero"

2011-12-28 Thread Kostya Shishkov
On Wed, Dec 28, 2011 at 12:50:35AM -0500, Derek Buitenhuis wrote: > From: Michael Niedermayer > > Signed-off-by: Michael Niedermayer > --- > libavformat/rawdec.c |4 ++-- > 1 files changed, 2 insertions(+), 2 deletions(-) > > diff --git a/libavformat/rawdec.c b/libavformat/rawdec.c > index

Re: [libav-devel] FATE : test for zmbv

2011-12-28 Thread Oana Andreea Stratulat
In lcov the coverage was 0%thats why i made those tests. But the tests are in screen.mak. On Dec 28, 2011, at 4:25 AM, Ronald S. Bultje wrote: > Hi, > > On Tue, Dec 27, 2011 at 5:57 PM, Derek Buitenhuis > wrote: >>> --- >>> tests/Makefile|4 >>> tests/fate/zmbv_15bit.mak |

Re: [libav-devel] [PATCH 11/11] rawdec: use a default sample rate if none is specified. Fixes "ffmpeg -f s16le -i /dev/zero"

2011-12-28 Thread Anton Khirnov
On Wed, 28 Dec 2011 09:23:31 +0100, Kostya Shishkov wrote: > On Wed, Dec 28, 2011 at 12:50:35AM -0500, Derek Buitenhuis wrote: > > From: Michael Niedermayer > > > > Signed-off-by: Michael Niedermayer > > --- > > libavformat/rawdec.c |4 ++-- > > 1 files changed, 2 insertions(+), 2 deleti

Re: [libav-devel] [PATCH 11/11] rawdec: use a default sample rate if none is specified. Fixes "ffmpeg -f s16le -i /dev/zero"

2011-12-28 Thread Kostya Shishkov
On Wed, Dec 28, 2011 at 10:56:34AM +0100, Anton Khirnov wrote: > > On Wed, 28 Dec 2011 09:23:31 +0100, Kostya Shishkov > wrote: > > On Wed, Dec 28, 2011 at 12:50:35AM -0500, Derek Buitenhuis wrote: > > > From: Michael Niedermayer > > > > > > Signed-off-by: Michael Niedermayer > > > --- > > >

Re: [libav-devel] [libav-commits] mpegvideo_enc: K&R cosmetics

2011-12-28 Thread Diego Biurrun
On Tue, Dec 27, 2011 at 07:29:55PM +0100, Aneesh Dogra wrote: > Module: libav > Branch: master > Commit: bd96be6e2739dbe5b7a467a318ebfb6241c15eba > > Author:Aneesh Dogra > Committer: Ronald S. Bultje > Date: Tue Dec 27 11:56:07 2011 +0530 > > mpegvideo_enc: K&R cosmetics > > Signed-of

Re: [libav-devel] [PATCH] fate: Add tests for xxan decoder

2011-12-28 Thread Diego Biurrun
On Wed, Dec 28, 2011 at 12:51:19PM +0530, Aneesh Dogra wrote: > --- > tests/fate2.mak |9 + > tests/ref/fate/xxan_wc3 | 1024 > + > tests/ref/fate/xxan_wc4 | 222 ++ > tests/ref/fate/xxan_wc4_2 | 68 +++ > 4 files changed, 1

[libav-devel] [PATCH] GCI task : code clean up 4xm.c, timefilter.c .

2011-12-28 Thread Yordan Makariev
--- libavcodec/4xm.c | 724 -- libavdevice/timefilter.c | 75 +++--- 2 files changed, 414 insertions(+), 385 deletions(-) diff --git a/libavcodec/4xm.c b/libavcodec/4xm.c index 83f0a12..9bf9734 100644 --- a/libavcodec/4xm.c +++ b/libavcodec/4

[libav-devel] random failures of fate-vsynth1-dnxhd_720p_rd

2011-12-28 Thread Michael Kostylev
Sometimes the vsynth1-dnxhd_720p_rd test fails on alpha, pa-risc and sparc FATE machines. It explicitly uses '-threads 4' when encoding, and for some reason vsynth1/dnxhd-720p-rd.dnxhd may be spoiled (the first frame contains a small number of randomly placed zero blocks). For comparison, the sing

Re: [libav-devel] [PATCH] GCI task : code clean up 4xm.c, timefilter.c .

2011-12-28 Thread Diego Biurrun
On Wed, Dec 28, 2011 at 01:18:16PM +0200, Yordan Makariev wrote: > --- > libavcodec/4xm.c | 724 > -- > libavdevice/timefilter.c | 75 +++--- > 2 files changed, 414 insertions(+), 385 deletions(-) I already reviewed your 4xm.c changes on Dec

Re: [libav-devel] [PATCH 2/2] configure: C3-2 supports cmov

2011-12-28 Thread Diego Biurrun
On Fri, Dec 16, 2011 at 06:18:27PM +, Måns Rullgård wrote: > Diego Biurrun writes: > > > > --- a/configure > > +++ b/configure > > @@ -2275,7 +2275,7 @@ elif enabled x86; then > > # targets that do support conditional mov (cmov) > > - > > i686|pentiumpro|pentium[23]|pentium-m|

[libav-devel] FATE : test for kgv1

2011-12-28 Thread Oana Andreea Stratulat
The samples are in the fate suite but no tests were made. link to sample file : http://google-melange.appspot.com/gci/work/download/google/gci2011/7129328?id=4 0001-FATE-test-for-kgv1.patch Description: Binary data ___ libav-devel mailing list libav-d

[libav-devel] [PATCH] applehttpproto: the same reload interval fix

2011-12-28 Thread Andrey Utkin
Applied for applehttpproto same changes that are in my and Martin's patches. --- libavformat/applehttpproto.c | 20 ++-- 1 files changed, 14 insertions(+), 6 deletions(-) diff --git a/libavformat/applehttpproto.c b/libavformat/applehttpproto.c index 2b15709..eb96b0a 100644 ---

Re: [libav-devel] [PATCH 2/2] applehttp: Use half the target duration as interval if the playlist didn't update

2011-12-28 Thread Andrey Utkin
Tested both of your patches during last couple of days, no issues. Please consider committing. 2011/12/27 Martin Storsjö : > This is mandated in draft-pantos-http-live-streaming-07, section > 6.3.4. > --- >  libavformat/applehttp.c |   11 --- >  1 files changed, 8 insertions(+), 3 deletion

Re: [libav-devel] [PATCH] fate: Add v210 codec regression tests

2011-12-28 Thread Ronald S. Bultje
Hi, On Tue, Dec 27, 2011 at 8:58 PM, Derek Buitenhuis wrote: > On 27/12/2011 11:50 PM, Ronald S. Bultje wrote: >> That can barely be OK, a pretty-much-lossless format like this should >> give a PSNR of>40 and a MAXDIFF of perhaps 1 or so. What is it >> testing here? > > It's testing against the 8

Re: [libav-devel] FATE : test for kgv1

2011-12-28 Thread Ronald S. Bultje
Hi, On Wed, Dec 28, 2011 at 4:47 AM, Oana Andreea Stratulat wrote: > The samples are in the fate suite but no tests were made. > link to sample file : > http://google-melange.appspot.com/gci/work/download/google/gci2011/7129328?id=4 Please check http://roundup.libav.org/issue2096 - does it stil

Re: [libav-devel] FATE : test for kgv1

2011-12-28 Thread Ronald S. Bultje
Hi, On Wed, Dec 28, 2011 at 7:26 AM, Ronald S. Bultje wrote: > On Wed, Dec 28, 2011 at 4:47 AM, Oana Andreea Stratulat > wrote: >> The samples are in the fate suite but no tests were made. >> link to sample file : >> http://google-melange.appspot.com/gci/work/download/google/gci2011/7129328?id=

Re: [libav-devel] [PATCH] Indeo 4 decoder

2011-12-28 Thread Ronald S. Bultje
Hi, On Tue, Dec 27, 2011 at 11:06 AM, Kostya Shishkov wrote: > On Tue, Dec 27, 2011 at 07:50:58PM +0100, Kostya Shishkov wrote: >> On Tue, Dec 27, 2011 at 07:29:44PM +0100, Diego Biurrun wrote: >> > On Sat, Dec 24, 2011 at 05:23:00PM +0100, Kostya Shishkov wrote: >> > > >> > > Here it is - more f

Re: [libav-devel] [PATCH] Indeo 4 decoder

2011-12-28 Thread Kostya Shishkov
On Wed, Dec 28, 2011 at 07:57:18AM -0800, Ronald S. Bultje wrote: > Hi, > > On Tue, Dec 27, 2011 at 11:06 AM, Kostya Shishkov > wrote: > > On Tue, Dec 27, 2011 at 07:50:58PM +0100, Kostya Shishkov wrote: > >> On Tue, Dec 27, 2011 at 07:29:44PM +0100, Diego Biurrun wrote: > >> > On Sat, Dec 24, 20

Re: [libav-devel] random failures of fate-vsynth1-dnxhd_720p_rd

2011-12-28 Thread Sean McGovern
On Wednesday, December 28, 2011, Michael Kostylev < michael.kosty...@gmail.com> wrote: > > Sometimes the vsynth1-dnxhd_720p_rd test fails on alpha, pa-risc and sparc > FATE machines. It explicitly uses '-threads 4' when encoding, and for some > reason vsynth1/dnxhd-720p-rd.dnxhd may be spoiled (the

Re: [libav-devel] [PATCH] fate: Add v210 codec regression tests

2011-12-28 Thread Måns Rullgård
"Ronald S. Bultje" writes: > Hi, > > On Tue, Dec 27, 2011 at 8:58 PM, Derek Buitenhuis > wrote: >> On 27/12/2011 11:50 PM, Ronald S. Bultje wrote: >>> That can barely be OK, a pretty-much-lossless format like this should >>> give a PSNR of>40 and a MAXDIFF of perhaps 1 or so. What is it >>> test

Re: [libav-devel] [PATCH 2/2] configure: C3-2 supports cmov

2011-12-28 Thread Måns Rullgård
Diego Biurrun writes: > On Fri, Dec 16, 2011 at 06:18:27PM +, Måns Rullgård wrote: >> Diego Biurrun writes: >> > >> > --- a/configure >> > +++ b/configure >> > @@ -2275,7 +2275,7 @@ elif enabled x86; then >> > # targets that do support conditional mov (cmov) >> > - >> > i686

Re: [libav-devel] FATE : test for xsub

2011-12-28 Thread Måns Rullgård
"Ronald S. Bultje" writes: > Hi, > > 2011/12/27 Måns Rullgård : >> "Ronald S. Bultje" writes: >>> On Mon, Dec 26, 2011 at 4:52 PM, Oana Andreea Stratulat >>> wrote: >>> [..] >>> >>> Note for others: this does test the decoder; but does so by encoding >>> into another format, because we have no

Re: [libav-devel] LibAV-FATE on Solaris x86

2011-12-28 Thread Måns Rullgård
Sean McGovern writes: > On Tue, Dec 27, 2011 at 7:19 PM, Sean McGovern wrote: >> 2011/10/21 Måns Rullgård : >>> "Sean McGovern" writes: >>> > > [snip..] > >>> the same thing.  Since they probably won't, installing a simple wrapper >>> as /bin/sh will work as well.  Something like this should do

Re: [libav-devel] [libav-commits] v4l2: remove unneded linux specific asm/types.h include

2011-12-28 Thread Måns Rullgård
Derek Buitenhuis writes: > On 27/12/2011 1:36 PM, Reinhard Tartler wrote: >> Looks great to me, this unbreaks compilation on Debian/kFreeBSD. > > http://git.videolan.org/?p=ffmpeg.git;a=commitdiff;h=ffa1f541b350946591ea80e134917b28a860fcad > > It seems FFmpeg has fix for his already. Theirs looks

Re: [libav-devel] [PATCH 01/11] rawdec: Support more QT 1bpp rawvideo files.

2011-12-28 Thread Måns Rullgård
Derek Buitenhuis writes: > From: ami_stuff Does this person have a real name? I don't like taking patches anonymously. -- Måns Rullgård m...@mansr.com ___ libav-devel mailing list libav-devel@libav.org https://lists.libav.org/mailman/listinfo/libav

Re: [libav-devel] [PATCH 05/11] rawdec: Fix linesize for some rawvideo samples with odd width.

2011-12-28 Thread Måns Rullgård
Kostya Shishkov writes: > On Wed, Dec 28, 2011 at 12:50:29AM -0500, Derek Buitenhuis wrote: >> From: Carl Eugen Hoyos >> >> --- >> libavcodec/rawdec.c |8 +++- >> 1 files changed, 7 insertions(+), 1 deletions(-) >> >> diff --git a/libavcodec/rawdec.c b/libavcodec/rawdec.c >> index ddd

Re: [libav-devel] [PATCH 06/11] rawdec: Fix internal buffer size for 2bpp and 4bpp raw video.

2011-12-28 Thread Måns Rullgård
Derek Buitenhuis writes: > From: Carl Eugen Hoyos > > --- > libavcodec/rawdec.c |4 +++- > 1 files changed, 3 insertions(+), 1 deletions(-) > > diff --git a/libavcodec/rawdec.c b/libavcodec/rawdec.c > index 340278b..0051425 100644 > --- a/libavcodec/rawdec.c > +++ b/libavcodec/rawdec.c > @@

Re: [libav-devel] [PATCH 10/11] rawdec: add check on sample_rate

2011-12-28 Thread Måns Rullgård
Derek Buitenhuis writes: > From: Stefano Sabatini > > Prevent error condition in case sample_rate is unset or set to a negative > value. In particular, fix divide-by-zero error occurring in ffmpeg due to > sample_rate set to 0 in output_packet(), in code: > > ist->next_pts += ((i

Re: [libav-devel] [PATCH 11/11] rawdec: use a default sample rate if none is specified. Fixes "ffmpeg -f s16le -i /dev/zero"

2011-12-28 Thread Måns Rullgård
Anton Khirnov writes: > On Wed, 28 Dec 2011 09:23:31 +0100, Kostya Shishkov > wrote: >> On Wed, Dec 28, 2011 at 12:50:35AM -0500, Derek Buitenhuis wrote: >> > From: Michael Niedermayer >> > >> > Signed-off-by: Michael Niedermayer >> > --- >> > libavformat/rawdec.c |4 ++-- >> > 1 files

Re: [libav-devel] [PATCH 05/11] rawdec: Fix linesize for some rawvideo samples with odd width.

2011-12-28 Thread Kostya Shishkov
On Wed, Dec 28, 2011 at 04:45:11PM +, Måns Rullgård wrote: > Kostya Shishkov writes: > > > On Wed, Dec 28, 2011 at 12:50:29AM -0500, Derek Buitenhuis wrote: > >> From: Carl Eugen Hoyos > >> > >> --- > >> libavcodec/rawdec.c |8 +++- > >> 1 files changed, 7 insertions(+), 1 deletion

[libav-devel] [PATCH] GCI task : code clean up 4xm.c, timefilter.c . I changed 4xm.c and I send timefilter.c to understand did I remove the trailing whitespace and do I have other mistakes :)

2011-12-28 Thread Yordan Makariev
--- libavcodec/4xm.c | 746 -- libavdevice/timefilter.c | 81 +++--- 2 files changed, 428 insertions(+), 399 deletions(-) diff --git a/libavcodec/4xm.c b/libavcodec/4xm.c index 83f0a12..42d6491 100644 --- a/libavcodec/4xm.c +++ b/libavcodec/4

Re: [libav-devel] FATE : test for zmbv

2011-12-28 Thread Ronald S. Bultje
Hi, On Wed, Dec 28, 2011 at 1:04 AM, Oana Andreea Stratulat wrote: > In lcov the coverage was 0%thats why i made those tests. But the tests are in > screen.mak. Are they disabled because you don't have libz or the dev headers for libz installed? Did you confirm that after adding the tests, cov

Re: [libav-devel] [PATCH] avformat: Add SMJPEG demuxer.

2011-12-28 Thread Paul B Mahol
On 12/24/11, Paul B Mahol wrote: > Signed-off-by: Paul B Mahol > --- > Changelog|1 + > doc/general.texi |2 + > libavformat/Makefile |1 + > libavformat/allformats.c |1 + > libavformat/smjpeg.c | 187 > +++

Re: [libav-devel] [PATCH 01/11] rawdec: Support more QT 1bpp rawvideo files.

2011-12-28 Thread Derek Buitenhuis
On 28/12/2011 11:43 AM, Måns Rullgård wrote: Does this person have a real name? I don't like taking patches anonymously. Not as far as I know, no. He's got some stuff in Libav too already. I don't see it as a huge issue, especially for something so tiny. - Derel _

Re: [libav-devel] [PATCH 05/11] rawdec: Fix linesize for some rawvideo samples with odd width.

2011-12-28 Thread Derek Buitenhuis
On 28/12/2011 12:02 PM, Kostya Shishkov wrote: looks stupid > > Disgusting even. I do agree, but may I remind you we recently pushed this: http://git.libav.org/?p=libav.git;a=commitdiff;h=b4dc68803bd7fa73c2e37f55ae73b5fdb293c68d I'm all for a better solution, but I don't hear anyone offerin

Re: [libav-devel] [PATCH 11/11] rawdec: use a default sample rate if none is specified. Fixes "ffmpeg -f s16le -i /dev/zero"

2011-12-28 Thread Derek Buitenhuis
On 28/12/2011 11:52 AM, Måns Rullgård wrote: > I'm against this patch. There is absolutely no reason to assume any > specific samplerate here and a wrong samplerate will produce garbage. > The user should always set a samplerate. +100 If the user failed to specify anything, no value is more

Re: [libav-devel] [PATCH 11/11] rawdec: use a default sample rate if none is specified. Fixes "ffmpeg -f s16le -i /dev/zero"

2011-12-28 Thread Derek Buitenhuis
On 28/12/2011 12:48 PM, Derek Buitenhuis wrote: I am inclined to agree with you both. Note to self: Do not cherry-pick patches at 2am. - Derek Which of course means patch dropped. - Derek ___ libav-devel mailing list libav-devel@libav.org https://l

Re: [libav-devel] [PATCH 10/11] rawdec: add check on sample_rate

2011-12-28 Thread Derek Buitenhuis
On 28/12/2011 11:51 AM, Måns Rullgård wrote: Looks like it belongs in a more central place to avoid scattering such checks across all demuxers. Hasn't this come up before Good question. I do sort of wonder... why is sample_rate signed at all? How does a negative sample rate even make sense?

Re: [libav-devel] [PATCH 02/11] rawdec: Support decoding of 1 bpp and 2bpp rawvideo in avi.

2011-12-28 Thread Derek Buitenhuis
On 28/12/2011 3:15 AM, Kostya Shishkov wrote: Since AVI video is related to BMP I suspect that 1-bpp case might require PAL8 too. BMP stores palette for bpp < 15 after all. Hmm. Is our PAL8 support still broken? - Derek ___ libav-devel mailing list l

Re: [libav-devel] [PATCH 10/11] rawdec: add check on sample_rate

2011-12-28 Thread Kostya Shishkov
On Wed, Dec 28, 2011 at 12:51:18PM -0500, Derek Buitenhuis wrote: > On 28/12/2011 11:51 AM, Måns Rullgård wrote: > >Looks like it belongs in a more central place to avoid scattering such > >checks across all demuxers. Hasn't this come up before > > Good question. > > I do sort of wonder... why i

Re: [libav-devel] [PATCH 02/11] rawdec: Support decoding of 1 bpp and 2bpp rawvideo in avi.

2011-12-28 Thread Kostya Shishkov
On Wed, Dec 28, 2011 at 12:52:36PM -0500, Derek Buitenhuis wrote: > On 28/12/2011 3:15 AM, Kostya Shishkov wrote: > >Since AVI video is related to BMP I suspect that 1-bpp case might require > >PAL8 > >too. BMP stores palette for bpp < 15 after all. > > Hmm. Is our PAL8 support still broken? sti

Re: [libav-devel] [PATCH 10/11] rawdec: add check on sample_rate

2011-12-28 Thread Ronald S. Bultje
Hi, 2011/12/28 Måns Rullgård : > Derek Buitenhuis writes: > >> From: Stefano Sabatini >> >> Prevent error condition in case sample_rate is unset or set to a negative >> value. In particular, fix divide-by-zero error occurring in ffmpeg due to >> sample_rate set to 0 in output_packet(), in code:

Re: [libav-devel] FATE : test for zmbv

2011-12-28 Thread Oana Andreea Stratulat
I didn`t test them, i have worked on another test for kgv1. The samples were in fate but the tests were not made . On Dec 28, 2011, at 7:28 PM, Ronald S. Bultje wrote: > Hi, > > On Wed, Dec 28, 2011 at 1:04 AM, Oana Andreea Stratulat > wrote: >> In lcov the coverage was 0%thats why i made those

[libav-devel] [PATCH] applehttpproto: Apply the same reload interval changes as for the demuxer

2011-12-28 Thread Martin Storsjö
From: Andrey Utkin Signed-off-by: Martin Storsjö --- Slightly touched up the patch to minimize the diff, and moved the reload_interval *= 100 to before the jump label where it should be. libavformat/applehttpproto.c | 14 -- 1 files changed, 12 insertions(+), 2 deletions(-)

Re: [libav-devel] [PATCH] mpegvideo_enc: K&R cosmetics

2011-12-28 Thread Ronald S. Bultje
Hi, On Tue, Dec 27, 2011 at 1:55 PM, Aneesh Dogra wrote: > -static int estimate_best_b_count(MpegEncContext *s){ > -AVCodec *codec= avcodec_find_encoder(s->avctx->codec_id); > +static int estimate_best_b_count(MpegEncContext *s) [..] This was already included in an earlier patch, please reba

Re: [libav-devel] [PATCH 2/2] mpegaudiodec: add SSE-optimized imdct36()

2011-12-28 Thread Ronald S. Bultje
Hi, dead thread ping. On Mon, Sep 5, 2011 at 6:33 PM, Loren Merritt wrote: > On Sat, Sep 3, 2011, Vitor Sessak wrote: >> On Sat, Sep 3, 2011, Ronald S. Bultje wrote: >>> On Thu, Sep 1, 2011, Vitor Sessak wrote: >>> +%macro LOADA64 2 +   movlps   %1, [%2] +   movhps   %1, [%2 +

[libav-devel] [PATCH] v410dec: Check for sufficient input data. Fixes crash

2011-12-28 Thread Derek Buitenhuis
From: Michael Niedermayer Signed-off-by: Michael Niedermayer --- libavcodec/v410dec.c |5 + 1 files changed, 5 insertions(+), 0 deletions(-) diff --git a/libavcodec/v410dec.c b/libavcodec/v410dec.c index 71cbe0b..6147aad 100644 --- a/libavcodec/v410dec.c +++ b/libavcodec/v410dec.c @@ -

Re: [libav-devel] [PATCH] v410dec: Check for sufficient input data. Fixes crash

2011-12-28 Thread Derek Buitenhuis
On 28/12/2011 2:56 PM, Derek Buitenhuis wrote: From: Michael Niedermayer Signed-off-by: Michael Niedermayer Err, woops. Definitely forgot to amend spaces. - Derek ___ libav-devel mailing list libav-devel@libav.org https://lists.libav.org/mailman/lis

[libav-devel] [PATCH] v410dec: Check for sufficient input data. Fixes crash

2011-12-28 Thread Derek Buitenhuis
From: Michael Niedermayer Signed-off-by: Michael Niedermayer --- libavcodec/v410dec.c |5 + 1 files changed, 5 insertions(+), 0 deletions(-) diff --git a/libavcodec/v410dec.c b/libavcodec/v410dec.c index 71cbe0b..54c9595 100644 --- a/libavcodec/v410dec.c +++ b/libavcodec/v410dec.c @@ -

Re: [libav-devel] [PATCH] v410dec: Check for sufficient input data. Fixes crash

2011-12-28 Thread Kostya Shishkov
On Wed, Dec 28, 2011 at 03:04:44PM -0500, Derek Buitenhuis wrote: > From: Michael Niedermayer > > Signed-off-by: Michael Niedermayer > --- > libavcodec/v410dec.c |5 + > 1 files changed, 5 insertions(+), 0 deletions(-) > > diff --git a/libavcodec/v410dec.c b/libavcodec/v410dec.c > inde

[libav-devel] [PATCH] fix grammar issue in Christmas release announcement

2011-12-28 Thread Justin Ruggles
--- src/index |2 +- 1 files changed, 1 insertions(+), 1 deletions(-) diff --git a/src/index b/src/index index 06b0b5e..4886170 100644 --- a/src/index +++ b/src/index @@ -36,7 +36,7 @@ with the latest developments by subscribing to both the December 25 2011 -Just in time for Christmas, w

Re: [libav-devel] [PATCH] fix grammar issue in Christmas release announcement

2011-12-28 Thread Ronald S. Bultje
Hi, On Wed, Dec 28, 2011 at 12:24 PM, Justin Ruggles wrote: > --- >  src/index |    2 +- >  1 files changed, 1 insertions(+), 1 deletions(-) > > diff --git a/src/index b/src/index > index 06b0b5e..4886170 100644 > --- a/src/index > +++ b/src/index > @@ -36,7 +36,7 @@ with the latest developments

[libav-devel] [PATCH] wavpack: determine sample_fmt before requesting a buffer

2011-12-28 Thread Hendrik Leppkes
Hi, attached patch fixes a memory corruption when decoding WavPack > 16-bit. It is caused by the decoder being initialized as S16, and requesting a buffer for that size, however when decoding the very first frame, the type can actually change after the buffer was requested, causing it to write bey

Re: [libav-devel] [PATCH] wavpack: determine sample_fmt before requesting a buffer

2011-12-28 Thread Hendrik Leppkes
On Wed, Dec 28, 2011 at 9:42 PM, Hendrik Leppkes wrote: > attached patch Should actually remember to attach the patch. 0001-wavpack-determine-sample_fmt-before-requesting-a-buf.patch Description: Binary data ___ libav-devel mailing list libav-devel@li

Re: [libav-devel] random failures of fate-vsynth1-dnxhd_720p_rd

2011-12-28 Thread Michael Kostylev
On Wed Dec 28 11:10:55 2011 Sean McGovern wrote: >> Sometimes the vsynth1-dnxhd_720p_rd test fails on alpha, pa-risc and sparc >> FATE machines. It explicitly uses '-threads 4' when encoding, and for some >> reason vsynth1/dnxhd-720p-rd.dnxhd may be spoiled (the first frame contains >> a small nu

Re: [libav-devel] [PATCH] wavpack: determine sample_fmt before requesting a buffer

2011-12-28 Thread Derek Buitenhuis
On 28/12/2011 3:42 PM, Hendrik Leppkes wrote: Hi, attached patch fixes a memory corruption when decoding WavPack > 16-bit. It is caused by the decoder being initialized as S16, and requesting a buffer for that size, however when decoding the very first frame, the type can actually change after t

Re: [libav-devel] random failures of fate-vsynth1-dnxhd_720p_rd

2011-12-28 Thread Ronald S. Bultje
Hi Michael, On Wed, Dec 28, 2011 at 4:07 AM, Michael Kostylev wrote: > Sometimes the vsynth1-dnxhd_720p_rd test fails on alpha, pa-risc and sparc > FATE machines. It explicitly uses '-threads 4' when encoding, and for some > reason vsynth1/dnxhd-720p-rd.dnxhd may be spoiled (the first frame conta

Re: [libav-devel] LibAV-FATE on Solaris x86

2011-12-28 Thread Sean McGovern
On Wednesday, December 28, 2011, Måns Rullgård wrote: > Sean McGovern writes: > >> On Tue, Dec 27, 2011 at 7:19 PM, Sean McGovern wrote: >>> 2011/10/21 Måns Rullgård : "Sean McGovern" writes: >> >> [snip..] >> the same thing. Since they probably won't, installing a simple wrappe

Re: [libav-devel] random failures of fate-vsynth1-dnxhd_720p_rd

2011-12-28 Thread Sean McGovern
On 12/28/11, Michael Kostylev wrote: > > On Wed Dec 28 11:10:55 2011 > Sean McGovern wrote: > >>> Sometimes the vsynth1-dnxhd_720p_rd test fails on alpha, pa-risc and >>> sparc >>> FATE machines. It explicitly uses '-threads 4' when encoding, and for >>> some >>> reason vsynth1/dnxhd-720p-rd.dnxhd

Re: [libav-devel] [PATCH] fate: Add tests for xxan decoder

2011-12-28 Thread Ronald S. Bultje
Hi, On Tue, Dec 27, 2011 at 11:21 PM, Aneesh Dogra wrote: > --- >  tests/fate2.mak           |    9 + >  tests/ref/fate/xxan_wc3   | 1024 > + >  tests/ref/fate/xxan_wc4   |  222 ++ >  tests/ref/fate/xxan_wc4_2 |   68 +++ >  4 files changed, 132

[libav-devel] [PATCH] snowenc: Fix encoding of GRAY8 videos

2011-12-28 Thread Derek Buitenhuis
snowenc previously had a hardcoded value of 3 for the amount of planes it was to process. Tested snow and snowll in fate. No regressions present. --- libavcodec/snowenc.c | 10 +++--- 1 files changed, 7 insertions(+), 3 deletions(-) diff --git a/libavcodec/snowenc.c b/libavcodec/snowenc.c

Re: [libav-devel] [PATCH 05/11] rawdec: Fix linesize for some rawvideo samples with odd width.

2011-12-28 Thread Måns Rullgård
Derek Buitenhuis writes: > On 28/12/2011 12:02 PM, Kostya Shishkov wrote: looks stupid >>> > >>> > Disgusting even. > > I do agree, but may I remind you we recently pushed this: > > http://git.libav.org/?p=libav.git;a=commitdiff;h=b4dc68803bd7fa73c2e37f55ae73b5fdb293c68d Two wrongs do not

Re: [libav-devel] [PATCH 01/11] rawdec: Support more QT 1bpp rawvideo files.

2011-12-28 Thread Måns Rullgård
Derek Buitenhuis writes: > On 28/12/2011 11:43 AM, Måns Rullgård wrote: >> Does this person have a real name? I don't like taking patches >> anonymously. > > Not as far as I know, no. He's got some stuff in Libav too already. Two wrongs still do not make a right. > I don't see it as a huge iss

Re: [libav-devel] [PATCH 10/11] rawdec: add check on sample_rate

2011-12-28 Thread Måns Rullgård
"Ronald S. Bultje" writes: > Hi, > > 2011/12/28 Måns Rullgård : >> Derek Buitenhuis writes: >> >>> From: Stefano Sabatini >>> >>> Prevent error condition in case sample_rate is unset or set to a negative >>> value. In particular, fix divide-by-zero error occurring in ffmpeg due to >>> sample_ra

Re: [libav-devel] [PATCH] v410dec: Check for sufficient input data. Fixes crash

2011-12-28 Thread Måns Rullgård
Derek Buitenhuis writes: > From: Michael Niedermayer > > Signed-off-by: Michael Niedermayer > --- > libavcodec/v410dec.c |5 + > 1 files changed, 5 insertions(+), 0 deletions(-) > > diff --git a/libavcodec/v410dec.c b/libavcodec/v410dec.c > index 71cbe0b..54c9595 100644 > --- a/libavco

Re: [libav-devel] random failures of fate-vsynth1-dnxhd_720p_rd

2011-12-28 Thread Måns Rullgård
Sean McGovern writes: > On 12/28/11, Michael Kostylev wrote: >> >> On Wed Dec 28 11:10:55 2011 >> Sean McGovern wrote: >> Sometimes the vsynth1-dnxhd_720p_rd test fails on alpha, pa-risc and sparc FATE machines. It explicitly uses '-threads 4' when encoding, and for some reason vs

Re: [libav-devel] LibAV-FATE on Solaris x86

2011-12-28 Thread Måns Rullgård
Sean McGovern writes: > On Wednesday, December 28, 2011, Måns Rullgård wrote: >> Sean McGovern writes: >> >>> On Tue, Dec 27, 2011 at 7:19 PM, Sean McGovern > wrote: 2011/10/21 Måns Rullgård : > "Sean McGovern" writes: > >>> >>> [snip..] >>> > the same thing. Since they prob

Re: [libav-devel] [PATCH] fate: Add tests for xxan decoder

2011-12-28 Thread Måns Rullgård
"Ronald S. Bultje" writes: > Hi, > > On Tue, Dec 27, 2011 at 11:21 PM, Aneesh Dogra wrote: >> --- >>  tests/fate2.mak           |    9 + >>  tests/ref/fate/xxan_wc3   | 1024 >> + >>  tests/ref/fate/xxan_wc4   |  222 ++ >>  tests/ref/fate/xxan_

Re: [libav-devel] [PATCH] snowenc: Fix encoding of GRAY8 videos

2011-12-28 Thread Måns Rullgård
Derek Buitenhuis writes: > snowenc previously had a hardcoded value of 3 for > the amount of planes it was to process. > > Tested snow and snowll in fate. No regressions present. FATE does not test snow with gray8 iirc. Is that even supposed to work? -- Måns Rullgård m...@mansr.com __

Re: [libav-devel] [PATCH] snowenc: Fix encoding of GRAY8 videos

2011-12-28 Thread Derek Buitenhuis
On 28/12/2011 6:19 PM, Måns Rullgård wrote: > snowenc previously had a hardcoded value of 3 for > the amount of planes it was to process. > > Tested snow and snowll in fate. No regressions present. FATE does not test snow with gray8 iirc. Is that even supposed to work? Supposedly, as suppo

Re: [libav-devel] [PATCH] snowenc: Fix encoding of GRAY8 videos

2011-12-28 Thread Derek Buitenhuis
On 28/12/2011 6:21 PM, Derek Buitenhuis wrote: I used FATE to test that i didn't break color decoding. Given that gray decoding wasn't even working until now, I have nothing to test against for that... - Derek s/decoding/encoding/ ___ libav-devel mai

Re: [libav-devel] [PATCH 10/11] rawdec: add check on sample_rate

2011-12-28 Thread Justin Ruggles
On 12/28/2011 06:07 PM, Måns Rullgård wrote: > "Ronald S. Bultje" writes: > >> Hi, >> >> 2011/12/28 Måns Rullgård : >>> Derek Buitenhuis writes: >>> From: Stefano Sabatini Prevent error condition in case sample_rate is unset or set to a negative value. In particular, fix di

Re: [libav-devel] [PATCH] fate: Add tests for xxan decoder

2011-12-28 Thread Ronald S. Bultje
Hi, 2011/12/28 Måns Rullgård : > "Ronald S. Bultje" writes: > >> Hi, >> >> On Tue, Dec 27, 2011 at 11:21 PM, Aneesh Dogra wrote: >>> --- >>>  tests/fate2.mak           |    9 + >>>  tests/ref/fate/xxan_wc3   | 1024 >>> + >>>  tests/ref/fate/xxan_wc4  

[libav-devel] FATE : test for dfa

2011-12-28 Thread Oana Andreea Stratulat
Link to sample files : http://google-melange.appspot.com/gci/work/download/google/gci2011/7140342?id=3001 0001-FATE-test-for-DFA.patch Description: Binary data ___ libav-devel mailing list libav-devel@libav.org https://lists.libav.org/mailman/listinfo/l

Re: [libav-devel] LibAV-FATE on Solaris x86

2011-12-28 Thread Sean McGovern
On Wednesday, December 28, 2011, Måns Rullgård wrote: > Sean McGovern writes: > >> On Wednesday, December 28, 2011, Måns Rullgård wrote: >>> Sean McGovern writes: >>> On Tue, Dec 27, 2011 at 7:19 PM, Sean McGovern >> wrote: > 2011/10/21 Måns Rullgård : >> "Sean McGovern" writes:

[libav-devel] [PATCH] FATE: Add tests for AAC

2011-12-28 Thread Gaurav Narula
--- tests/fate/aac.mak | 16 1 files changed, 16 insertions(+), 0 deletions(-) diff --git a/tests/fate/aac.mak b/tests/fate/aac.mak index f17c914..3cf3ec1 100644 --- a/tests/fate/aac.mak +++ b/tests/fate/aac.mak @@ -14,14 +14,30 @@ FATE_AAC += fate-aac-al07_96 fate-aac-al07_96

Re: [libav-devel] FATE : test for dfa

2011-12-28 Thread Justin Ruggles
On 12/28/2011 08:50 PM, Oana Andreea Stratulat wrote: > Link to sample files : > http://google-melange.appspot.com/gci/work/download/google/gci2011/7140342?id=3001 These tests should all be put in a single mak file. Also, please list the justification for the inclusion of each test. We don't ne

Re: [libav-devel] [PATCH 0/3] ADX decoder fixes

2011-12-28 Thread Justin Ruggles
On 12/19/2011 09:11 AM, Justin Ruggles wrote: > The patches from Michael have been slightly revised, but they remain basically > the same. > > Justin Ruggles (1): > adxdec: Validate channel count to fix a division by zero. > > Michael Niedermayer (2): > adxdec: Do not require extradata. >

Re: [libav-devel] random failures of fate-vsynth1-dnxhd_720p_rd

2011-12-28 Thread Ronald S. Bultje
Hi, On Wed, Dec 28, 2011 at 8:34 PM, Michael Kostylev wrote: > > On Wed Dec 28 23:13:50 2011 > Måns Rullgård wrote: > >> Sometimes the vsynth1-dnxhd_720p_rd test fails on alpha, pa-risc >> and sparc FATE machines. It explicitly uses '-threads 4' when >> encoding, and for some reason v

Re: [libav-devel] random failures of fate-vsynth1-dnxhd_720p_rd

2011-12-28 Thread Michael Kostylev
On Wed Dec 28 23:13:50 2011 Måns Rullgård wrote: > Sometimes the vsynth1-dnxhd_720p_rd test fails on alpha, pa-risc > and sparc FATE machines. It explicitly uses '-threads 4' when > encoding, and for some reason vsynth1/dnxhd-720p-rd.dnxhd may be > spoiled (the first frame contain

Re: [libav-devel] [RFC PATCH] threads: default to automatic thread count detection

2011-12-28 Thread Ronald S. Bultje
Hi, On Mon, Dec 26, 2011 at 5:01 AM, Janne Grunau wrote: > Hi, > > On 2011-12-25 17:02:01 +0100, Luca Barbato wrote: >> On 25/12/11 16:26, Janne Grunau wrote: >> >+- Automatic thread count based on detction number of (available) CPU cores >> typo and some tabs... >> >> > -        if (options) >>

Re: [libav-devel] [PATCH] Nuke libpostproc

2011-12-28 Thread Ronald S. Bultje
Hi On Fri, Dec 16, 2011 at 10:50 AM, Derek Buitenhuis wrote: > On 15/12/2011 9:21 AM, Anton Khirnov wrote: >> >> I see no point in pretending it's a part of Libav, when it's clearly >> not. Nobody has maintained it since forever and apparrently nobody wants >> to. >> Let's get rid of it, whoever

Re: [libav-devel] random failures of fate-vsynth1-dnxhd_720p_rd

2011-12-28 Thread Michael Kostylev
On Wed Dec 28 20:36:28 2011 Ronald S. Bultje wrote: >>> Sometimes the vsynth1-dnxhd_720p_rd test fails on alpha, pa-risc >>> and sparc FATE machines. It explicitly uses '-threads 4' when >>> encoding, and for some reason vsynth1/dnxhd-720p-rd.dnxhd may be >>> spoiled (the first fr

Re: [libav-devel] LibAV-FATE on Solaris x86

2011-12-28 Thread Michael Kostylev
On Wed Dec 28 16:41:10 2011 Måns Rullgård wrote: the same thing.  Since they probably won't, installing a simple wrapper as /bin/sh will work as well.  Something like this should do it: #include #include int main(int argc, char **argv) {    if (geten

Re: [libav-devel] LibAV-FATE on Solaris x86

2011-12-28 Thread Michael Kostylev
On Wed Dec 28 20:55:11 2011 Sean McGovern wrote: > The check_exec_crash function is supposed to run the test in a subshell so > that the SIGTERM that it raises does not cause 'configure' itself to > terminate, only the subshell it was executing inside. I'm not certain how > to test /usr/xpg4/bin/

Re: [libav-devel] [PATCH] wavpack: determine sample_fmt before requesting a buffer

2011-12-28 Thread Kostya Shishkov
On Wed, Dec 28, 2011 at 09:42:58PM +0100, Hendrik Leppkes wrote: > On Wed, Dec 28, 2011 at 9:42 PM, Hendrik Leppkes wrote: > > attached patch > > Should actually remember to attach the patch. looks OK ___ libav-devel mailing list libav-devel@libav.org