On 13/12/13 00:58, Duy Nguyen wrote: > On Fri, Dec 13, 2013 at 6:15 AM, Ramsay Jones > <ramsay@xxxxxxxxxxxxxxxxxxx> wrote: >> BTW, I have not been following these patches, but I noticed that the >> 'remove_nonexistent_ours_in_pack()' function has no callers. (There are >> two commented out callers - but they seem to have *always* been commented >> out ;-) ). So, step 5 is no longer required? :-D > > It's more of an optimization than a requirement, to avoid expensive > calculation later. Uncommenting is possible but I need to pass the > pack file name around a bit. > Ah, OK. (I should probably refrain from commenting on code I haven't read ... :-P ) ATB, Ramsay Jones -- 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