[FFmpeg-devel] [PATCH] libavfilter-soc: regression test for libavfilter

Michael Niedermayer michaelni
Sun Sep 20 12:02:25 CEST 2009


On Fri, Sep 18, 2009 at 09:30:47PM +0200, Stefano Sabatini wrote:
> On date Thursday 2009-08-13 12:41:52 +0200, Michael Niedermayer encoded:
> > On Sat, Jul 25, 2009 at 09:31:34PM +0200, Stefano Sabatini wrote:
> > > On date Tuesday 2009-07-21 03:45:56 +0200, Michael Niedermayer encoded:
> > > > On Sun, Jul 12, 2009 at 01:39:43PM +0200, Stefano Sabatini wrote:
> > > > > On date Sunday 2009-07-05 20:47:57 +0200, Stefano Sabatini encoded:
> > > > > > On date Sunday 2009-06-21 22:58:26 +0200, Stefano Sabatini encoded:
> > > > > > > Hi, as in subject.
> > > > > > > 
> > > > > > > Especially ideas for more meaningful tests are very welcome.
> > > > > 
> > > > > Patch updated with some factorization and the vflip test removed.
> > > > > 
> > > > > I also changed the subject as I consider the reg-test already useful
> > > > > when testing it, so I'm proposing it for committing in the soc repo.
> > > > 
> > > > Is there something preventing this to be commited to main? (with the test
> > > > disabled of course until the needed parts are in too)
> > > 
> > > Something like in attached? 
> > 
> > there is nothing attached
> 
> Look again...

there is still nothing attached to your previous mail


> 
> > > Since it is completely unused now I see no
> > > convenience in that, but I don't object in applying it if you like it.
> > 
> > i would like to see lavfi merged into ffmpeg and for that individual
> > parts must be commited before they can be used, the whole as one patch of
> > everything would instantly be rejected with a "split it" comment.
> 
> I know.
> 
> With this patch applied to add a test is as easy as:
> 
> * write the do_lavfi test specification in tests/codec-regression.sh
>   and the name of the test in the Makefile under LAVFI_TEST
> * create the file tests/lavfi.regression.ref and put there the result
>   of the test
> * and finally decomment the lavfitest target in the Makefile

patch looks ok,if noone else has an comment

[...]
-- 
Michael     GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB

Everything should be made as simple as possible, but not simpler.
-- Albert Einstein
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 189 bytes
Desc: Digital signature
URL: <http://lists.mplayerhq.hu/pipermail/ffmpeg-devel/attachments/20090920/13a5934c/attachment.pgp>



More information about the ffmpeg-devel mailing list