Junio C Hamano <gitster@xxxxxxxxx> writes: > Other than that, I like what this patch attempts to do. A nicely > identified low-hanging fruit ;-). Having said that, this will have a bad interaction with another topic in flight: <20171121213341.13939-1-rafa.almas@xxxxxxxxx> Perhaps this should wait until the other topic lands and stabilizes. We'd need to rethink if the approach taken by this patch, i.e. to still pass the info to load() but holding onto it until the time lazy_load() actually uses it, is a sensible way forward, or we would want to change the calling convention to help making it easier to implement the lazy loading. Thanks.