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
