From: Junio C Hamano <gitster@pobox.com>
Date: Thu, 13 Feb 2020 18:34:45 GMT
Subject: Re: [PATCH 05/13] rev-list: factor out bitmap-optimized routines
Message-ID: <xmqq4kvus5dm.fsf@gitster-ct.c.googlers.com>
In-Reply-To: <20200213021820.GE1126038@coredump.intra.peff.net>

Jeff King <peff@peff.net> writes:

> There are a few operations in rev-list that are optimized for bitmaps.
> Rather than having the code inline in cmd_rev_list(), let's move them
> into helpers. This not only makes the flow of the main function simpler,
> but it lets us replace the complex "can we do the optimization?"
> conditionals with a series of early returns from the functions. That
> also makes it easy to add comments explaining those conditions.

... and these new comments makes the resulting code much easier to
follow.  Nicely done.