radix_tree: fix pmemcheck annotation#1123
Conversation
Use annotation only if PMEMCHECK is enabled (not just any valgrind tool).
lukaszstolarczuk
left a comment
There was a problem hiding this comment.
Reviewed 1 of 1 files at r1.
Reviewable status: all files reviewed, 1 unresolved discussion (waiting on @igchor)
a discussion (no related file):
just to double-check - it only applies in this one place...? 😉
kilobyte
left a comment
There was a problem hiding this comment.
Reviewable status: all files reviewed, 1 unresolved discussion (waiting on @lukaszstolarczuk)
a discussion (no related file):
Previously, lukaszstolarczuk (Łukasz Stolarczuk) wrote…
just to double-check - it only applies in this one place...? 😉
Yeah — at least everything builds for me with stock valgrind (Debian unstable + clang12 from experimental).
Codecov Report
@@ Coverage Diff @@
## master #1123 +/- ##
==========================================
+ Coverage 94.20% 94.26% +0.05%
==========================================
Files 52 52
Lines 5301 5298 -3
==========================================
Hits 4994 4994
+ Misses 307 304 -3
Flags with carried forward coverage won't be shown. Click here to find out more.
Continue to review full report at Codecov.
|
Use annotation only if PMEMCHECK is enabled (not just any valgrind tool).
This change is