diff options
author | MarcoFalke <falke.marco@gmail.com> | 2021-01-05 12:19:03 +0100 |
---|---|---|
committer | MarcoFalke <falke.marco@gmail.com> | 2021-01-05 12:19:08 +0100 |
commit | 417f95fa453d97087a33d4176523ab278bef21a1 (patch) | |
tree | 1f1bc910d2458e291a155d6aec2ef42ebde40ae8 /src | |
parent | fafd725a7c6cfc64f37af5074a8cf660db93e0ea (diff) | |
parent | 0e51a355128a825d428fe2b9017c25085731fc04 (diff) | |
download | bitcoin-417f95fa453d97087a33d4176523ab278bef21a1.tar.xz |
Merge #19915: p2p, refactor: Use Mutex type for some mutexes in CNode class
0e51a355128a825d428fe2b9017c25085731fc04 refactor: Use Mutex type for some mutexes in CNode class (Hennadii Stepanov)
Pull request description:
No need the `RecursiveMutex` type for the `CNode::cs_vSend`, `CNode::cs_hSocket` and `CNode::cs_vRecv`.
Related to #19303.
ACKs for top commit:
jnewbery:
utACK 0e51a35512
MarcoFalke:
review ACK 0e51a355128a825d428fe2b9017c25085731fc04 🔊
Tree-SHA512: 678ee5e3c15ad21a41cb86ec7179741bd505a138638fdc07f41d6d677c38fbf2208219bfc0509e3675e721fc8d8816e858070db7b87c5d72ad93aae81f7e1636
Diffstat (limited to 'src')
-rw-r--r-- | src/net.h | 6 |
1 files changed, 3 insertions, 3 deletions
@@ -853,9 +853,9 @@ public: size_t nSendOffset{0}; // offset inside the first vSendMsg already sent uint64_t nSendBytes GUARDED_BY(cs_vSend){0}; std::deque<std::vector<unsigned char>> vSendMsg GUARDED_BY(cs_vSend); - RecursiveMutex cs_vSend; - RecursiveMutex cs_hSocket; - RecursiveMutex cs_vRecv; + Mutex cs_vSend; + Mutex cs_hSocket; + Mutex cs_vRecv; RecursiveMutex cs_vProcessMsg; std::list<CNetMessage> vProcessMsg GUARDED_BY(cs_vProcessMsg); |