diff options
| author | MarcoFalke <falke.marco@gmail.com> | 2021-12-07 18:48:14 +0100 |
|---|---|---|
| committer | MarcoFalke <falke.marco@gmail.com> | 2021-12-07 18:48:33 +0100 |
| commit | eaf1c56502455006962ee50390d7485befde0869 (patch) | |
| tree | aaa71c08a3ab790d277e01aeef68b5731e3f9eb0 /doc | |
| parent | 13f41855c5fedf11d4a2525f2f0dd214533e5e62 (diff) | |
| parent | 275e9390e1c84ac021b3c781ee239ad9ba7b78d4 (diff) | |
| download | bitcoin-eaf1c56502455006962ee50390d7485befde0869.tar.xz | |
Merge bitcoin/bitcoin#23692: mining, refactor: add m_mempool.cs thread safety lock assertions
275e9390e1c84ac021b3c781ee239ad9ba7b78d4 mining, refactor: add m_mempool.cs thread safety lock assertions (Jon Atack)
Pull request description:
in src/node/miner to
- BlockAssembler::addPackageTxs()
- BlockAssembler::SkipMapTxEntry()
- BlockAssembler::UpdatePackagesForAdded()
These functions have thread safety lock annotations in their declarations but are missing the corresponding run-time lock assertions in their definitions.
Per doc/developer-notes.md: "Combine annotations in function declarations with run-time asserts in function definitions."
ACKs for top commit:
shaavan:
ACK 275e9390e1c84ac021b3c781ee239ad9ba7b78d4. Thanks for catching and fixing this!
Tree-SHA512: 1c6f1ad1bbd94ff391fc8ce1e3b95d88bd3db5db804a1a5ef4636e54b29f5801f79aa9ed753d34c9a79a58cf01c7ed890e7681ff1c7b0f16335dc062bbac31cc
Diffstat (limited to 'doc')
0 files changed, 0 insertions, 0 deletions