On Wed, Mar 25, 2015 at 10:27:49AM -0700, Junio C Hamano wrote:

> > diff --git a/t/t3060-ls-files-with-tree.sh b/t/t3060-ls-files-with-tree.sh
> > index 61c1f53..36b10f7 100755
> > --- a/t/t3060-ls-files-with-tree.sh
> > +++ b/t/t3060-ls-files-with-tree.sh
> > @@ -25,15 +25,10 @@ test_expect_success setup '
> >             do
> >                     num=00$n$m &&
> >                     >sub/file-$num &&
> > -                   echo file-$num >>expected || {
> > -                           bad=t
> > -                           break
> > -                   }
> > -           done && test -z "$bad" || {
> > -                   bad=t
> > -                   break
> > -           }
> > -   done && test -z "$bad" &&
> > +                   echo file-$num >>expected ||
> > +                   return 1
> > +           done
> > +   done &&
> >     git add . &&
> >     git commit -m "add a bunch of files" &&
> 
> The empty initialization for $bad can also go for this one, right?

Yeah, it can.

-Peff
--
To unsubscribe from this list: send the line "unsubscribe git" in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html

Reply via email to