diff options
author | MarcoFalke <falke.marco@gmail.com> | 2021-02-17 09:53:14 +0100 |
---|---|---|
committer | MarcoFalke <falke.marco@gmail.com> | 2021-02-17 09:53:29 +0100 |
commit | 8639c446d88b5d521485dbec00029b3028d01492 (patch) | |
tree | 594c14f3ffe7f3a94c07be0a44268a6cebc970a9 /src/bitcoin-cli.cpp | |
parent | 7f831346cb739e4831692a61332bbaec399a41bc (diff) | |
parent | fafddfadda0c77876ba764c5b65ee5fa8e53a5e0 (diff) | |
download | bitcoin-8639c446d88b5d521485dbec00029b3028d01492.tar.xz |
Merge #21188: scripted-diff: Remove redundant lock annotations in net processing
fafddfadda0c77876ba764c5b65ee5fa8e53a5e0 scripted-diff: Remove shadowing lock annotations (MarcoFalke)
Pull request description:
Would be good to not redundantly copy the lock annotation from the class declaration to the member implementation. Otherwise it may not result in a compile failure if a new lock requirement is added to the member implementation, but not the class declaration.
ACKs for top commit:
amitiuttarwar:
ACK `fafddfadda`, confirmed that the annotations removed were all redundant. confirmed the claim of potential issue :)
hebasto:
ACK fafddfadda0c77876ba764c5b65ee5fa8e53a5e0
jonatack:
Light utACK fafddfadda0c77876ba764c5b65ee5fa8e53a5e0 verified that the removed annotations in the definitions correspond to those in their respective declarations
Tree-SHA512: ea095c6d4e0bedd70d4e2d8a42b06cfd90c161ebfcaac13558c5dc065601a732e5f812f332104b7daa087aa57b8b0242b177799d22eef7628d77d4d87f443bf2
Diffstat (limited to 'src/bitcoin-cli.cpp')
0 files changed, 0 insertions, 0 deletions