Add missing REMOVE slot

Model derived from tableview is expected to have remove slot,
which is missing in DivePlannerDisplay currently. This patch fixes
the annoying warning message while starting Subsurface because of
not including remove slot.

Currently there is lot of work to do, that either requires modifying
the model to skip 'remove' slot or find a way to make them inactive.

Signed-off-by: Lakshman Anumolu <acrlakshman@gmail.com>
Signed-off-by: Dirk Hohndel <dirk@hohndel.org>
This commit is contained in:
Lakshman 2014-04-02 08:51:10 -05:00 committed by Dirk Hohndel
parent b629905ec5
commit 4722104e1c
2 changed files with 16 additions and 0 deletions

View file

@ -75,6 +75,8 @@ QVariant DivePlannerDisplay::data(const QModelIndex &index, int role) const
}
} else if (role == Qt::DecorationRole) {
switch (index.column()) {
case REMOVE:
return QIcon(":trash");
}
} else if (role == Qt::FontRole) {
return defaultModelFont();
@ -159,6 +161,15 @@ void DivePlannerDisplay::insertPoint(const struct computedPoint &p)
computedPoints.append(p);
}
void DivePlannerDisplay::remove(const QModelIndex &index)
{
if (index.column() != REMOVE)
return;
beginRemoveRows(QModelIndex(), index.row(), index.row());
endRemoveRows();
}
static DivePlannerPointsModel *plannerModel = DivePlannerPointsModel::instance();
DivePlannerGraphics::DivePlannerGraphics(QWidget *parent) : QGraphicsView(parent),

View file

@ -31,6 +31,7 @@ private:
public:
static DivePlannerDisplay *instance();
enum Sections {
REMOVE,
COMPUTED_DEPTH,
COMPUTED_DURATION,
COLUMNS
@ -47,6 +48,10 @@ public:
void removeStops();
void addStops();
void insertPoint(const struct computedPoint &p);
public
slots:
void remove(const QModelIndex &index);
};
class DivePlannerPointsModel : public QAbstractTableModel {