HomePhabricator

Move some static functions out of wallet.h/cpp

Description

Move some static functions out of wallet.h/cpp

Summary:
Backport of Core PR10976

This change is desirable because wallet.h/cpp are monolithic and hard to
navigate, so pulling things out and grouping together pieces of related
functionality should improve the organization.

Another proximate motivation is the wallet process separation work in
https://github.com/bitcoin/bitcoin/pull/10973, where (at least initially)
parameter parsing and fee estimation are still done in the main process rather
than the wallet process, and having functions that run in different processes
scrambled up throughout wallet.cpp is unnecessarily confusing.

Test Plan:

make check

Reviewers: #bitcoin_abc, deadalnix, jasonbcox

Reviewed By: #bitcoin_abc, deadalnix, jasonbcox

Subscribers: jasonbcox, deadalnix, teamcity

Differential Revision: https://reviews.bitcoinabc.org/D1785

Details

Provenance
Russell Yanofsky <russ@yanofsky.org>Authored on Sep 27 2018, 14:31
schancelCommitted on Sep 27 2018, 14:31
schancelPushed on Sep 27 2018, 15:24
Reviewer
Restricted Project
Differential Revision
D1785: Move some static functions out of wallet.h/cpp
Parents
rSTAGING1c79e0a3d0cc: [cmake] Set the proper flags for Qt mock when the wallet is enabled.
Branches
Unknown
Tags
Unknown
References
tag: phabricator/base/5144, tag: phabricator/base/5143, tag: phabricator/base/5114