FitParameterItem: ptrs --> unique ptrs

GUI/Model/Job/FitParameterItem.cpp

@@ -227,10 +227,10 @@ const double range_factor = 0.5;
 
 FitParameterItem::FitParameterItem(QObject* parent)
     : QObject(parent)
-    , m_typeItem(new FitTypeItem(fitParameterTypeCombo(), this))
-    , m_startValueItem(new FitDoubleItem(0.0, this))
-    , m_minItem(new FitEditableDoubleItem(0.0, true, this))
-    , m_maxItem(new FitEditableDoubleItem(0.0, false, this))
+    , m_typeItem(std::make_unique<FitTypeItem>(fitParameterTypeCombo(), this))
+    , m_startValueItem(std::make_unique<FitDoubleItem>(0.0, this))
+    , m_minItem(std::make_unique<FitEditableDoubleItem>(0.0, true, this))
+    , m_maxItem(std::make_unique<FitEditableDoubleItem>(0.0, false, this))
 {
     setObjectName("FitParameter");
     m_startValueItem->setObjectName("Value");
@@ -320,7 +320,7 @@ void FitParameterItem::setStartValue(double start_value)
 
 QObject* FitParameterItem::startValueItem() const
 {
-    return m_startValueItem;
+    return m_startValueItem.get();
 }
 
 double FitParameterItem::minimum() const
@@ -335,7 +335,7 @@ void FitParameterItem::setMinimum(double minimum)
 
 QObject* FitParameterItem::minimumItem() const
 {
-    return m_minItem;
+    return m_minItem.get();
 }
 
 double FitParameterItem::maximum() const
@@ -350,7 +350,7 @@ void FitParameterItem::setMaximum(double maximum)
 
 QObject* FitParameterItem::maximumItem() const
 {
-    return m_maxItem;
+    return m_maxItem.get();
 }
 
 void FitParameterItem::addLink(const QString& title, const QString& link)
@@ -399,7 +399,7 @@ QString FitParameterItem::currentType() const
 
 QObject* FitParameterItem::typeItem() const
 {
-    return m_typeItem;
+    return m_typeItem.get();
 }
 
 void FitParameterItem::writeTo(QXmlStreamWriter* writer) const
@@ -480,28 +480,28 @@ void FitParameterItem::readFrom(QXmlStreamReader* reader)
 void FitParameterItem::onTypeChange()
 {
     if (isFixed()) {
-        setLimitEnabled(m_minItem, false);
-        setLimitEnabled(m_maxItem, false);
+        setLimitEnabled(m_minItem.get(), false);
+        setLimitEnabled(m_maxItem.get(), false);
     }
 
     else if (isLimited()) {
-        setLimitEnabled(m_minItem, true);
-        setLimitEnabled(m_maxItem, true);
+        setLimitEnabled(m_minItem.get(), true);
+        setLimitEnabled(m_maxItem.get(), true);
     }
 
     else if (isLowerLimited()) {
-        setLimitEnabled(m_minItem, true);
-        setLimitEnabled(m_maxItem, false);
+        setLimitEnabled(m_minItem.get(), true);
+        setLimitEnabled(m_maxItem.get(), false);
     }
 
     else if (isUpperLimited()) {
-        setLimitEnabled(m_minItem, false);
-        setLimitEnabled(m_maxItem, true);
+        setLimitEnabled(m_minItem.get(), false);
+        setLimitEnabled(m_maxItem.get(), true);
     }
 
     else if (isFree()) {
-        setLimitEnabled(m_minItem, false);
-        setLimitEnabled(m_maxItem, false);
+        setLimitEnabled(m_minItem.get(), false);
+        setLimitEnabled(m_maxItem.get(), false);
     }
 }
 

GUI/Model/Job/FitParameterItem.h

@@ -124,10 +124,10 @@ private:
     bool isFixed() const;
 
     QString m_displayName;
-    FitTypeItem* m_typeItem;
-    FitDoubleItem* m_startValueItem;
-    FitEditableDoubleItem* m_minItem;
-    FitEditableDoubleItem* m_maxItem;
+    std::unique_ptr<FitTypeItem> m_typeItem;
+    std::unique_ptr<FitDoubleItem> m_startValueItem;
+    std::unique_ptr<FitEditableDoubleItem> m_minItem;
+    std::unique_ptr<FitEditableDoubleItem> m_maxItem;
     QVector<FitParameterLinkItem*> m_links;
 };