Fix Coverity issues for acl_event_test.cpp

[IlanTruanovsky]

For the copy paste error, the supporting comment (and a brief look over the code) makes me believe this was intentional.

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?

test/acl_event_test.cpp:903:27:
  Type: Invalid type in argument to printf format specifier (PRINTF_ARGS)

test/acl_event_test.cpp:903:27:
  invalid_type: Argument "queued_times[i]" to format specifier "%d" was expected to have type "int" but has type "unsigned long".
src/acl_hal.cpp:123:3:
  path: Condition "debug_mode > 0", taking true branch.
src/acl_hal.cpp:126:5:
  printf_function: Calling "vprintf" which uses a "printf"-style format string. [Note: The source code implementation of the function has been overridden by a builtin model.]

test/acl_event_test.cpp:903:44:
  Type: Invalid type in argument to printf format specifier (PRINTF_ARGS)

test/acl_event_test.cpp:903:44:
  invalid_type: Argument "submit_times[i]" to format specifier "%d" was expected to have type "int" but has type "unsigned long".
src/acl_hal.cpp:123:3:
  path: Condition "debug_mode > 0", taking true branch.
src/acl_hal.cpp:126:5:
  printf_function: Calling "vprintf" which uses a "printf"-style format string. [Note: The source code implementation of the function has been overridden by a builtin model.]

test/acl_event_test.cpp:903:61:
  Type: Invalid type in argument to printf format specifier (PRINTF_ARGS)

test/acl_event_test.cpp:903:61:
  invalid_type: Argument "start_times[i]" to format specifier "%d" was expected to have type "int" but has type "unsigned long".
src/acl_hal.cpp:123:3:
  path: Condition "debug_mode > 0", taking true branch.
src/acl_hal.cpp:126:5:
  printf_function: Calling "vprintf" which uses a "printf"-style format string. [Note: The source code implementation of the function has been overridden by a builtin model.]

test/acl_event_test.cpp:904:27:
  Type: Invalid type in argument to printf format specifier (PRINTF_ARGS)

test/acl_event_test.cpp:904:27:
  invalid_type: Argument "end_times[i]" to format specifier "%d" was expected to have type "int" but has type "unsigned long".
src/acl_hal.cpp:123:3:
  path: Condition "debug_mode > 0", taking true branch.
src/acl_hal.cpp:126:5:
  printf_function: Calling "vprintf" which uses a "printf"-style format string. [Note: The source code implementation of the function has been overridden by a builtin model.]

[pcolberg]

Thanks @IlanTruanovsky, the change itself is good. Could you fixup the commits so that each commit contains the change it describes? (The printf format change is currently in the copy-paste commit.) There is a trailing line a in the copy-paste commit message.