HomePhabricator

remove CChainParams from wallet methods that don't need it

Description

remove CChainParams from wallet methods that don't need it

Summary:
While working on CWallet, I noticed that it has methods that take a CChainParams as an argument but never use it. I removed those arguments. There is no other change than removing such parameters from function definitions and call-sites.

Now, the only remaining mention of CChainParams in those files is mostly related to CWallet::GetChainParams, which forwards the chain parameters from CWallet::m_chain

This is a follow-up to D7041.

Test Plan: ninja all check-all

Reviewers: #bitcoin_abc, deadalnix

Reviewed By: #bitcoin_abc, deadalnix

Subscribers: deadalnix

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

Details

Provenance
PiRKAuthored on Sep 23 2021, 11:53
PiRKPushed on Sep 27 2021, 06:13
Reviewer
Restricted Project
Differential Revision
D10188: remove CChainParams from wallet methods that don't need it
Parents
rABCdf5b53eeee60: wallet: Avoid multiple BerkeleyBatch in DelAddressBook
Branches
Unknown
Tags
Unknown