diff options
| author | fanquake <fanquake@gmail.com> | 2020-01-15 13:23:49 +0800 |
|---|---|---|
| committer | fanquake <fanquake@gmail.com> | 2020-01-15 13:46:21 +0800 |
| commit | af05bd9e1e362c3148e3b434b7fac96a9a5155a1 (patch) | |
| tree | 4b76b482bac58d459a506e057535832d80b9a553 /test/functional/rpc_setban.py | |
| parent | ceb789cf3a9075729efa07f5114ce0369d8606c3 (diff) | |
| parent | e09c701e0110350f78366fb837308c086b6503c0 (diff) | |
Merge #17891: scripted-diff: Replace CCriticalSection with RecursiveMutex
e09c701e0110350f78366fb837308c086b6503c0 scripted-diff: Bump copyright of files changed in 2020 (MarcoFalke)
6cbe6209646db8914b87bf6edbc18c6031a16f1e scripted-diff: Replace CCriticalSection with RecursiveMutex (MarcoFalke)
Pull request description:
`RecursiveMutex` better clarifies that the mutex is recursive, see also the standard library naming: https://en.cppreference.com/w/cpp/thread/recursive_mutex
For that reason, and to avoid different people asking me the same question repeatedly (e.g. https://github.com/bitcoin/bitcoin/pull/15932#pullrequestreview-339175124 ), remove the outdated alias `CCriticalSection` with a scripted-diff
ACKs for top commit:
Empact:
ACK https://github.com/bitcoin/bitcoin/pull/17891/commits/e09c701e0110350f78366fb837308c086b6503c0 diff and scripts look correct
promag:
ACK e09c701e0110350f78366fb837308c086b6503c0
practicalswift:
ACK e09c701e0110350f78366fb837308c086b6503c0 -- scripted diff looks correct
Tree-SHA512: 4bd7b5de1befdcf91dc8f43c127a1fee49679e06895a43216f160344a395c8e426dc68d529fbd2d5e1c215625a5a392dc415b1bce4127316aae7ecf98030c855
Diffstat (limited to 'test/functional/rpc_setban.py')
0 files changed, 0 insertions, 0 deletions