Re: [PATCH Outreachy] mru: use double-linked list from list.h

From: Junio C Hamano <gitster@pobox.com>
Date: Sep 28, 2017, 21:56 UTC
Message-ID: <xmqq4lrm1o8j.fsf@gitster.mtv.corp.google.com>
In-Reply-To: <20170928204705.7ixxspiflmhsdh7d@sigill.intra.peff.net>

Jeff King <peff@peff.net> writes:

> But I also think this patch may be a stepping stone to dropping "struct
> mru" entirely, and just pushing a "struct list_head mru" into the
> packed_git object itself (or of course any object you like). At which
> point we'd just directly use the list iterators anyway.

The endgame state implied by your statement would mean that we won't have mru_mark() and instead have these open-coded in terms of the two calls into the list API. There only are two callers of it in the current system, so it is not the end of the world, so I guess I can agree that this is a good preparation step toward the longer term goal, which says "mru API is over-engineered and what it offers over the plain vanilla list API is almost never used except for a few callsite; let's remove it and use the bare list API instead".

> I'd make the same claims here as above (both that I agree your proposed
> interface looks nicer, but also that I think we eventually do want to
> expose that this is tightly coupled with list.h).

I agree. I just do not yet know if I fully embrace the idea that we just should use bare list API, getting rid of the mru API.