On Wed, Oct 09, 2019 at 04:44:22PM -0700, Jonathan Tan wrote: > Signed-off-by: Jonathan Tan <jonathantanmy@xxxxxxxxxx> > --- > builtin/index-pack.c | 267 ++++++++++++++++++++----------------------- > 1 file changed, 127 insertions(+), 140 deletions(-) I think this is a good direction to go in. I confess I didn't carefully go over the implementation details, since you've marked this as RFC and it sounds like you're mainly asking about direction. It looks pretty reasonable from a high level, though. -Peff