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