diff options
| author | Russell Yanofsky <[email protected]> | 2019-04-22 18:08:51 -0400 |
|---|---|---|
| committer | Russell Yanofsky <[email protected]> | 2019-11-07 23:08:22 -0400 |
| commit | 7f40528cd50fc43ac0bd3e785de24d661adddb7a (patch) | |
| tree | a687d73e4cc320c7c4dee5dd96b1f99ce21c2122 /src/httpserver.cpp | |
| parent | Add util::Settings struct and helper functions. (diff) | |
| download | discoin-7f40528cd50fc43ac0bd3e785de24d661adddb7a.tar.xz discoin-7f40528cd50fc43ac0bd3e785de24d661adddb7a.zip | |
Deduplicate settings merge code
Get rid of settings merging code in util/system.cpp repeated 5 places,
inconsistently:
- ArgsManagerHelper::GetArg
- ArgsManagerHelper::GetNetBoolArg
- ArgsManager::GetArgs
- ArgsManager::IsArgNegated
- ArgsManager::GetUnsuitableSectionOnlyArgs
Having settings merging code separated from parsing simplifies parsing somewhat
(for example negated values can simply be represented as false values instead
of partially cleared or emply placeholder lists).
Having settings merge happen one place instead of 5 makes it easier to add new
settings sources and harder to introduce new inconsistencies in the way
settings are merged.
This commit does not change behavior in any way.
Diffstat (limited to 'src/httpserver.cpp')
0 files changed, 0 insertions, 0 deletions