testthreadexecutor.cpp: added TODO

firewave · firewave · commit ae8c19d12e01 · 2023-09-27T11:44:18.000+02:00
diff --git a/test/testthreadexecutor.cpp b/test/testthreadexecutor.cpp
@@ -260,6 +260,7 @@ class TestThreadExecutor : public TestFixture {
         return n;
     }
 
+    // TODO: should this be logged only once like summary?
     void showtime_top5() {
         REDIRECT; // should not cause TSAN failures as the showtime logging is synchronized
         check(2, 2, 0,

Original file line number	Diff line number	Diff line change
`@@ -260,6 +260,7 @@ class TestThreadExecutor : public TestFixture {`
`260`	`260`	`return n;`
`261`	`261`	`}`
`262`	`262`
	`263`	`+ // TODO: should this be logged only once like summary?`
`263`	`264`	`void showtime_top5() {`
`264`	`265`	`REDIRECT; // should not cause TSAN failures as the showtime logging is synchronized`
`265`	`266`	`check(2, 2, 0,`