Reduce cs_main lock in listunspent and refactor replacing three mapAddressBook lookups with one.
Backport of Core PR12639
https://github.com/bitcoin/bitcoin/pull/12639/
Differential D4516
Merge #12639: Reduce cs_main lock and avoid extra lookups of mapAddressBook in listunspent RPC nakihito on Nov 22 2019, 23:24. Authored by Tags None Subscribers None
Details
Reduce cs_main lock in listunspent and refactor replacing three mapAddressBook lookups with one. Backport of Core PR12639 ../configure --enable-debug make check test_runner.py
Diff Detail
Event TimelineComment Actions The lock scope change currently conflicts with the AssertLockHeld(cs_main) added in D2980. I'm currently investing where the lock is not properly being applied. I figured I could do the two changes separately since they are pretty much independent changes. Edit: D4542 actually shows the call site where cs_main is not being locked. |