Remove unreachable return statement, and no need to reset model when …

…modifying data.
disorderedmaterials · May 14, 2024 · 2040c29 · 2040c29
1 parent 1c2000c
commit 2040c29
Showing 1 changed file with 2 additions and 4 deletions.
diff --git a/src/gui/models/masterBondModel.cpp b/src/gui/models/masterBondModel.cpp
@@ -36,8 +36,6 @@ QVariant MasterBondModel::getTermData(int row, MasterTermModelData::DataType dat
         default:
             return {};
     }
-
-    return {};
 }
 
 bool MasterBondModel::setTermData(int row, MasterTermModelData::DataType dataType, const QVariant &value)
@@ -47,7 +45,6 @@ bool MasterBondModel::setTermData(int row, MasterTermModelData::DataType dataTyp
 
     auto &t = sourceData_[row];
 
-    beginResetModel();
     switch (dataType)
     {
         case (MasterTermModelData::DataType::Name):
@@ -71,7 +68,8 @@ bool MasterBondModel::setTermData(int row, MasterTermModelData::DataType dataTyp
         default:
             return false;
     }
-    endResetModel();
+
+    Q_EMIT(dataChanged({}, {}));
 
     return true;
 }