First beautiful comment

Author: EuphoricThinking

test/memspaces/memspace_lowest_latency.cpp

@@ -10,12 +10,7 @@
 #include "test_helpers.h"
 
 /*
-
-canQueryLatency is used in a parameter generator as a functional pointer
- https://man.archlinux.org/man/hwloc_memattr_get_value.3.en
- 0 on success - hwloc_memattr_get_value
- assume overall failure
-expect instead of add_failure: more verbosity
+canQueryLatency is used in a parameter generator as a functional pointer and is not a standalone test, thus it cannot contain GTEST asserts. EXPECT_* or ADD_FAILURE() do not cancel the execution of the code following after, but they interfere with GTEST_SKIP() in fixtures: the execution of the tests included in the suite assigned to the given fixture is not cancelled if EXPECT_* or ADD_FAILURE() are used. Therefore a custom logging macro is used.
 */
 
 static bool canQueryLatency(size_t nodeId) {