Subject: Re: [PATCH 05/24] revision.[ch]: provide and start using a release_revisions()

From: Taylor Blau <[email-masked]>
Date: Wed, 9 Mar 2022 16:32:53 -0500

On Wed, Mar 09, 2022 at 02:16:35PM +0100, Ævar Arnfjörð Bjarmason wrote:
> Let's provide a release_revisions() function for these users, and
> start migrating them over per the plan outlined in [1]. Right now this
> only handles the "pending" member of the struct, but more will be
> added in subsequent commits.

I could not be more excited about this change! The lack of easy
mechanism to free everything in the rev_info structure has been a major
impediment for me to get much value out of the SANITIZE=leak builds, and
this is a huge step towards making those more useful.

This plan looks good to me, as does this first patch to introduce the
new API.

Should it be documented anywhere in e.g.,
Documentation/MyFirstObjectWalk.txt or elsewhere?

Thanks,
Taylor