[PATCH 0/3] lsan test-lib readability
- From
Jeff King <peff@peff.net>
- Date
- Jan 7, 2025, 07:04 UTC
- Message-ID
- <20250107070409.GA584456@coredump.intra.peff.net>
- In-Reply-To
- <20250101201226.GA3304465@coredump.intra.peff.net>
On Wed, Jan 01, 2025 at 03:12:26PM -0500, Jeff King wrote:
Show 6 quoted lines
> [1/6]: test-lib: use individual lsan dir for --stress runs > [2/6]: Revert "index-pack: spawn threads atomically" > [3/6]: test-lib: rely on logs to detect leaks > [4/6]: test-lib: simplify leak-log checking > [5/6]: test-lib: check leak logs for presence of DEDUP_TOKEN > [6/6]: test-lib: ignore leaks in the sanitizer's thread code
And here are a few cosmetic improvements on top based on the review from Patrick. They should apply on top of jk/lsan-race-ignore-false-positive, though that is also in 'master' now.
[1/3]: test-lib: invert return value of check_test_results_san_file_empty [2/3]: test-lib: simplify lsan results check [3/3]: test-lib: add a few comments to LSan log checking
t/test-lib-functions.sh | 2 +- t/test-lib.sh | 20 ++++++++++---------- 2 files changed, 11 insertions(+), 11 deletions(-)
-Peff