Closed nmcclatchey closed 6 years ago
I'm not really sure why GitHub Desktop added all the extraneous commits at the beginning.
A question: Should the recursive-lock checking be disabled by NDEBUG
? The C++ standard only mandates "undefined behavior", so defaulting to high-performance would be the expected behavior. I recommend the following:
Document a new macro: STDMUTEX_RECURSION_CHECKS
.
!defined(NDEBUG)
.I've added a commit with this behavior, to be reverted if there is no support for it.
Improves debugging features of the library. Specifically,
shared_mutex
.std::terminate()
) ifNDEBUG
is undefined and a thread is moved to or destroyed while in a joinable state.