Page MenuHomePhabricator

Remove redundant locks
ClosedPublic

Authored by nakihito on Mar 18 2019, 20:20.

Details

Reviewers
jasonbcox
deadalnix
Fabien
Group Reviewers
Restricted Owners Package(Owns No Changed Paths)
Restricted Project
Commits
rSTAGINGbf97bcfb8254: Remove redundant locks
rABCbf97bcfb8254: Remove redundant locks
Summary
  • SetAddressBook(...) is locking cs_wallet internally
  • DelAddressBook(...) is locking cs_wallet internally

Completes T557

Backport of PR11733
https://github.com/bitcoin/bitcoin/pull/11733

Test Plan

make check
test_runner.py

Diff Detail

Repository
rABC Bitcoin ABC
Branch
BackportPR11733
Lint
Lint Passed
Unit
No Test Coverage
Build Status
Buildable 5234
Build 8531: Bitcoin ABC Buildbot (legacy)
Build 8530: arc lint + arc unit

Event Timeline

Owners added a reviewer: Restricted Owners Package.Mar 18 2019, 20:20
This revision is now accepted and ready to land.Mar 18 2019, 20:36
This revision was automatically updated to reflect the committed changes.