On Fri, Jul 20, 2007 at 11:11:19PM -0700, Linus Torvalds wrote: > Here's a patch. Not very well tested, but it makes gitk happy and passes > all the tests. And I really think Junio's code was very dubious (it did > that "p = p->next" *every* time, and then did "pp = &p->next", so "pp" > would end up jumping by multiple entries. I just independently came to the same conclusion. Your patch looks correct to me (though there are some unrelated formatting changes which made it a little harder to read). Acked-by: Jeff King <peff@xxxxxxxx> -Peff - 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