tree 2ee8b88459c08d2b91faeed59c3095d34e25c2b1
parent da5df37c4d1a2841c6f9cdc626386d096f5f312d
author Chris Dickens <christopher.a.dickens@gmail.com> 1598488979 -0700
committer Chris Dickens <christopher.a.dickens@gmail.com> 1599980771 -0700

core: Simplify thread abstractions and add debug checks

The POSIX thread mutex initialization function can potentially fail, but
in practice this is unlikely to occur. There is also inconsistent use of
the result of the mutex initialization within the library. The result is
only checked for mutexes in the libusb_device and libusb_device_handle
structures but is ignored in all other cases.

Simplify the mutex initialization function by changing the abstraction's
wrapper to a void function, much like all the other functions that
already exist. To that end, introduce macros for the abstractions that
will check the return value on debug builds.

Also remove the dependence on the core library needing errno.h to
translate errors from usbi_cond_timedwait(). The abstraction will
convert the implementation-specific error codes to LIBUSB_ERROR values.

Signed-off-by: Chris Dickens <christopher.a.dickens@gmail.com>