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 majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html