Re: First/oldest entry in reflog dropped
- From
Jeff King <peff@peff.net>
- Date
- Nov 25, 2010, 16:15 UTC
- Message-ID
- <20101125161530.GB24426@sigill.intra.peff.net>
- In-Reply-To
- <7vzksztv6f.fsf@alter.siamese.dyndns.org>
On Tue, Nov 23, 2010 at 04:35:52PM -0800, Junio C Hamano wrote:
Show 14 quoted lines
> > So let's do this, which is the equivalent behavior to your gotos, but > > this structure makes more sense to me as a reader (and it doesn't > > involve goto :) ). > > I have a feeling that we probably should have structured the code a bit > more modularly, placing some info in "rev" that tells us how to "pop" a > commit from the rev->list (typically "not the ones that we have shown"), > what other commits to push back into the queue (typically, "all the > parents that are not interesting"), and what side effects we should cause > when we do so (typically, "mark uninteresting parents"), etc., instead of > the current "if we are walking reflog, here is the special codepath we > take", so that the walking is more generalized when we did the reflog > walking (in fact, if we did this properly we probably wouldn't be calling > it "bolted on"). But for now let's refrain from doing such a rewrite.
FWIW, I agree with all of that. I don't think it's worth fixing now, but perhaps in libgit2. :)
-Peff