From 4a50badb57efd7725fa540ea7bb13f1e334cb83c Mon Sep 17 00:00:00 2001 From: Berthold Stoeger Date: Mon, 5 Oct 2020 10:12:12 +0200 Subject: cleanup: use std::vector in struct device_table Since we converted from QString to std::string, let's also use std::vector instead of QVector. We don't need COW semantics and all the rigmarole. Let's try to keep Qt data structures out of the core. Signed-off-by: Berthold Stoeger --- qt-models/divecomputermodel.cpp | 8 ++++---- qt-models/divecomputermodel.h | 2 +- 2 files changed, 5 insertions(+), 5 deletions(-) (limited to 'qt-models') diff --git a/qt-models/divecomputermodel.cpp b/qt-models/divecomputermodel.cpp index c675ad641..d0b9f7637 100644 --- a/qt-models/divecomputermodel.cpp +++ b/qt-models/divecomputermodel.cpp @@ -11,7 +11,7 @@ DiveComputerModel::DiveComputerModel(QObject *parent) : CleanerTableModel(parent QVariant DiveComputerModel::data(const QModelIndex &index, int role) const { - if (index.row() < 0 || index.row() >= dcs.size()) + if (index.row() < 0 || index.row() >= (int)dcs.size()) return QVariant(); const device &node = dcs[index.row()]; @@ -55,7 +55,7 @@ Qt::ItemFlags DiveComputerModel::flags(const QModelIndex &index) const bool DiveComputerModel::setData(const QModelIndex &index, const QVariant &value, int) { // We should test if the role == Qt::EditRole - if (index.row() < 0 || index.row() >= dcs.size()) + if (index.row() < 0 || index.row() >= (int)dcs.size()) return false; device &node = dcs[index.row()]; @@ -66,10 +66,10 @@ bool DiveComputerModel::setData(const QModelIndex &index, const QVariant &value, void DiveComputerModel::remove(const QModelIndex &index) { - if (index.row() < 0 || index.row() >= dcs.size()) + if (index.row() < 0 || index.row() >= (int)dcs.size()) return; beginRemoveRows(QModelIndex(), index.row(), index.row()); - dcs.remove(index.row()); + dcs.erase(dcs.begin() + index.row()); endRemoveRows(); } diff --git a/qt-models/divecomputermodel.h b/qt-models/divecomputermodel.h index d50e1cd96..b21eadcc9 100644 --- a/qt-models/divecomputermodel.h +++ b/qt-models/divecomputermodel.h @@ -27,7 +27,7 @@ slots: void remove(const QModelIndex &index); private: - QVector dcs; + std::vector dcs; }; class DiveComputerSortedModel : public QSortFilterProxyModel { -- cgit v1.2.3-70-g09d2