Re: [PATCH v2 06/15] rev-list: make --count work with --objects
- From
Jeff King <peff@peff.net>
- Date
- Feb 18, 2020, 19:55 UTC
- Message-ID
- <20200218195531.GA21774@coredump.intra.peff.net>
- In-Reply-To
- <xmqqsgj7kdog.fsf@gitster-ct.c.googlers.com>
On Tue, Feb 18, 2020 at 09:28:31AM -0800, Junio C Hamano wrote:
Show 20 quoted lines
> >> I think the question reader had was "why *right*?" > > > > Ah. The answer is: because it's not SYMMETRIC_LEFT. ;) > > > > The counting code accumulates there by default when there are no side > > markings, so that's what it will report when there's no left_right or > > cherry_mark (which we know will be the case here, since we die() > > otherwise). > > Yup. > > /* > * The object count is always accumulated in the .count_right > * field for traversal that is not a left-right traversal, > * and cmd_rev_list() made sure that a .count request that > * wants to count non-commit objects, which is handled by > * the show_object() callback, does not ask for .left_right. > */ > > Overkill? I dunno.
No, I think it looks good. Want to squash that in (and I'll do likewise locally)?
-Peff