From: Junio C Hamano Date: Sun, 09 Sep 2012 10:20:18 GMT Subject: Re: [PATCH v3 00/14] Clean up how fetch_pack() handles the heads list Message-ID: <7v7gs3sdjx.fsf@alter.siamese.dyndns.org> In-Reply-To: <1347171589-13327-1-git-send-email-mhagger@alum.mit.edu> Michael Haggerty writes: > This patch series depends on the "Add some string_list-related > functions" series that I just submitted. Makes sense. The only worry (without reading the series first) I have is that the use of string list may make the responsiblity of sorting the list fuzzier. I am guessing that we never sorted the refs we asked to fetch (so that FETCH_HEAD comes out in an expected order), so use of unsorted string list would be perfectly fine. Queued and pushed out in 'pu' without reading, but will take a look sometime tomorrow (eh, today, that is).