Page MenuHomePhabricator
Differential D7529

wallet: Tidy CWallet::SetUsedDestinationState
ClosedPublic
Actions

Authored by deadalnix on Sep 22 2020, 22:26.

Details

Reviewers
majcosta
Group Reviewers
Restricted Project
Commits
rABC0b16c6b94811: wallet: Tidy CWallet::SetUsedDestinationState
Summary
  • wallet: Avoid recursive lock in CWallet::SetUsedDestinationState
  • wallet: Reuse existing batch in CWallet::SetUsedDestinationState

This is a backport of Core PR17354

Test Plan
ninja all check-all

Diff Detail

Event Timeline

deadalnix created this revision.Sep 22 2020, 22:26
Herald added a reviewer: Restricted Project. · View Herald TranscriptSep 22 2020, 22:26
deadalnix requested review of this revision.Sep 22 2020, 22:26
teamcity edited the summary of this revision. (Show Details)Sep 22 2020, 22:26

[Bot Message]
One or more PR numbers were detected in the summary.
Links to those PRs have been inserted into the summary for reference.

Harbormaster completed remote builds in B12867: Diff 23697.Sep 22 2020, 22:33
majcosta accepted this revision.Sep 22 2020, 23:31
This revision is now accepted and ready to land.Sep 22 2020, 23:31
Closed by commit rABC0b16c6b94811: wallet: Tidy CWallet::SetUsedDestinationState (authored by João Barbosa <joao.paulo.barbosa@gmail.com>, committed by deadalnix). · Explain WhySep 22 2020, 23:40
This revision was automatically updated to reflect the committed changes.
deadalnix added a commit: rABC0b16c6b94811: wallet: Tidy CWallet::SetUsedDestinationState.

Revision Contents
Changeset List

PathSize
src/
interfaces/
6 lines
wallet/
test/
7 lines
11 lines
24 lines

Diff 23697

View Options

src/interfaces/wallet.cpp

Loading...
View Options

src/wallet/test/wallet_tests.cpp

Loading...
View Options

src/wallet/wallet.h

Loading...
View Options

src/wallet/wallet.cpp

Loading...