diff options
| author | Wladimir J. van der Laan <laanwj@gmail.com> | 2017-08-25 21:01:51 +0200 |
|---|---|---|
| committer | Wladimir J. van der Laan <laanwj@gmail.com> | 2017-08-25 21:30:42 +0200 |
| commit | 07c92b98e2a0a29d19b5ccdbbfa9addc1e80b306 (patch) | |
| tree | 9e6afca5e1454189a68b3066acb9d2fd14d54c5a /test/functional/getblocktemplate_longpoll.py | |
| parent | 3f726c99f819f97f2ab21b94d34c6b3129cd883a (diff) | |
| parent | f01103c1e0a204fc7f40a06755f6c3adb5480cf8 (diff) | |
Merge #10976: [MOVEONLY] Move some static functions out of wallet.h/cpp
f01103c MOVEONLY: Init functions wallet/wallet.cpp -> wallet/init.cpp (Russell Yanofsky)
e7fe320 MOVEONLY: Fee functions wallet/wallet.cpp -> wallet/fees.cpp (Russell Yanofsky)
d97fe20 Move some static functions out of wallet.h/cpp (Russell Yanofsky)
Pull request description:
This just moves some static wallet fee and init functions out of `wallet/wallet.cpp` and into new `wallet/fees.cpp` and `wallet/init.cpp` source files. There is one commit updating declarations and callers, followed by two MOVEONLY commits actually moving the function bodies.
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 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.
Tree-SHA512: 6e6982ff82b2ab4e681c043907e2b1801ceb9513394730070f16c46ad338278a863f5b3759aa13db76a259b268b1c919c81f4e339f0796a3cfb990161e8c316d
Diffstat (limited to 'test/functional/getblocktemplate_longpoll.py')
0 files changed, 0 insertions, 0 deletions