diff options
-rw-r--r-- | qt-models/filtermodels.cpp | 28 | ||||
-rw-r--r-- | qt-models/filtermodels.h | 5 |
2 files changed, 13 insertions, 20 deletions
diff --git a/qt-models/filtermodels.cpp b/qt-models/filtermodels.cpp index f6fa45617..684999fbd 100644 --- a/qt-models/filtermodels.cpp +++ b/qt-models/filtermodels.cpp @@ -10,6 +10,7 @@ #endif #include <QDebug> +#include <algorithm> #define CREATE_INSTANCE_METHOD( CLASS ) \ CLASS *CLASS::instance() \ @@ -39,8 +40,7 @@ bool CLASS::setData(const QModelIndex &index, const QVariant &value, int role) \ #define CREATE_CLEAR_FILTER_METHOD( CLASS ) \ void CLASS::clearFilter() \ { \ - memset(checkState, false, rowCount()); \ - checkState[rowCount() - 1] = false; \ + std::fill(checkState.begin(), checkState.end(), false); \ anyChecked = false; \ emit dataChanged(createIndex(0,0), createIndex(rowCount()-1, 0)); \ } @@ -128,10 +128,8 @@ void SuitsFilterModel::repopulate() qSort(list); list << tr("No suit set"); setStringList(list); - delete[] checkState; - checkState = new bool[list.count()]; - memset(checkState, false, list.count()); - checkState[list.count() - 1] = false; + checkState.resize(list.count()); + std::fill(checkState.begin(), checkState.end(), false); anyChecked = false; } @@ -153,10 +151,8 @@ void TagFilterModel::repopulate() qSort(list); list << tr("Empty tags"); setStringList(list); - delete[] checkState; - checkState = new bool[list.count()]; - memset(checkState, false, list.count()); - checkState[list.count() - 1] = false; + checkState.resize(list.count()); + std::fill(checkState.begin(), checkState.end(), false); anyChecked = false; } @@ -248,10 +244,8 @@ void BuddyFilterModel::repopulate() qSort(list); list << tr("No buddies"); setStringList(list); - delete[] checkState; - checkState = new bool[list.count()]; - memset(checkState, false, list.count()); - checkState[list.count() - 1] = false; + checkState.resize(list.count()); + std::fill(checkState.begin(), checkState.end(), false); anyChecked = false; } @@ -301,10 +295,8 @@ void LocationFilterModel::repopulate() qSort(list); list << tr("No location set"); setStringList(list); - delete[] checkState; - checkState = new bool[list.count()]; - memset(checkState, false, list.count()); - checkState[list.count() - 1] = false; + checkState.resize(list.count()); + std::fill(checkState.begin(), checkState.end(), false); anyChecked = false; } diff --git a/qt-models/filtermodels.h b/qt-models/filtermodels.h index 992582776..cda6e2369 100644 --- a/qt-models/filtermodels.h +++ b/qt-models/filtermodels.h @@ -5,13 +5,14 @@ #include <QStringListModel> #include <QSortFilterProxyModel> #include <stdint.h> +#include <vector> class MultiFilterInterface { public: - MultiFilterInterface() : checkState(NULL), anyChecked(false) {} + MultiFilterInterface() : anyChecked(false) {} virtual bool doFilter(struct dive *d, QModelIndex &index0, QAbstractItemModel *sourceModel) const = 0; virtual void clearFilter() = 0; - bool *checkState; + std::vector<char> checkState; bool anyChecked; }; |