[Ffmpeg-cvslog] r7522 - trunk/tests/regression.sh

Diego Biurrun diego
Mon Jan 15 08:56:32 CET 2007


On Mon, Jan 15, 2007 at 02:04:31AM +0000, M?ns Rullg?rd wrote:
> diego <subversion at mplayerhq.hu> writes:
> 
> > Log:
> > Remove unnecessary variable indirection.
> >
> > --- trunk/tests/regression.sh	(original)
> > +++ trunk/tests/regression.sh	Mon Jan 15 01:23:58 2007
> > @@ -116,33 +116,31 @@
> >
> >  do_ffmpeg()
> >  {
> > -    f="$1"
> >      shift
>        ^^^^^
> >      echo $ffmpeg $FFMPEG_OPTS $*
> >      $ffmpeg $FFMPEG_OPTS -benchmark $* > $bench 2> /tmp/ffmpeg$$
> >      egrep -v "^(Stream|Press|Input|Output|frame|  Stream|  Duration|video:)" /tmp/ffmpeg$$ || true
> >      rm -f /tmp/ffmpeg$$
> > -    do_md5sum $f >> $logfile
> > +    do_md5sum $1 >> $logfile
>                  ^^
> 
> You broke it!

Ouch, a typical artifact of tiredness.  I have reworked those functions
some more so that this makes sense, but while extracting a change I made
this stupid mistake.  I reverted the change.

Diego




More information about the ffmpeg-cvslog mailing list