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

Diego Biurrun diego
Mon Oct 19 23:35:12 CEST 2009


On Mon, Oct 19, 2009 at 09:16:52PM +0200, Stefano Sabatini wrote:
> 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.

Both look good.

Diego



More information about the ffmpeg-devel mailing list