diff options
| author | MarcoFalke <[email protected]> | 2020-04-16 12:26:01 -0400 |
|---|---|---|
| committer | MarcoFalke <[email protected]> | 2020-04-16 12:26:09 -0400 |
| commit | faf989f93695d29099f6e152d5a2e117cd304183 (patch) | |
| tree | b7b882536225539faabbeaaca1c241f63e9a2c2c /src/util/system.cpp | |
| parent | bench: Remove unused argsman.ClearArgs (diff) | |
| download | discoin-faf989f93695d29099f6e152d5a2e117cd304183.tar.xz discoin-faf989f93695d29099f6e152d5a2e117cd304183.zip | |
util: Document why ArgsManager (con/de)structor is not inline
Diffstat (limited to 'src/util/system.cpp')
| -rw-r--r-- | src/util/system.cpp | 9 |
1 files changed, 5 insertions, 4 deletions
diff --git a/src/util/system.cpp b/src/util/system.cpp index b0a538b52..69a7be96d 100644 --- a/src/util/system.cpp +++ b/src/util/system.cpp @@ -226,10 +226,11 @@ static bool CheckValid(const std::string& key, const util::SettingsValue& val, u return true; } -ArgsManager::ArgsManager() -{ - // nothing to do -} +// Define default constructor and destructor that are not inline, so code instantiating this class doesn't need to +// #include class definitions for all members. +// For example, m_settings has an internal dependency on univalue. +ArgsManager::ArgsManager() {} +ArgsManager::~ArgsManager() {} const std::set<std::string> ArgsManager::GetUnsuitableSectionOnlyArgs() const { |