Jeff King <peff@xxxxxxxx> writes: > On Sun, Sep 09, 2012 at 03:20:18AM -0700, Junio C Hamano wrote: > >> Michael Haggerty <mhagger@xxxxxxxxxxxx> 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. > > I haven't read the series yet, but both the list of heads from the user > and the list of heads from the remote should have been sorted by 4435968 > and 9e8e704f, respectively. OK. As long as the sort order matches the order string-list internally uses for its bisection search, it won't be a problem, then. Thanks. -- To unsubscribe from this list: send the line "unsubscribe git" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html