diff options
author | MarcoFalke <falke.marco@gmail.com> | 2022-01-17 08:50:36 +0100 |
---|---|---|
committer | MarcoFalke <falke.marco@gmail.com> | 2022-01-17 08:50:40 +0100 |
commit | dbf81a73e37141f86a65a0f96ff0f740f74ec045 (patch) | |
tree | 01ef5e20a2f68c37a596c4950cc047d6363dc03b /src/wallet/test/wallet_tests.cpp | |
parent | 92a71f65be7bf084ff30fe6712d5edaf1ed83364 (diff) | |
parent | 30927cb5306d01da2716786c2d9457c49ec49d0f (diff) | |
download | bitcoin-dbf81a73e37141f86a65a0f96ff0f740f74ec045.tar.xz |
Merge bitcoin/bitcoin#24079: refactor: replace RecursiveMutex `cs_SubVer` with Mutex (and rename)
30927cb5306d01da2716786c2d9457c49ec49d0f refactor: replace RecursiveMutex `m_subver_mutex` with Mutex (Sebastian Falbesoner)
0639aba42ad8449fe43eb2aad0ffbe3390aabd37 scripted-diff: rename `cs_SubVer` -> `m_subver_mutex` (Sebastian Falbesoner)
Pull request description:
This PR is related to #19303 and gets rid of the RecursiveMutex `cs_SubVer`. Both of the critical sections only directly access the guarded variable, i.e. it is not possible that within one section another one is called, and we can use a regular Mutex.
ACKs for top commit:
hebasto:
ACK 30927cb5306d01da2716786c2d9457c49ec49d0f, I have reviewed the code and it looks OK, I agree it can be merged.
Tree-SHA512: 2faead792ea0b2f79f9f7fe99acde5cf2bfcd2f15c51fbb6cb1099d4f81276001a492f7d46a5139055f4366c2d58a36a8ba19f21d56df20e0ed93af3141dbe11
Diffstat (limited to 'src/wallet/test/wallet_tests.cpp')
0 files changed, 0 insertions, 0 deletions