Subject: Re: ab/commit-plug-leaks + ab/diff-free-more

From: Junio C Hamano <[email-masked]>
Date: Thu, 24 Feb 2022 15:38:28 -0800

Ævar Arnfjörð Bjarmason <[email-masked]> writes:

> 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?

Do not count and aim to reduce the number of UNLEAK in builtin/
directory.  Re-read [1] until you get it.

Thanks.