Page MenuHomePhabricator

[refactor] replace global Params() use with CWallet chainParams member
AbandonedPublic

Authored by majcosta on Oct 2 2020, 14:24.

Details

Reviewers
deadalnix
Group Reviewers
Restricted Project
Summary

as suggested in D7041. next diff replaces the member with a method that gets CChainParams from interfaces::Chain (where available)

Test Plan
ninja all check check-functional

Diff Detail

Repository
rABC Bitcoin ABC
Branch
replace_params_with_cwallet_params
Lint
Lint Passed
Unit
No Test Coverage
Build Status
Buildable 13160
Build 26370: Build Diffbuild-diff · build-without-wallet · lint-circular-dependencies · build-clang · build-clang-tidy
Build 26369: arc lint + arc unit

Event Timeline

majcosta requested review of this revision.Oct 2 2020, 14:24
deadalnix requested changes to this revision.Oct 2 2020, 14:37
deadalnix added a subscriber: deadalnix.
deadalnix added inline comments.
src/wallet/wallet.cpp
2302

Same problem as D7736 . Also, why a new revision? That makes review harder.

This revision now requires changes to proceed.Oct 2 2020, 14:37

Apologies, I have no idea why there's a new revision. I'll just arc diff --update on the other one.