Page MenuHomePhabricator

Pass config down to InitializeNode
ClosedPublic

Authored by deadalnix on Jun 21 2017, 13:28.

Details

Summary

As per title. Eventually, we'll be able to get rid of these GetConfig calls.

Test Plan
make check

Diff Detail

Repository
rABC Bitcoin ABC
Branch
config
Lint
Lint Passed
Unit
No Test Coverage
Build Status
Buildable 378
Build 378: arc lint + arc unit

Event Timeline

See message, looks good and make check passes.

src/test/DoS_tests.cpp
61

Minor nit of the being anal about it variant: You factor out the config variable here and at the other, similar locations, this isn't really related to passing the config down - so maybe add 'and minor test refactoring' to the commit msg?

This revision is now accepted and ready to land.Jun 22 2017, 15:16
src/test/DoS_tests.cpp
61

This people know already by looking at the code.

This revision was automatically updated to reflect the committed changes.