Re: [PATCH 0/4] mark tests as leak-free

From: Jeff King <peff@peff.net>
Date: Jan 30, 2024, 05:54 UTC
Message-ID: <20240130055448.GC166761@coredump.intra.peff.net>
In-Reply-To: <45eb0748-6415-4e52-a54f-8d4e5ad57dde@gmail.com>

On Mon, Jan 29, 2024 at 10:04:10PM +0100, Rubén Justo wrote:

Show 10 quoted lines

> The tests: t0080, t5332 and t6113 can be annotated as leak-free.
> 
> I used:
>   $ make SANITIZE=leak GIT_TEST_PASSING_SANITIZE_LEAK=check GIT_TEST_SANITIZE_LEAK_LOG=true test
> 
> Rubén Justo (4):
>   t0080: mark as leak-free
>   t5332: mark as leak-free
>   t6113: mark as leak-free
>   test-lib: check for TEST_PASSES_SANITIZE_LEAK

These all looked reasonable to me. Thank you for not just fixing them, but including the background for each case (e.g., leak-free as of commit XYZ, etc).

-Peff