Re: [PATCH v3 00/14] Clean up how fetch_pack() handles the heads list

Michael Haggerty [off-list ref] writes:

This patch series depends on the "Add some string_list-related
functions" series that I just submitted.

This series is a significant rewrite of v2 based on the realization
that the <nr_heads,heads> pair that is passed around in these
functions is better expressed as a string_list.  I hope you will find
that this version is shorter and clearer than its predecessors, even
though its basic logic is mostly the same.

Sorry for the false starts in v1 and v2 and the extra reviewing work
that this will cost.

Thanks for a pleasant read, nicely broken down into digestible
pieces.