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

Cleanup: return each dive site only once in selectedDiveSites()

Browse source 
In TabDiveSite::selectedDiveSites(), the QItemSelectionModel::
selectedIndexes() function was used. Thus for every selected
dive site 8 entries were added to the return-vector!

Instead, use the QItemSelectionModel::selectedRows() function.

Signed-off-by: Berthold Stoeger <bstoeger@mail.tuwien.ac.at>
This commit is contained in:
Berthold Stoeger 2019-09-21 14:30:04 +02:00 • committed by Dirk Hohndel
parent 44a291a09f
commit 4b401eb2de
1 changed files with 1 additions and 1 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

2
desktop-widgets/tab-widgets/TabDiveSite.cpp
View file

@ -85,7 +85,7 @@ void TabDiveSite::on_filterText_textChanged(const QString &text)
QVector<dive_site *> TabDiveSite::selectedDiveSites()
{
const QModelIndexList indexes = ui.diveSites->view()->selectionModel()->selectedIndexes();
const QModelIndexList indexes = ui.diveSites->view()->selectionModel()->selectedRows();
QVector<dive_site *> sites;
sites.reserve(indexes.size());
for (const QModelIndex &idx: indexes) {