test: Replace boost::mutex with std::mutex #18695

pull hebasto wants to merge 1 commits into bitcoin:master from hebasto:200417-boost-mu changing 1 files +5 −3

hebasto commented at 11:08 PM on April 17, 2020: member

This PR replaces boost::mutex with std::mutex in the scheduler_tests test suite.
test: Replace boost::mutex with std::mutex 27abd1a4f4
fanquake added the label Tests on Apr 17, 2020
theStack approved
theStack commented at 6:52 PM on April 18, 2020: member

ACK https://github.com/bitcoin/bitcoin/pull/18695/commits/27abd1a4f4c7a3d092d59edbbaa1e0f324c8b0ef
sipa commented at 11:46 PM on April 18, 2020: member

utACK 27abd1a4f4c7a3d092d59edbbaa1e0f324c8b0ef
fanquake merged this on Apr 19, 2020
fanquake closed this on Apr 19, 2020
MarcoFalke commented at 10:16 AM on April 19, 2020: member

Don't we use Mutex and LOCK() for that?
hebasto deleted the branch on Apr 19, 2020
hebasto commented at 2:58 PM on April 20, 2020: member

@MarcoFalke

Don't we use Mutex and LOCK() for that?

Sorry, I did not think it is important to use thread safety annotated entities in unit tests.

Should I submit a new pr?
ComputerCraftr referenced this in commit e71d02d9e4 on Jun 16, 2020
Fabcien referenced this in commit 5d76027dc4 on Jan 18, 2021
vijaydasmp referenced this in commit 8038726736 on Aug 22, 2021
PastaPastaPasta referenced this in commit f5abcec7b3 on Oct 20, 2021
pravblockc referenced this in commit 61be456c0b on Nov 18, 2021
DrahtBot locked this on Feb 15, 2022

Contributors

Labels