On Wed, May 18, 2011 at 03:08:37AM -0400, Jeff King wrote: > Part of the problem is that with the current code, all you care about is > "Is it loose, packed non-delta, or packed delta?". > [...] > The patch below implements the cheap "is_delta" check. Hmm, sorry, this patch works well on top of cd36b7b, where I first detected the slowness, but later in the series we actually do look at the pack information in the object_info. So my patch breaks that code path horribly (I still think the concept of avoiding the revindex should still work in principle, though). -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