Unverified Commit 817c6e01 authored by Dominik Charousset's avatar Dominik Charousset Committed by GitHub

Apply suggestions from code review

Co-authored-by: default avatarJoseph Noir <josephnoir@users.noreply.github.com>
parent 49ac3643
...@@ -74,7 +74,7 @@ private: ...@@ -74,7 +74,7 @@ private:
config_value::list xs_; config_value::list xs_;
}; };
/// Consumes a series of key-vale pairs from an application configuration. /// Consumes a series of key-value pairs from an application configuration.
class CAF_CORE_EXPORT config_consumer { class CAF_CORE_EXPORT config_consumer {
public: public:
// -- constructors, destructors, and assignment operators -------------------- // -- constructors, destructors, and assignment operators --------------------
......
...@@ -352,7 +352,7 @@ error actor_system_config::parse(string_list args, ...@@ -352,7 +352,7 @@ error actor_system_config::parse(string_list args,
if (!conf.is_open()) { if (!conf.is_open()) {
conf.open("caf-application.ini"); conf.open("caf-application.ini");
if (conf.is_open()) { if (conf.is_open()) {
// Consume the ini file here, because the parse() overloead for taking // Consume the ini file here, because the parse() overload that takes
// an istream assumes the new config format. // an istream assumes the new config format.
if (auto err = parse_ini(conf, custom_options_, content)) if (auto err = parse_ini(conf, custom_options_, content))
return err; return err;
......
...@@ -156,7 +156,7 @@ void merge_into_place(settings& src, settings& dst) { ...@@ -156,7 +156,7 @@ void merge_into_place(settings& src, settings& dst) {
} // namespace } // namespace
pec config_consumer::value_impl(config_value&& x) { pec config_consumer::value_impl(config_value&& x) {
// See whether there's an config_option associated to this key and perform a // See whether there's a config_option associated to this key and perform a
// type check if necessary. // type check if necessary.
const config_option* opt; const config_option* opt;
if (options_ == nullptr) { if (options_ == nullptr) {
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment