From: Taylor Blau <me@ttaylorr.com>
Date: Thu, 11 Feb 2021 03:04:51 GMT
Subject: Re: [PATCH 7/9] pack-revindex: read multi-pack reverse indexes
Message-ID: <YCSe0/OIXisrgNP6@nand.local>
In-Reply-To: <3680ce51-81ba-d6a3-a0b6-6f3a7cc14503@gmail.com>

On Wed, Feb 10, 2021 at 09:53:23PM -0500, Derrick Stolee wrote:
> ...this assumes the hash is of the same length as the_hash_algo,
> so you are doing the right thing. Currently, I think we check
> that 'm->hash_len == the_hash_algo->rawsz' on load. We'll need
> to check this again later when in the transition phase of the
> new hash work.
>
> (No changes are needed to your patch.)

All makes sense, thanks.

Taylor