diff options
author | David Crocker <dcrocker@eschertech.com> | 2021-06-29 12:58:57 +0300 |
---|---|---|
committer | David Crocker <dcrocker@eschertech.com> | 2021-06-29 12:58:57 +0300 |
commit | 9298f2a4e52d8b011b19331041455a45d57678b6 (patch) | |
tree | e60a89f4ef6ed4a75c79adc49250602a386e169a /src/FilamentMonitors | |
parent | 60a2067eaac8fa64473c3068caaca5e7902817d7 (diff) |
Refactored swap-and-delete operations
Diffstat (limited to 'src/FilamentMonitors')
-rw-r--r-- | src/FilamentMonitors/FilamentMonitor.cpp | 12 |
1 files changed, 3 insertions, 9 deletions
diff --git a/src/FilamentMonitors/FilamentMonitor.cpp b/src/FilamentMonitors/FilamentMonitor.cpp index dd39bbcb..cf2ab21a 100644 --- a/src/FilamentMonitors/FilamentMonitor.cpp +++ b/src/FilamentMonitors/FilamentMonitor.cpp @@ -139,9 +139,7 @@ bool FilamentMonitor::IsValid() const noexcept if (sensor != nullptr) { sensor->Disable(); - sensor = nullptr; - std::swap(sensor, filamentSensors[extruder]); - delete sensor; + DeleteObject(filamentSensors[extruder]); reprap.SensorsUpdated(); } @@ -350,9 +348,7 @@ bool FilamentMonitor::IsValid() const noexcept for (FilamentMonitor *&f : filamentSensors) { - FilamentMonitor *temp; - std::swap(temp, f); - delete temp; + DeleteObject(f); } } @@ -387,9 +383,7 @@ bool FilamentMonitor::IsValid() const noexcept { reply.lcatf("Filament monitor for extruder %u has been deleted due to configuration change", extruder); warn = true; - FilamentMonitor *f = nullptr; - std::swap(f, filamentSensors[extruder]); - delete f; + DeleteObject(filamentSensors[extruder]); } } return warn; |