Dive pictures: use modelReset signal when reseting picture list

In the old code a combination of removeRows()/insertRows() was used
to signal a model reset.

Replace this by a single, modelReset signal. This saves a call to
plotPictures() and will allow us to be smarter in the future,
when removing pictures.

Reset-model and remove-items are semantically different.

Signed-off-by: Berthold Stoeger <bstoeger@mail.tuwien.ac.at>

1 files changed, 2 insertions, 7 deletions

diff --git a/qt-models/divepicturemodel.cpp b/qt-models/divepicturemodel.cpp
index 2fc7b8311..9b993cb5a 100644
--- a/qt-models/divepicturemodel.cpp
+++ b/qt-models/divepicturemodel.cpp
@@ -57,10 +57,9 @@ void DivePictureModel::updateThumbnails()
 
 void DivePictureModel::updateDivePictures()
 {
+	beginResetModel();
 	if (!pictures.isEmpty()) {
-		beginRemoveRows(QModelIndex(), 0, pictures.count() - 1);
 		pictures.clear();
-		endRemoveRows();
 		rowDDStart = rowDDEnd = 0;
 		Thumbnailer::instance()->clearWorkQueue();
 	}
@@ -83,11 +82,7 @@ void DivePictureModel::updateDivePictures()
 	}
 
 	updateThumbnails();
-
-	if (!pictures.isEmpty()) {
-		beginInsertRows(QModelIndex(), 0, pictures.count() - 1);
-		endInsertRows();
-	}
+	endResetModel();
 }
 
 int DivePictureModel::columnCount(const QModelIndex &parent) const