[wallet] Remove duplicate mapWallet lookups
ClosedPublic
Actions

Authored by jasonbcox on Jan 15 2019, 21:51.

Details

Reviewers

deadalnix
Fabien

Group Reviewers

Restricted Project

Commits

rSTAGING33e101d845ef: [wallet] Remove duplicate mapWallet lookups
rABC33e101d845ef: [wallet] Remove duplicate mapWallet lookups

Summary

Backport of Core PR12276
Completes T518

Test Plan

ninja check
bitcoin-qt

Wallet loads as expected.

Diff Detail

Repository

rABC Bitcoin ABC

Branch

pr12276

Lint

Lint Passed

Unit

No Test Coverage

Build Status

Buildable 4535
Build 7133: Bitcoin ABC Buildbot (legacy)
Build 7132: arc lint + arc unit

Event Timeline

jasonbcox created this revision.Jan 15 2019, 21:51

Herald added a reviewer: Restricted Project. · View Herald TranscriptJan 15 2019, 21:51

Herald added a subscriber: schancel. · View Herald Transcript

jasonbcox edited the summary of this revision. (Show Details)Jan 15 2019, 21:51

Build Bitcoin-ABC / Diffs / Bitcoin-ABC Diff Testing started.

Build Bitcoin-ABC / Diffs / Bitcoin-ABC Diff Testing passed.

Harbormaster completed remote builds in B4535: Diff 6683.Jan 15 2019, 22:01

Fabien accepted this revision.Jan 17 2019, 15:17

This revision is now accepted and ready to land.Jan 17 2019, 15:17

deadalnix edited the summary of this revision. (Show Details)Jan 17 2019, 17:03

Closed by commit rABC33e101d845ef: [wallet] Remove duplicate mapWallet lookups (authored by João Barbosa <joao.paulo.barbosa@gmail.com>, committed by jasonbcox). · Explain WhyJan 17 2019, 17:15

This revision was automatically updated to reflect the committed changes.

Revision Contents
Changeset List

Path

Size

src/

wallet/

wallet.cpp

13 lines

Diff 6683

View Options

[wallet] Remove duplicate mapWallet lookupsClosedPublicActions