Jonathan Tan <jonathantanmy@xxxxxxxxxx> writes: > To answer Junio's questions in [1], I think it's best to include the > full patch set that I'm developing, so here it is. The original patch is > now patch 3 of this set. Yeah, taking it out of context did make its purpose fuzzy. Without the other patches in the series that set the overall direction of reducing ls-refs, it was unclear why some transports are marked to specifically want to automatically call transport_get_remote_refs(). With these surrounding changes, "we call it as needed, because an earlier step may not have called it" starts to sound quite sensible.