Re: [PATCH 06/18] t5613: clarify "too deep" recursion tests
- From
Jacob Keller <jacob.keller@gmail.com>
- Date
- Oct 4, 2016, 21:43 UTC
- Message-ID
- <CA+P7+xoDz2sOPrDrJhAhrqDQsRR8NVU-8kh6+G=8FJeXEJ1dtg@mail.gmail.com>
- In-Reply-To
- <20161004205510.6bhisw7ixbgcvvwn@sigill.intra.peff.net>
On Tue, Oct 4, 2016 at 1:55 PM, Jeff King <peff@peff.net> wrote:
Show 22 quoted lines
> On Tue, Oct 04, 2016 at 01:52:19PM -0700, Jacob Keller wrote: > >> >> >> > +# Note: These tests depend on the hard-coded value of 5 as "too deep". We start >> >> >> > +# the depth at 0 and count links, not repositories, so in a chain like: >> >> >> > +# >> >> >> > +# A -> B -> C -> D -> E -> F -> G -> H >> >> >> > +# 0 1 2 3 4 5 6 >> >> >> > +# >> [...] >> > No, we count links, not repositories. So the "A->B" link is "0", "B->C" >> > is "1", and so on. >> >> If you need to re-roll for some other reason I would add some spaces >> around the numbers so they line up better with the links so that this >> becomes more clear. > > Hmm. Now I am puzzled, because I _did_ line up them specifically to make > this clear. I put the numbers under the ">" of the arrow. Did I screw up > the spacing somehow so that isn't how they look to you? Or are you just > saying you would prefer them under the "-" of the arrow? > > -Peff
I bet they line up in a monospace font and I just happened to be viewing this from GMail which isn't showing it in monospace and so it doesn't line up. Ignore me and carry on
Thanks, Jake