diff --git a/xs/src/libslic3r/Config.hpp b/xs/src/libslic3r/Config.hpp index c5b0fb880..06f37a78d 100644 --- a/xs/src/libslic3r/Config.hpp +++ b/xs/src/libslic3r/Config.hpp @@ -26,7 +26,7 @@ extern bool unescape_strings_cstyle(const std::string &str, std::vector class ConfigOptionSingle : public ConfigOption { - public: +public: T value; ConfigOptionSingle(T _value) : value(_value) {}; operator T() const { return this->value; }; @@ -55,7 +55,7 @@ class ConfigOptionSingle : public ConfigOption { // Value of a vector valued option (bools, ints, floats, strings, points) class ConfigOptionVectorBase : public ConfigOption { - public: +public: virtual ~ConfigOptionVectorBase() {}; // Currently used only to initialize the PlaceholderParser. virtual std::vector vserialize() const = 0; @@ -65,7 +65,7 @@ class ConfigOptionVectorBase : public ConfigOption { template class ConfigOptionVector : public ConfigOptionVectorBase { - public: +public: virtual ~ConfigOptionVector() {}; std::vector values; @@ -82,7 +82,7 @@ class ConfigOptionVector : public ConfigOptionVectorBase class ConfigOptionFloat : public ConfigOptionSingle { - public: +public: ConfigOptionFloat() : ConfigOptionSingle(0) {}; ConfigOptionFloat(double _value) : ConfigOptionSingle(_value) {}; @@ -103,8 +103,7 @@ class ConfigOptionFloat : public ConfigOptionSingle class ConfigOptionFloats : public ConfigOptionVector { - public: - +public: std::string serialize() const { std::ostringstream ss; for (std::vector::const_iterator it = this->values.begin(); it != this->values.end(); ++it) { @@ -141,7 +140,7 @@ class ConfigOptionFloats : public ConfigOptionVector class ConfigOptionInt : public ConfigOptionSingle { - public: +public: ConfigOptionInt() : ConfigOptionSingle(0) {}; ConfigOptionInt(double _value) : ConfigOptionSingle(int(floor(_value + 0.5))) {}; @@ -163,8 +162,7 @@ class ConfigOptionInt : public ConfigOptionSingle class ConfigOptionInts : public ConfigOptionVector { - public: - +public: std::string serialize() const { std::ostringstream ss; for (std::vector::const_iterator it = this->values.begin(); it != this->values.end(); ++it) { @@ -201,7 +199,7 @@ class ConfigOptionInts : public ConfigOptionVector class ConfigOptionString : public ConfigOptionSingle { - public: +public: ConfigOptionString() : ConfigOptionSingle("") {}; ConfigOptionString(std::string _value) : ConfigOptionSingle(_value) {}; @@ -217,8 +215,7 @@ class ConfigOptionString : public ConfigOptionSingle // semicolon-separated strings class ConfigOptionStrings : public ConfigOptionVector { - public: - +public: std::string serialize() const { return escape_strings_cstyle(this->values); }; @@ -234,7 +231,7 @@ class ConfigOptionStrings : public ConfigOptionVector class ConfigOptionPercent : public ConfigOptionFloat { - public: +public: ConfigOptionPercent() : ConfigOptionFloat(0) {}; ConfigOptionPercent(double _value) : ConfigOptionFloat(_value) {}; @@ -263,7 +260,7 @@ class ConfigOptionPercents : public ConfigOptionFloats public: std::string serialize() const { std::ostringstream ss; - for (const auto v : this->values) { + for (const auto &v : this->values) { if (&v != &this->values.front()) ss << ","; ss << v << "%"; } @@ -300,7 +297,7 @@ public: class ConfigOptionFloatOrPercent : public ConfigOptionPercent { - public: +public: bool percent; ConfigOptionFloatOrPercent() : ConfigOptionPercent(0), percent(false) {}; ConfigOptionFloatOrPercent(double _value, bool _percent) @@ -340,7 +337,7 @@ class ConfigOptionFloatOrPercent : public ConfigOptionPercent class ConfigOptionPoint : public ConfigOptionSingle { - public: +public: ConfigOptionPoint() : ConfigOptionSingle(Pointf(0,0)) {}; ConfigOptionPoint(Pointf _value) : ConfigOptionSingle(_value) {}; @@ -364,8 +361,7 @@ class ConfigOptionPoint : public ConfigOptionSingle class ConfigOptionPoints : public ConfigOptionVector { - public: - +public: std::string serialize() const { std::ostringstream ss; for (Pointfs::const_iterator it = this->values.begin(); it != this->values.end(); ++it) { @@ -409,7 +405,7 @@ class ConfigOptionPoints : public ConfigOptionVector class ConfigOptionBool : public ConfigOptionSingle { - public: +public: ConfigOptionBool() : ConfigOptionSingle(false) {}; ConfigOptionBool(bool _value) : ConfigOptionSingle(_value) {}; @@ -427,8 +423,7 @@ class ConfigOptionBool : public ConfigOptionSingle class ConfigOptionBools : public ConfigOptionVector { - public: - +public: std::string serialize() const { std::ostringstream ss; for (std::vector::const_iterator it = this->values.begin(); it != this->values.end(); ++it) { @@ -465,7 +460,7 @@ typedef std::map t_config_enum_values; template class ConfigOptionEnum : public ConfigOptionSingle { - public: +public: // by default, use the first value (0) of the T enum type ConfigOptionEnum() : ConfigOptionSingle(static_cast(0)) {}; ConfigOptionEnum(T _value) : ConfigOptionSingle(_value) {}; @@ -495,7 +490,7 @@ class ConfigOptionEnum : public ConfigOptionSingle // In the StaticConfig, it is better to use the specialized ConfigOptionEnum containers. class ConfigOptionEnumGeneric : public ConfigOptionInt { - public: +public: const t_config_enum_values* keys_map; std::string serialize() const { @@ -548,7 +543,7 @@ enum ConfigOptionType { // Definition of a configuration value for the purpose of GUI presentation, editing, value mapping and config file handling. class ConfigOptionDef { - public: +public: // What type? bool, int, string etc. ConfigOptionType type; // Default value of this option. The default value object is owned by ConfigDef, it is released in its destructor.