From: Jeff King <peff@peff.net>
Date: Tue, 12 Jan 2021 08:50:26 GMT
Subject: Re: [PATCH 04/20] write_reused_pack_verbatim(): convert to new revindex API
Message-ID: <X/1i0pLQ5fcrlIoj@coredump.intra.peff.net>
In-Reply-To: <14b35d01a062f2dfdd710718b659064042dc21d6.1610129796.git.me@ttaylorr.com>

On Fri, Jan 08, 2021 at 01:16:56PM -0500, Taylor Blau wrote:

> diff --git a/builtin/pack-objects.c b/builtin/pack-objects.c
> index ea7df9270f..4341bc27b4 100644
> --- a/builtin/pack-objects.c
> +++ b/builtin/pack-objects.c
> @@ -948,7 +948,7 @@ static size_t write_reused_pack_verbatim(struct hashfile *out,
>  		off_t to_write;
>  
>  		written = (pos * BITS_IN_EWORD);
> -		to_write = reuse_packfile->revindex[written].offset
> +		to_write = pack_pos_to_offset(reuse_packfile, written)
>  			- sizeof(struct pack_header);

This one is obviously correct.

-Peff