Suggested wallet code cleanups from #14711
ClosedPublic
Actions

Authored by Fabien on May 4 2020, 19:59.

Details

Reviewers

Group Reviewers

Restricted Project

Commits

rSTAGING7bd5ac0f8c1c: Suggested wallet code cleanups from #14711
rABC7bd5ac0f8c1c: Suggested wallet code cleanups from #14711

Summary

This implements suggested changes from #14711 review comments that
didn't make make it in before merging.

There are no changes in behavior in this PR, just documentation updates,
simplifications, and variable renames.

Backport of core PR15342.

Test Plan

ninja all check check-functional

Diff Detail

Repository

rABC Bitcoin ABC

Lint

Lint Not Applicable

Unit

Tests Not Applicable

Event Timeline

Fabien created this revision.May 4 2020, 19:59

Herald added a reviewer: Restricted Project. · View Herald TranscriptMay 4 2020, 19:59

Fabien requested review of this revision.May 4 2020, 19:59

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

Build Bitcoin-ABC / Diffs / Diff Testing (build-diff) started.

Build Bitcoin-ABC / Diffs / Diff Testing (build-diff) passed.

Harbormaster completed remote builds in B10579: Diff 19418.May 4 2020, 20:09

jasonbcox accepted this revision.May 4 2020, 20:27

jasonbcox added a subscriber: jasonbcox.

jasonbcox added inline comments.

src/wallet/rpcwallet.cpp
4016 ↗	(On Diff #19418)	Note to other reviewers: This essentially skips over PR15279, but it doesn't matter because that PR is only a fixup for this help string. This patch brings this text most up-to-date.