diff options
| author | Wladimir J. van der Laan <laanwj@protonmail.com> | 2019-11-06 00:04:22 +0100 |
|---|---|---|
| committer | Wladimir J. van der Laan <laanwj@protonmail.com> | 2019-11-06 00:04:38 +0100 |
| commit | 45e65376ac9e88704c2a01c6105e0ee506a13324 (patch) | |
| tree | 3b7fac4e2a7499dd5b128f1ed78dea6f614dd610 /src/wallet/test/init_tests.cpp | |
| parent | 40b6070ad7e84fa1fce920d9200edbf3facdbd6f (diff) | |
| parent | fa5facd3e72b6d61374b0b93b722b55e2b090020 (diff) | |
Merge #17382: rpc: Remove unused boost::this_thread::interruption_point
fa5facd3e72b6d61374b0b93b722b55e2b090020 rpc: Remove unused boost::this_thread::interruption_point (MarcoFalke)
Pull request description:
There are predefined interruption points for `boost::thread`: https://www.boost.org/doc/libs/1_71_0/doc/html/thread/thread_management.html#interruption_points
However, the rpc threads are `std::thread`, which does not have an `std::thread::interrupt` member function to request interruption: https://dev.visucore.com/bitcoin/doxygen/httpserver_8cpp.html#ae1a63374e18b9abd348eb74e4243ea34
Thus, the interruption points can be removed.
ACKs for top commit:
laanwj:
ACK fa5facd3e72b6d61374b0b93b722b55e2b090020, this does nothing.
practicalswift:
ACK fa5facd3e72b6d61374b0b93b722b55e2b090020
jamesob:
ACK https://github.com/bitcoin/bitcoin/pull/17382/commits/fa5facd3e72b6d61374b0b93b722b55e2b090020
Tree-SHA512: 4e29a44df1f2702cbd1ffdffa559440a8bb800baab64b4116e2c3d27cd64d8d1e8aafe1dc21b1a4e3988470d03be19cae294bd5669f7abf6d487685dc8fd8d7e
Diffstat (limited to 'src/wallet/test/init_tests.cpp')
0 files changed, 0 insertions, 0 deletions