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

Stefano Sabatini stefano.sabatini-lala
Fri Sep 18 21:30:47 CEST 2009


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...

> > 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

Regards.
-- 
FFmpeg = Freak & Fundamentalist MultiPurpose Elastic Guru
-------------- next part --------------
A non-text attachment was scrubbed...
Name: lavfi-regression-test.patch
Type: text/x-diff
Size: 3401 bytes
Desc: not available
URL: <http://lists.mplayerhq.hu/pipermail/ffmpeg-devel/attachments/20090918/7e0f4328/attachment.patch>



More information about the ffmpeg-devel mailing list