From: Jeff King Date: Wed, 18 May 2011 07:50:23 GMT Subject: Re: [PATCH 00/11] writing out a huge blob to working tree Message-ID: <20110518075023.GD27482@sigill.intra.peff.net> In-Reply-To: <20110518070837.GC27482@sigill.intra.peff.net> 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