[FFmpeg-devel] [PATCH] Gives symbolic names to codec-regression.sh arguments

Stefano Sabatini stefano.sabatini-lala
Mon Oct 19 21:16:52 CEST 2009


On date Sunday 2009-10-18 14:06:15 +0200, Diego Biurrun encoded:
> On Sun, Oct 18, 2009 at 12:20:31AM +0200, Stefano Sabatini wrote:
> > On date Saturday 2009-10-17 22:55:21 +0100, M?ns Rullg?rd encoded:
> > > Stefano Sabatini <stefano.sabatini-lala at poste.it> writes:
> > > > this should improve the readability of the file, also it's required if
> > > > we want to export that variables in a regression-funcs.sh file, as
> > > > we cant use the $N variables in that case when loading it.
> > > 
> > > Your patch does more than that.
> > 
> > It should be minimal now.
> 
> Looks good.

After more thought I decided that I prefer $test_ref over $test_class,
as the latter is too generic.

In attachment another cosmetic patch which should ease the following
split.

Regards.
-- 
FFmpeg = Fundamentalist & Freak Mind-dumbing Practical Energized Gadget
-------------- next part --------------
A non-text attachment was scrubbed...
Name: name-codec-regression-args.patch
Type: text/x-diff
Size: 1147 bytes
Desc: not available
URL: <http://lists.mplayerhq.hu/pipermail/ffmpeg-devel/attachments/20091019/27089aff/attachment.patch>
-------------- next part --------------
A non-text attachment was scrubbed...
Name: move-up-test-def.patch
Type: text/x-diff
Size: 567 bytes
Desc: not available
URL: <http://lists.mplayerhq.hu/pipermail/ffmpeg-devel/attachments/20091019/27089aff/attachment-0001.patch>



More information about the ffmpeg-devel mailing list