(Included a separate patch in this patch series, they all concern the same file)
Don't report late timeouts using CU_FAIL as this interferes with the cunit
test framework. Just count and report (using LOG_DBG) the number of late
timeouts. (Discussed with Mike Holmes)
Don't dereference pointer after successful check for NULL as this makes Coverity
complain. (Coverity CID 85397, https://bugs.linaro.org/show_bug.cgi?id=1056)
Use nanosleep instead of the deprecated usleep. Define POSIX version to be able
to use rand_r. (https://bugs.linaro.org/show_bug.cgi?id=1048)
Use CU_ASSERT_FATAL instead of plain assert so to work better with the cunit
test framework.
Use LOG_DBG instead of printf. Remove some unnecessary printouts.
Ensure we run at least one worker thread.
Replace usage of perror and abort with cunit friendly operations.


Ola Liljedahl (5):
  validation: odp_timer.c: avoid cunit warnings for late timeouts
  validation: odp_timer.c: avoid dereferencing ptr after NULL check
  validation: odp_timer.c: use nanosleep instead of usleep
  validation: odp_timer.c: cunit cleanup
  validation: odp_timer.c: remove usage of perror/abort

 test/validation/odp_timer.c | 105 +++++++++++++++++++++++++-------------------
 1 file changed, 60 insertions(+), 45 deletions(-)

-- 
1.9.1


_______________________________________________
lng-odp mailing list
[email protected]
http://lists.linaro.org/mailman/listinfo/lng-odp

Reply via email to