On 3/30/2017 4:44 PM, Junio C Hamano wrote:
Jeff King <peff@xxxxxxxx> writes:Still, I'm not sure the extra layer of cache is all that valuable. It should be a single hash lookup in the config cache (in an operation that otherwise reads the entire index).OK, let's drop that part, then.
Yes, let's omit the caching there. That way perf tests can run it multiple times with it on or off as they see fit. And in the normal case it will only be executed once anyway.