mifr/subsurface
1
0
Fork 0
mirror of https://github.com/subsurface/subsurface.git synced 2025-02-19 22:16:15 +00:00
Code Issues Projects Releases Packages Wiki Activity

After DC import don't do RemoveRows if there is nothing to remove

Browse source 
This fixes an issue where beginRemoveRows is called with argument -1
in DiveImportedModel::setImportedDivesIndexes.

Signed-off-by: Stefan Fuchs <sfuchs@gmx.de>
This commit is contained in:
Stefan Fuchs 2017-04-21 19:46:13 +02:00 • committed by Dirk Hohndel
parent 441cfb3f05
commit 02e768a61b
1 changed files with 4 additions and 2 deletions
Download patch file Download diff file Expand all files Collapse all files

6
desktop-widgets/downloadfromdivecomputer.cpp
View file

@ -733,8 +733,10 @@ void DiveImportedModel::clearTable()
void DiveImportedModel::setImportedDivesIndexes(int first, int last) void DiveImportedModel::setImportedDivesIndexes(int first, int last)
{ {
Q_ASSERT(last >= first); Q_ASSERT(last >= first);
beginRemoveRows(QModelIndex(), 0, lastIndex - firstIndex); if (lastIndex >= firstIndex) {
endRemoveRows(); beginRemoveRows(QModelIndex(), 0, lastIndex - firstIndex);
endRemoveRows();
}
beginInsertRows(QModelIndex(), 0, last - first); beginInsertRows(QModelIndex(), 0, last - first);
lastIndex = last; lastIndex = last;
firstIndex = first; firstIndex = first;