Subject: ab/commit-plug-leaks + ab/diff-free-more (was: What's cooking in git.git (Feb 2022, #06; Wed, 23))

From: Ævar Arnfjörð Bjarmason <[email-masked]>
Date: Thu, 24 Feb 2022 10:08:59 +0100

On Wed, Feb 23 2022, Junio C Hamano wrote:

> * ab/commit-plug-leaks (2022-02-16) 2 commits
>  - commit: use strbuf_release() instead of UNLEAK()
>  - commit: fix "author_ident" leak
>
>  Leakfixes in the top-level called-once function.
>
>  Expecting a reroll.
>  I think UNLEAK->strbuf_release() is a regression.
>  source: <[email-masked]>

After you noted that in [1] I followed-up in [2] giving reasons for
wanting to keep that UNLEAK->strbuf_release() change. What do you think
about that?

> * ab/diff-free-more (2022-02-16) 2 commits
>   (merged to 'next' on 2022-02-17 at be8ae78a2d)
>  + diff.[ch]: have diff_free() free options->parseopts
>  + diff.[ch]: have diff_free() call clear_pathspec(opts.pathspec)
>
>  Leakfixes.
>
>  Will merge to 'master'.
>  source: <[email-masked]>

Thanks! It will be great to have this merged down!

1. https://lore.kernel.org/git/[email-masked]/
2. https://lore.kernel.org/git/[email-masked]