diff options
| author | Wladimir J. van der Laan <[email protected]> | 2018-02-14 17:25:57 +0100 |
|---|---|---|
| committer | Wladimir J. van der Laan <[email protected]> | 2018-02-14 17:26:53 +0100 |
| commit | e782099a151a680523cd8ff5e3fcb9d5cc182bdd (patch) | |
| tree | beec2c41d354e8ca7174e2a9a29d7c3d0c0f2e75 /src/httpserver.cpp | |
| parent | Merge #11866: Do not un-mark fInMempool on wallet txn if ATMP fails. (diff) | |
| parent | Remove redundant locks (diff) | |
| download | discoin-e782099a151a680523cd8ff5e3fcb9d5cc182bdd.tar.xz discoin-e782099a151a680523cd8ff5e3fcb9d5cc182bdd.zip | |
Merge #11733: qt: Remove redundant locks
d6f3a73 Remove redundant locks (practicalswift)
Pull request description:
Remove redundant locks:
* ~~`FindNode(...)` is locking `cs_vNodes` internally~~
* `SetAddressBook(...)` is locking `cs_wallet` internally
* `DelAddressBook(...)` is locking `cs_wallet` internally
**Note to reviewers:** From what I can tell these locks are redundantly held from a data integrity perspective (guarding specific variables), and they do not appear to be needed from a data consistency perspective (ensuring a consistent state at the right points). Review thoroughly and please let me know if I'm mistaken :-)
Tree-SHA512: 7e3ca2d52fecb16385dc65051b5b20d81b502c0025d70b0c489eb3881866bdd57947a9c96931f7b213f5a8a76b6d2c7b084dff0ef2028a1e9ca9ccfd83e5b91e
Diffstat (limited to 'src/httpserver.cpp')
0 files changed, 0 insertions, 0 deletions