From: Junio C Hamano Date: Tue, 30 Jan 2024 16:01:57 GMT Subject: Re: [PATCH 0/4] mark tests as leak-free Message-ID: In-Reply-To: <20240130055448.GC166761@coredump.intra.peff.net> Jeff King writes: > On Mon, Jan 29, 2024 at 10:04:10PM +0100, Rubén Justo wrote: > >> 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). Yup, the background description was very useful to read. Thanks.