diff options
| author | fanquake <fanquake@gmail.com> | 2021-01-11 09:38:14 +0800 |
|---|---|---|
| committer | fanquake <fanquake@gmail.com> | 2021-01-11 10:14:11 +0800 |
| commit | c4458cc3a1d13bc0fb2f9c627e71cd3c8914575d (patch) | |
| tree | d9f6b2a665a9bdb3da2bf13b70aeea28933df6ae /src/test/fuzz/connman.cpp | |
| parent | 094402430925ec5aac6edbbf52d74f10c665da43 (diff) | |
| parent | fad1f0fd33e5e7a65b702237c7ca8e1b694852d2 (diff) | |
Merge #18819: net: Replace cs_feeFilter with simple std::atomic
fad1f0fd33e5e7a65b702237c7ca8e1b694852d2 net: Remove unused cs_feeFilter (MarcoFalke)
Pull request description:
A `RecursiveMutex` is overkill for setting or reading a plain integer. Even a `Mutex` is overkill, when a plain `std::atomic` can be used.
This removes 11 lines of code. Also, it is cutting down on the number of locks put on the stack at the same time, which complicates review looking out for potential lock contention.
ACKs for top commit:
jnewbery:
utACK fad1f0fd33e5e7a65b702237c7ca8e1b694852d2
practicalswift:
cr ACK fad1f0fd33e5e7a65b702237c7ca8e1b694852d2: patch looks correct
Tree-SHA512: 647f9b954fbf52e138d3e710937eb9131b390fef0deae03fd6a162d5a18b9f194010800bbddc8f89208d91be2802dff11c3884d04b3dd233865abd12aa3cde06
Diffstat (limited to 'src/test/fuzz/connman.cpp')
0 files changed, 0 insertions, 0 deletions