diff options
| author | fanquake <fanquake@gmail.com> | 2022-05-19 16:14:52 +0100 |
|---|---|---|
| committer | fanquake <fanquake@gmail.com> | 2022-05-19 16:32:56 +0100 |
| commit | 0de36941eca1bff91420dd878eb097db2b1a596c (patch) | |
| tree | 610d1378db4c5f139fc46999c3827498e3c81f59 /src/util/system.cpp | |
| parent | e18fd4763e77d1e19208effa9f1a08c5b29fea8e (diff) | |
| parent | fa9af218780b7960d756db80c57222e5bf2137b1 (diff) | |
| download | bitcoin-0de36941eca1bff91420dd878eb097db2b1a596c.tar.xz | |
Merge bitcoin/bitcoin#25153: scripted-diff: Use getInt<T> over get_int/get_int64
fa9af218780b7960d756db80c57222e5bf2137b1 scripted-diff: Use getInt<T> over get_int/get_int64 (MacroFake)
Pull request description:
Seems better to see the return type directly and be able to modify it easier, as the return type is used for exceptions (in-range checking and parsing feedback).
ACKs for top commit:
fanquake:
ACK fa9af218780b7960d756db80c57222e5bf2137b1
Tree-SHA512: 284aa2527d0f663ca01550115025c9c64c787531d595f866c718f6ad09b9b0cac1e683a7d77f8009b75de990fd37166b44063ffa83fba8a04e9a31600b4c2725
Diffstat (limited to 'src/util/system.cpp')
| -rw-r--r-- | src/util/system.cpp | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/src/util/system.cpp b/src/util/system.cpp index 7697991c4b..cbe9d2ceb0 100644 --- a/src/util/system.cpp +++ b/src/util/system.cpp @@ -606,7 +606,7 @@ std::string ArgsManager::GetArg(const std::string& strArg, const std::string& st int64_t ArgsManager::GetIntArg(const std::string& strArg, int64_t nDefault) const { const util::SettingsValue value = GetSetting(strArg); - return value.isNull() ? nDefault : value.isFalse() ? 0 : value.isTrue() ? 1 : value.isNum() ? value.get_int64() : LocaleIndependentAtoi<int64_t>(value.get_str()); + return value.isNull() ? nDefault : value.isFalse() ? 0 : value.isTrue() ? 1 : value.isNum() ? value.getInt<int64_t>() : LocaleIndependentAtoi<int64_t>(value.get_str()); } bool ArgsManager::GetBoolArg(const std::string& strArg, bool fDefault) const |