Differential D6413

Have importpubkey use CWallet's ImportScriptPubKeys and ImportPubKeys functions
ClosedPublic
Actions

Authored by deadalnix on Jun 6 2020, 17:56.

Tags

None

Subscribers

None

Details

Reviewers

Group Reviewers

Restricted Project

Commits

rABCf5c67f45cc70: Have importpubkey use CWallet's ImportScriptPubKeys and ImportPubKeys functions

Summary

Behavior changes:

If any scripts for the pubkey were already in the wallet, their timestamps will be set to 1 and label updated

This is a partial backport of Core PR16301 : https://github.com/bitcoin/bitcoin/pull/16301/commits/a00d1e5ec5eb019f8bbeb060a2b09e341d360fe5

Depends on D6410

Test Plan

ninja all check-all

Diff Detail

Repository

rABC Bitcoin ABC

Lint

Lint Not Applicable

Unit

Tests Not Applicable

Event Timeline

deadalnix created this revision.Jun 6 2020, 17:56

Herald added a reviewer: Restricted Project. · View Herald TranscriptJun 6 2020, 17:56

deadalnix requested review of this revision.Jun 6 2020, 17:56

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

deadalnix added a child revision: D6414: Have importaddress use ImportScripts and ImportScriptPubKeys.Jun 6 2020, 18:01

Harbormaster completed remote builds in B11262: Diff 20998.Jun 6 2020, 18:06

majcosta accepted this revision.Jun 7 2020, 21:00

This revision is now accepted and ready to land.Jun 7 2020, 21:00

Closed by commit rABCf5c67f45cc70: Have importpubkey use CWallet's ImportScriptPubKeys and ImportPubKeys functions (authored by Andrew Chow <achow101-github@achow101.com>, committed by deadalnix). · Explain WhyJun 7 2020, 21:40

This revision was automatically updated to reflect the committed changes.

deadalnix added a commit: rABCf5c67f45cc70: Have importpubkey use CWallet's ImportScriptPubKeys and ImportPubKeys functions.

Revision Contents
Changeset List

Path

Size

src/

wallet/

15 lines

Diff 21060

src/wallet/rpcdump.cpp

Loading...