Skip to content

Commit 487c7ef

Browse files
IlanTruanovskypcolberg
authored andcommitted
Fix Coverity COPY_PASTE_ERROR
This was not a real copy paste error. We intended to have the code do what we indicated. So, our two options were to tell Coverity to ignore this line of code, or change the code slightly so Coverity doesn't recognize it as a copy-paste error. We chose to do the latter. Fixes: test/acl_event_test.cpp:918:9: Type: Copy-paste error (COPY_PASTE_ERROR) test/acl_event_test.cpp:914:9: original: "submit_times[i]" looks like the original copy. test/acl_event_test.cpp:918:9: copy_paste_error: "submit_times" in "submit_times[i]" looks like a copy-paste error. test/acl_event_test.cpp:918:9: remediation: Should it say "end_times" instead?
1 parent 6635c79 commit 487c7ef

File tree

1 file changed

+1
-1
lines changed

1 file changed

+1
-1
lines changed

test/acl_event_test.cpp

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -918,7 +918,7 @@ MT_TEST(acl_event_default_config, event_dependency_exec_order) {
918918

919919
// Check separation between dependent events.
920920
// That is, they actually waited for the prior events.
921-
CHECK(end_times[i - 1] < submit_times[i]);
921+
CHECK(submit_times[i] > end_times[i - 1]);
922922
}
923923
}
924924

0 commit comments

Comments
 (0)