On Thu, 6 Aug 2009, Nick Edelen wrote:
Hrmm, I just realized that it dosn't actually cache paths/names...
This obviously has no bearing on its use in packing, but I should
either add that in or restrict usage in non-packing-related walks.
Weird how things like that escape you.
Actually it is really the packing related walk that would benefit the
most from this work. The "counting objects" phase of a clone may take
quite a while with some repositories. Most other operations don't care
as much because the rev walk is done incrementally whereas the packing
operation needs to perform it all up front.
Nicolas