Fix 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?

a

Author: IlanTruanovsky

test/acl_event_test.cpp

@@ -24,6 +24,8 @@
 #include <acl_util.h>
 #include <unref.h>
 
+#include <inttypes.h>
+
 #include "acl_test.h"
 
 MT_TEST_GROUP(acl_event) {
@@ -899,8 +901,9 @@ MT_TEST(acl_event_default_config, event_dependency_exec_order) {
       // Internally consistent event profile.
       // Recall the example HAL bumps up the timestamp on every call.
       CHECK(queued_times[i] < submit_times[i]);
-      acl_print_debug_msg(" Event[%d] times = { %lu %lu %lu %lu }\n", i,
-                          queued_times[i], submit_times[i], start_times[i],
+      acl_print_debug_msg(" Event[%d] times = { %" PRIu64 " %" PRIu64
+                          " %" PRIu64 " %" PRIu64 " }\n",
+                          i, queued_times[i], submit_times[i], start_times[i],
                           end_times[i]);
       CHECK(submit_times[i] < start_times[i]);
       CHECK(start_times[i] < end_times[i]);
@@ -915,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]);
       }
     }