Brandon Williams <bmw...@google.com> writes: > Refactor find_non_local_tags and get_ref_map to only take the > information they need instead of the entire transport struct. Besides > improving code clarity, this also improves their flexibility, allowing > for a different set of refs to be used instead of relying on the ones > stored in the transport struct.
Makes sense. One can argue that the original has less risk of set of refs used for calls to this function vs calls for others go out of sync, but the objective of this series is to allow them to be different ;-) I am not sure if I understand/agree with the split of parameters done to get_ref_map() at this step, but hopefully its benefit would become obvious once I read later steps.
