move-only: Move settings to the common library

The background of this commit is an ongoing effort to decouple the
libbitcoinkernel library from code that is not strictly required by it.
The settings code belongs into the common library and namespace, since
the kernel library should not depend on it. See doc/design/libraries.md
for more information on this rationale.

Changing the namespace of the moved functions is scripted in the
following commit.

2 files changed, 0 insertions, 372 deletions

diff --git a/src/util/settings.cpp b/src/util/settings.cpp
deleted file mode 100644
index db3d60046e..0000000000
--- a/src/util/settings.cpp
+++ /dev/null
@@ -1,258 +0,0 @@
-// Copyright (c) 2019-2022 The Bitcoin Core developers
-// Distributed under the MIT software license, see the accompanying
-// file COPYING or http://www.opensource.org/licenses/mit-license.php.
-
-#include <util/fs.h>
-#include <util/settings.h>
-
-#include <tinyformat.h>
-#include <univalue.h>
-
-#include <fstream>
-#include <map>
-#include <string>
-#include <vector>
-
-namespace util {
-namespace {
-
-enum class Source {
-   FORCED,
-   COMMAND_LINE,
-   RW_SETTINGS,
-   CONFIG_FILE_NETWORK_SECTION,
-   CONFIG_FILE_DEFAULT_SECTION
-};
-
-//! Merge settings from multiple sources in precedence order:
-//! Forced config > command line > read-write settings file > config file network-specific section > config file default section
-//!
-//! This function is provided with a callback function fn that contains
-//! specific logic for how to merge the sources.
-template <typename Fn>
-static void MergeSettings(const Settings& settings, const std::string& section, const std::string& name, Fn&& fn)
-{
-    // Merge in the forced settings
-    if (auto* value = FindKey(settings.forced_settings, name)) {
-        fn(SettingsSpan(*value), Source::FORCED);
-    }
-    // Merge in the command-line options
-    if (auto* values = FindKey(settings.command_line_options, name)) {
-        fn(SettingsSpan(*values), Source::COMMAND_LINE);
-    }
-    // Merge in the read-write settings
-    if (const SettingsValue* value = FindKey(settings.rw_settings, name)) {
-        fn(SettingsSpan(*value), Source::RW_SETTINGS);
-    }
-    // Merge in the network-specific section of the config file
-    if (!section.empty()) {
-        if (auto* map = FindKey(settings.ro_config, section)) {
-            if (auto* values = FindKey(*map, name)) {
-                fn(SettingsSpan(*values), Source::CONFIG_FILE_NETWORK_SECTION);
-            }
-        }
-    }
-    // Merge in the default section of the config file
-    if (auto* map = FindKey(settings.ro_config, "")) {
-        if (auto* values = FindKey(*map, name)) {
-            fn(SettingsSpan(*values), Source::CONFIG_FILE_DEFAULT_SECTION);
-        }
-    }
-}
-} // namespace
-
-bool ReadSettings(const fs::path& path, std::map<std::string, SettingsValue>& values, std::vector<std::string>& errors)
-{
-    values.clear();
-    errors.clear();
-
-    // Ok for file to not exist
-    if (!fs::exists(path)) return true;
-
-    std::ifstream file;
-    file.open(path);
-    if (!file.is_open()) {
-      errors.emplace_back(strprintf("%s. Please check permissions.", fs::PathToString(path)));
-      return false;
-    }
-
-    SettingsValue in;
-    if (!in.read(std::string{std::istreambuf_iterator<char>(file), std::istreambuf_iterator<char>()})) {
-        errors.emplace_back(strprintf("Unable to parse settings file %s", fs::PathToString(path)));
-        return false;
-    }
-
-    if (file.fail()) {
-        errors.emplace_back(strprintf("Failed reading settings file %s", fs::PathToString(path)));
-        return false;
-    }
-    file.close(); // Done with file descriptor. Release while copying data.
-
-    if (!in.isObject()) {
-        errors.emplace_back(strprintf("Found non-object value %s in settings file %s", in.write(), fs::PathToString(path)));
-        return false;
-    }
-
-    const std::vector<std::string>& in_keys = in.getKeys();
-    const std::vector<SettingsValue>& in_values = in.getValues();
-    for (size_t i = 0; i < in_keys.size(); ++i) {
-        auto inserted = values.emplace(in_keys[i], in_values[i]);
-        if (!inserted.second) {
-            errors.emplace_back(strprintf("Found duplicate key %s in settings file %s", in_keys[i], fs::PathToString(path)));
-            values.clear();
-            break;
-        }
-    }
-    return errors.empty();
-}
-
-bool WriteSettings(const fs::path& path,
-    const std::map<std::string, SettingsValue>& values,
-    std::vector<std::string>& errors)
-{
-    SettingsValue out(SettingsValue::VOBJ);
-    for (const auto& value : values) {
-        out.__pushKV(value.first, value.second);
-    }
-    std::ofstream file;
-    file.open(path);
-    if (file.fail()) {
-        errors.emplace_back(strprintf("Error: Unable to open settings file %s for writing", fs::PathToString(path)));
-        return false;
-    }
-    file << out.write(/* prettyIndent= */ 4, /* indentLevel= */ 1) << std::endl;
-    file.close();
-    return true;
-}
-
-SettingsValue GetSetting(const Settings& settings,
-    const std::string& section,
-    const std::string& name,
-    bool ignore_default_section_config,
-    bool ignore_nonpersistent,
-    bool get_chain_type)
-{
-    SettingsValue result;
-    bool done = false; // Done merging any more settings sources.
-    MergeSettings(settings, section, name, [&](SettingsSpan span, Source source) {
-        // Weird behavior preserved for backwards compatibility: Apply negated
-        // setting even if non-negated setting would be ignored. A negated
-        // value in the default section is applied to network specific options,
-        // even though normal non-negated values there would be ignored.
-        const bool never_ignore_negated_setting = span.last_negated();
-
-        // Weird behavior preserved for backwards compatibility: Take first
-        // assigned value instead of last. In general, later settings take
-        // precedence over early settings, but for backwards compatibility in
-        // the config file the precedence is reversed for all settings except
-        // chain type settings.
-        const bool reverse_precedence =
-            (source == Source::CONFIG_FILE_NETWORK_SECTION || source == Source::CONFIG_FILE_DEFAULT_SECTION) &&
-            !get_chain_type;
-
-        // Weird behavior preserved for backwards compatibility: Negated
-        // -regtest and -testnet arguments which you would expect to override
-        // values set in the configuration file are currently accepted but
-        // silently ignored. It would be better to apply these just like other
-        // negated values, or at least warn they are ignored.
-        const bool skip_negated_command_line = get_chain_type;
-
-        if (done) return;
-
-        // Ignore settings in default config section if requested.
-        if (ignore_default_section_config && source == Source::CONFIG_FILE_DEFAULT_SECTION &&
-            !never_ignore_negated_setting) {
-            return;
-        }
-
-        // Ignore nonpersistent settings if requested.
-        if (ignore_nonpersistent && (source == Source::COMMAND_LINE || source == Source::FORCED)) return;
-
-        // Skip negated command line settings.
-        if (skip_negated_command_line && span.last_negated()) return;
-
-        if (!span.empty()) {
-            result = reverse_precedence ? span.begin()[0] : span.end()[-1];
-            done = true;
-        } else if (span.last_negated()) {
-            result = false;
-            done = true;
-        }
-    });
-    return result;
-}
-
-std::vector<SettingsValue> GetSettingsList(const Settings& settings,
-    const std::string& section,
-    const std::string& name,
-    bool ignore_default_section_config)
-{
-    std::vector<SettingsValue> result;
-    bool done = false; // Done merging any more settings sources.
-    bool prev_negated_empty = false;
-    MergeSettings(settings, section, name, [&](SettingsSpan span, Source source) {
-        // Weird behavior preserved for backwards compatibility: Apply config
-        // file settings even if negated on command line. Negating a setting on
-        // command line will ignore earlier settings on the command line and
-        // ignore settings in the config file, unless the negated command line
-        // value is followed by non-negated value, in which case config file
-        // settings will be brought back from the dead (but earlier command
-        // line settings will still be ignored).
-        const bool add_zombie_config_values =
-            (source == Source::CONFIG_FILE_NETWORK_SECTION || source == Source::CONFIG_FILE_DEFAULT_SECTION) &&
-            !prev_negated_empty;
-
-        // Ignore settings in default config section if requested.
-        if (ignore_default_section_config && source == Source::CONFIG_FILE_DEFAULT_SECTION) return;
-
-        // Add new settings to the result if isn't already complete, or if the
-        // values are zombies.
-        if (!done || add_zombie_config_values) {
-            for (const auto& value : span) {
-                if (value.isArray()) {
-                    result.insert(result.end(), value.getValues().begin(), value.getValues().end());
-                } else {
-                    result.push_back(value);
-                }
-            }
-        }
-
-        // If a setting was negated, or if a setting was forced, set
-        // done to true to ignore any later lower priority settings.
-        done |= span.negated() > 0 || source == Source::FORCED;
-
-        // Update the negated and empty state used for the zombie values check.
-        prev_negated_empty |= span.last_negated() && result.empty();
-    });
-    return result;
-}
-
-bool OnlyHasDefaultSectionSetting(const Settings& settings, const std::string& section, const std::string& name)
-{
-    bool has_default_section_setting = false;
-    bool has_other_setting = false;
-    MergeSettings(settings, section, name, [&](SettingsSpan span, Source source) {
-        if (span.empty()) return;
-        else if (source == Source::CONFIG_FILE_DEFAULT_SECTION) has_default_section_setting = true;
-        else has_other_setting = true;
-    });
-    // If a value is set in the default section and not explicitly overwritten by the
-    // user on the command line or in a different section, then we want to enable
-    // warnings about the value being ignored.
-    return has_default_section_setting && !has_other_setting;
-}
-
-SettingsSpan::SettingsSpan(const std::vector<SettingsValue>& vec) noexcept : SettingsSpan(vec.data(), vec.size()) {}
-const SettingsValue* SettingsSpan::begin() const { return data + negated(); }
-const SettingsValue* SettingsSpan::end() const { return data + size; }
-bool SettingsSpan::empty() const { return size == 0 || last_negated(); }
-bool SettingsSpan::last_negated() const { return size > 0 && data[size - 1].isFalse(); }
-size_t SettingsSpan::negated() const
-{
-    for (size_t i = size; i > 0; --i) {
-        if (data[i - 1].isFalse()) return i; // Return number of negated values (position of last false value)
-    }
-    return 0;
-}
-
-} // namespace util
diff --git a/src/util/settings.h b/src/util/settings.h
deleted file mode 100644
index bb1fe585e1..0000000000
--- a/src/util/settings.h
+++ /dev/null
@@ -1,114 +0,0 @@
-// Copyright (c) 2019-2022 The Bitcoin Core developers
-// Distributed under the MIT software license, see the accompanying
-// file COPYING or http://www.opensource.org/licenses/mit-license.php.
-
-#ifndef BITCOIN_UTIL_SETTINGS_H
-#define BITCOIN_UTIL_SETTINGS_H
-
-#include <util/fs.h>
-
-#include <map>
-#include <string>
-#include <vector>
-
-class UniValue;
-
-namespace util {
-
-//! Settings value type (string/integer/boolean/null variant).
-//!
-//! @note UniValue is used here for convenience and because it can be easily
-//!       serialized in a readable format. But any other variant type that can
-//!       be assigned strings, int64_t, and bool values and has get_str(),
-//!       getInt<int64_t>(), get_bool(), isNum(), isBool(), isFalse(), isTrue() and
-//!       isNull() methods can be substituted if there's a need to move away
-//!       from UniValue. (An implementation with boost::variant was posted at
-//!       https://github.com/bitcoin/bitcoin/pull/15934/files#r337691812)
-using SettingsValue = UniValue;
-
-//! Stored settings. This struct combines settings from the command line, a
-//! read-only configuration file, and a read-write runtime settings file.
-struct Settings {
-    //! Map of setting name to forced setting value.
-    std::map<std::string, SettingsValue> forced_settings;
-    //! Map of setting name to list of command line values.
-    std::map<std::string, std::vector<SettingsValue>> command_line_options;
-    //! Map of setting name to read-write file setting value.
-    std::map<std::string, SettingsValue> rw_settings;
-    //! Map of config section name and setting name to list of config file values.
-    std::map<std::string, std::map<std::string, std::vector<SettingsValue>>> ro_config;
-};
-
-//! Read settings file.
-bool ReadSettings(const fs::path& path,
-    std::map<std::string, SettingsValue>& values,
-    std::vector<std::string>& errors);
-
-//! Write settings file.
-bool WriteSettings(const fs::path& path,
-    const std::map<std::string, SettingsValue>& values,
-    std::vector<std::string>& errors);
-
-//! Get settings value from combined sources: forced settings, command line
-//! arguments, runtime read-write settings, and the read-only config file.
-//!
-//! @param ignore_default_section_config - ignore values in the default section
-//!                                        of the config file (part before any
-//!                                        [section] keywords)
-//! @param ignore_nonpersistent - ignore non-persistent settings values (forced
-//!                               settings values and values specified on the
-//!                               command line). Only return settings in the
-//!                               read-only config and read-write settings
-//!                               files.
-//! @param get_chain_type - enable special backwards compatible behavior
-//!                         for GetChainType
-SettingsValue GetSetting(const Settings& settings,
-    const std::string& section,
-    const std::string& name,
-    bool ignore_default_section_config,
-    bool ignore_nonpersistent,
-    bool get_chain_type);
-
-//! Get combined setting value similar to GetSetting(), except if setting was
-//! specified multiple times, return a list of all the values specified.
-std::vector<SettingsValue> GetSettingsList(const Settings& settings,
-    const std::string& section,
-    const std::string& name,
-    bool ignore_default_section_config);
-
-//! Return true if a setting is set in the default config file section, and not
-//! overridden by a higher priority command-line or network section value.
-//!
-//! This is used to provide user warnings about values that might be getting
-//! ignored unintentionally.
-bool OnlyHasDefaultSectionSetting(const Settings& settings, const std::string& section, const std::string& name);
-
-//! Accessor for list of settings that skips negated values when iterated over.
-//! The last boolean `false` value in the list and all earlier values are
-//! considered negated.
-struct SettingsSpan {
-    explicit SettingsSpan() = default;
-    explicit SettingsSpan(const SettingsValue& value) noexcept : SettingsSpan(&value, 1) {}
-    explicit SettingsSpan(const SettingsValue* data, size_t size) noexcept : data(data), size(size) {}
-    explicit SettingsSpan(const std::vector<SettingsValue>& vec) noexcept;
-    const SettingsValue* begin() const; //!< Pointer to first non-negated value.
-    const SettingsValue* end() const;   //!< Pointer to end of values.
-    bool empty() const;                 //!< True if there are any non-negated values.
-    bool last_negated() const;          //!< True if the last value is negated.
-    size_t negated() const;             //!< Number of negated values.
-
-    const SettingsValue* data = nullptr;
-    size_t size = 0;
-};
-
-//! Map lookup helper.
-template <typename Map, typename Key>
-auto FindKey(Map&& map, Key&& key) -> decltype(&map.at(key))
-{
-    auto it = map.find(key);
-    return it == map.end() ? nullptr : &it->second;
-}
-
-} // namespace util
-
-#endif // BITCOIN_UTIL_SETTINGS_H