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?

Author: IlanTruanovsky

test/acl_event_test.cpp

@@ -918,7 +918,7 @@ MT_TEST(acl_event_default_config, event_dependency_exec_order) {
 
         // Check separation between dependent events.
         // That is, they actually waited for the prior events.
-        CHECK(end_times[i - 1] < submit_times[i]);
+        CHECK(submit_times[i] > end_times[i - 1]);
       }
     }