Get rid of BindWallet
ClosedPublic
Actions

Authored by PiRK on Jan 27 2021, 10:17.

Details

Reviewers

majcosta

Group Reviewers

Restricted Project

Commits

rABC846f417721ee: Get rid of BindWallet

Summary

CWalletTx initialization has been fixed so it's no longer necessary to change
which wallet a transaction is bound to.

This concludes backport of Core PR9381 [5/5]
https://github.com/bitcoin/bitcoin/pull/9381/commits/28b112e9bd3fd1181c0720306051ba7efca8b436
Depends on D9077

Test Plan

ninja all check-all

Diff Detail

Repository

rABC Bitcoin ABC

Lint

Lint Not Applicable

Unit

Tests Not Applicable

Event Timeline

PiRK created this revision.Jan 27 2021, 10:17

Herald added a reviewer: Restricted Project. · View Herald TranscriptJan 27 2021, 10:17

PiRK requested review of this revision.Jan 27 2021, 10:17

teamcity edited the summary of this revision. (Show Details)Jan 27 2021, 10:17

Harbormaster completed remote builds in B15010: Diff 27354.Jan 27 2021, 10:48

majcosta accepted this revision.Jan 27 2021, 15:07

This revision is now accepted and ready to land.Jan 27 2021, 15:07

Closed by commit rABC846f417721ee: Get rid of BindWallet (authored by Russell Yanofsky <russ@yanofsky.org>, committed by PiRK). · Explain WhyJan 27 2021, 17:17

This revision was automatically updated to reflect the committed changes.

PiRK added a commit: rABC846f417721ee: Get rid of BindWallet.

Revision Contents
Changeset List

Path

Size

src/

wallet/

wallet.h

5 lines

wallet.cpp

3 lines

Diff 27370

View Options

src/wallet/wallet.h