| Message ID | 20231003202609.GA7812@coredump.intra.peff.net (mailing list archive) |
|---|---|
| State | Accepted |
| Commit | be4b578c69e0ac1d974ebc9163ff5793a018da8e |
| Headers | show |
| Series | some commit-graph leak fixes | expand |
On Tue, Oct 03, 2023 at 04:26:09PM -0400, Jeff King wrote: > This test has never leaked since it was added. Let's annotate it to make > sure it stays that way (and to reduce noise when looking for other > leak-free scripts after we fix some leaks). > > Signed-off-by: Jeff King <peff@peff.net> > --- > Obviously not directly related to the rest; this could be spun off to > its own series, or put atop jk/tree-name-and-depth-limit and merged from > there. I wondered how I missed this in tb/mark-more-tests-as-leak-free, but the answer is trivial: that topic preceded your tree depth stuff ;-). This change makes good sense, and I don't think it's worth spinning off into its own series. Thanks, Taylor
diff --git a/t/t6700-tree-depth.sh b/t/t6700-tree-depth.sh index e410c41234..9e70a7c763 100755 --- a/t/t6700-tree-depth.sh +++ b/t/t6700-tree-depth.sh @@ -1,6 +1,8 @@ #!/bin/sh test_description='handling of deep trees in various commands' + +TEST_PASSES_SANITIZE_LEAK=true . ./test-lib.sh # We'll test against two depths here: a small one that will let us check the
This test has never leaked since it was added. Let's annotate it to make sure it stays that way (and to reduce noise when looking for other leak-free scripts after we fix some leaks). Signed-off-by: Jeff King <peff@peff.net> --- Obviously not directly related to the rest; this could be spun off to its own series, or put atop jk/tree-name-and-depth-limit and merged from there. t/t6700-tree-depth.sh | 2 ++ 1 file changed, 2 insertions(+)