On Thu, 29 Nov 2007, Linus Torvalds wrote: > On Fri, 30 Nov 2007, Jakub Narebski wrote: > > > > Isn't there a better way to do this sorting? What is needed here is > > (stable) _bucket_ sort / _pigeonhole_ sort (or counting sort), which > > is O(n); quicksort is perhaps simpler to use, but I'm not sure if > > faster in this situation. That particular sort takes under a second here with the Linux repo. Pretty insignificant compared to the time required to repack. > Actually, I doubt you need to do any sorting at all: what would be easiest > would be to simply change "traverse_commit_list()" to use different lists > for different object types, and just output them in type order (semi-sane > order choice: commits first, then tags, then trees, and finally blobs). Yes! That's what I thought initially, but since list-objects.c is completely unknown territory to me, I sorted them in pack-object.c instead, out of pure laziness. Nicolas - 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
