From b3e5e5eef4dd7d64ec98702311f017648cefa856 Mon Sep 17 00:00:00 2001 From: Berthold Stoeger Date: Sat, 21 Sep 2019 14:38:10 +0200 Subject: Cleanup: remove bogus assignment in startFilterDiveSites() In MultiFilterSortModel::startFilterDiveSites(), the setting of the dive sites to be filtered is done later in the code. Therefore, remove the assignment in the first line of the function. Under some circumstances, this would prevent a needed map reload! Signed-off-by: Berthold Stoeger --- qt-models/filtermodels.cpp | 1 - 1 file changed, 1 deletion(-) diff --git a/qt-models/filtermodels.cpp b/qt-models/filtermodels.cpp index e9fbb9866..22a0d19cb 100644 --- a/qt-models/filtermodels.cpp +++ b/qt-models/filtermodels.cpp @@ -299,7 +299,6 @@ void MultiFilterSortModel::clearFilter() void MultiFilterSortModel::startFilterDiveSites(QVector ds) { - dive_sites = ds; if (++diveSiteRefCount > 1) { setFilterDiveSite(ds); } else { -- cgit v1.2.3-70-g09d2