diff options
author | MarcoFalke <falke.marco@gmail.com> | 2020-06-09 08:54:52 -0400 |
---|---|---|
committer | MarcoFalke <falke.marco@gmail.com> | 2020-06-09 08:54:57 -0400 |
commit | 9ad6f14175c19b5d47267c3dd5840c14bad43c83 (patch) | |
tree | 23af49ae07c02d1150a28256e93d0ba6708d6f90 /src/secp256k1/include/secp256k1_recovery.h | |
parent | a79bca2f1fb25f433d6e100a31a3acfde2656ce1 (diff) | |
parent | bacbfb61eee6d3c32de3db4dea3f585c7159b643 (diff) |
Merge #19220: refactor: Replace RecursiveMutex with Mutex in warnings.cpp
bacbfb61eee6d3c32de3db4dea3f585c7159b643 refactor: Replace RecursiveMutex with Mutex in warnings.cpp (Hennadii Stepanov)
Pull request description:
The functions that could lock this mutex, i.e., `SetMiscWarning()`, `{S,G}etfLargeWorkForkFound()`, `SetfLargeWorkInvalidChainFound()`, `GetWarnings()`, do not call itself recursively, and do not call each other either directly or indirectly. Therefore, the `g_warnings_mutex` could be a non-recursive mutex.
Related to #19180.
ACKs for top commit:
laanwj:
Code review ACK bacbfb61eee6d3c32de3db4dea3f585c7159b643
MarcoFalke:
ACK bacbfb61eee6d3c32de3db4dea3f585c7159b643 , reviewed with -W --word-diff-regex=. 🎿
Tree-SHA512: cc06d3d30e4051115d176dcfbd496c8562a70087369bccde756c1de42d7dc3f415ef20d3d69ad2599c1d0cd4228d604d7564adc17beac7b6ff92b924b8c20d54
Diffstat (limited to 'src/secp256k1/include/secp256k1_recovery.h')
0 files changed, 0 insertions, 0 deletions