diff options
author | MarcoFalke <falke.marco@gmail.com> | 2021-05-20 13:46:15 +0200 |
---|---|---|
committer | fanquake <fanquake@gmail.com> | 2021-06-30 10:23:54 +0800 |
commit | 513613d8a87337f1d1f639bc9426165c3b6be62e (patch) | |
tree | 51897fcf61dd00a94d5b78998acafdc202f6026c /src/util/system.cpp | |
parent | 70eac6fcd02b6c44cb4b1f2fb895eae147e3f490 (diff) | |
download | bitcoin-513613d8a87337f1d1f639bc9426165c3b6be62e.tar.xz |
Cleanup -includeconf error message
Remove the erroneous trailing newline '\n'. Also, print only the first
value to remove needless redundancy in the error message.
Github-Pull: #22002
Rebased-From: fad0867d6ab9430070aa7d60bf7617a6508e0586
Diffstat (limited to 'src/util/system.cpp')
-rw-r--r-- | src/util/system.cpp | 10 |
1 files changed, 4 insertions, 6 deletions
diff --git a/src/util/system.cpp b/src/util/system.cpp index 3512bb8bff..a21d58a19d 100644 --- a/src/util/system.cpp +++ b/src/util/system.cpp @@ -337,14 +337,12 @@ bool ArgsManager::ParseParameters(int argc, const char* const argv[], std::strin } // we do not allow -includeconf from command line - bool success = true; if (auto* includes = util::FindKey(m_settings.command_line_options, "includeconf")) { - for (const auto& include : util::SettingsSpan(*includes)) { - error += "-includeconf cannot be used from commandline; -includeconf=" + include.write() + "\n"; - success = false; - } + const auto& include{*util::SettingsSpan(*includes).begin()}; // pick first value as example + error = "-includeconf cannot be used from commandline; -includeconf=" + include.write(); + return false; } - return success; + return true; } Optional<unsigned int> ArgsManager::GetArgFlags(const std::string& name) const |