From: Thomas Rast <tr@thomasrast.ch>
Date: Fri, 22 Nov 2013 16:52:37 GMT
Subject: Re: What's cooking in git.git (Nov 2013, #05; Thu, 21)
Message-ID: <87d2lsjs4q.fsf@linux-k42r.v.cablecom.net>
In-Reply-To: <20131122102345.GC12042@sigill.intra.peff.net>

Jeff King <peff@peff.net> writes:

>> * jk/pack-bitmap (2013-11-18) 22 commits
[...]
>>  Borrows the bitmap index into packfiles from JGit to speed up
>>  enumeration of objects involved in a commit range without having to
>>  fully traverse the history.
>
> Looks like you picked up my latest re-roll with Ramsay's fix on top.
> There wasn't a lot of review on this past round (I'm not surprised; it's
> a dauntingly large chunk to review).  I outlined a few possible open
> issues in the cover letter, but I'd be happy to build those on top,
> which I think will make review of them a lot easier.
>
> Do we want to try this in 'next' post-1.8.5, or should I try to prod an
> area expert like Shawn into doing another round of review?

Hmm, maybe I missed something, but AFAICS you (or Vicent) never acted on
or responded to my June reviews in this thread:

  http://thread.gmane.org/gmane.comp.version-control.git/228918

and again mentioned here, though I didn't point out all of them:

  http://thread.gmane.org/gmane.comp.version-control.git/236587/focus=236740

Granted, the way I verified this was checking whether you renamed
rlw_xor_run_bit() to something more fitting, so perhaps you just forgot
that one thing but did all the rest.

-- 
Thomas Rast
tr@thomasrast.ch