diff options
author | bubnikv <bubnikv@gmail.com> | 2020-01-03 18:32:56 +0300 |
---|---|---|
committer | bubnikv <bubnikv@gmail.com> | 2020-01-03 18:33:04 +0300 |
commit | 9406b5044753cff3ed20ec28f6c5b6067e6fd5a3 (patch) | |
tree | 6ef6723a890c3baed35df058f0acbddced154919 /src/slic3r/GUI/PresetBundle.cpp | |
parent | 30f7a2b8e5b369319ada02455d5fec7633c46b8b (diff) |
Const correctness improvements:
removed some unnecessary const_casts that remove const.
Diffstat (limited to 'src/slic3r/GUI/PresetBundle.cpp')
-rw-r--r-- | src/slic3r/GUI/PresetBundle.cpp | 8 |
1 files changed, 5 insertions, 3 deletions
diff --git a/src/slic3r/GUI/PresetBundle.cpp b/src/slic3r/GUI/PresetBundle.cpp index 3ede7a4e0..219d7b89d 100644 --- a/src/slic3r/GUI/PresetBundle.cpp +++ b/src/slic3r/GUI/PresetBundle.cpp @@ -560,9 +560,11 @@ DynamicPrintConfig PresetBundle::full_fff_config() const while (filament_configs.size() < num_extruders) filament_configs.emplace_back(&this->filaments.first_visible().config); for (const DynamicPrintConfig *cfg : filament_configs) { - compatible_printers_condition.emplace_back(Preset::compatible_printers_condition(*const_cast<DynamicPrintConfig*>(cfg))); - compatible_prints_condition .emplace_back(Preset::compatible_prints_condition(*const_cast<DynamicPrintConfig*>(cfg))); - inherits .emplace_back(Preset::inherits(*const_cast<DynamicPrintConfig*>(cfg))); + // The compatible_prints/printers_condition() returns a reference to configuration key, which may not yet exist. + DynamicPrintConfig &cfg_rw = *const_cast<DynamicPrintConfig*>(cfg); + compatible_printers_condition.emplace_back(Preset::compatible_printers_condition(cfg_rw)); + compatible_prints_condition .emplace_back(Preset::compatible_prints_condition(cfg_rw)); + inherits .emplace_back(Preset::inherits(cfg_rw)); } // Option values to set a ConfigOptionVector from. std::vector<const ConfigOption*> filament_opts(num_extruders, nullptr); |