Dive pictures: Fix crash on picture delete

The recent simplification of the close button code introduced a crash:
Deletion of pictures caused an invalid memory access, because the
CloseButtonItem was deleted with the parent DivePicture item.
For some (not fully understood!) reason, a reference to this button
was stored in the depths of Qt.

Empirically, it was found out that removing the first line of the pair
       QGraphicsItem::mousePressEvent(event);
       emit clicked();
fixed the crash.

It seemed therefore prudent to remove the whole questionable signal/slot
mechanism and directly call the removePicture() function of the parent.
Thus, the intermediate DiveButtonItem class became unnecessary and was
removed, leading to a shallower class hierarchy.

Unfortunately, CloseButtonItem must still be derived from QObject owing
to the Q_PROPERTY machinery, which is in turn needed for animation.

To make this compile on mobile, the conditional compilation of
removePicture() (#ifndef SUBSURFACE_MOBILE) was removed. After all,
if DivePixmapItem is used, there are pictures, so removePicture()
should be functional. Conditional compilation should concern the
whole class, not only this function.

Signed-off-by: Berthold Stoeger <bstoeger@mail.tuwien.ac.at>

1 files changed, 3 insertions, 13 deletions

diff --git a/profile-widget/divepixmapitem.h b/profile-widget/divepixmapitem.h
index 16dbdf8fa..6abe410a6 100644
--- a/profile-widget/divepixmapitem.h
+++ b/profile-widget/divepixmapitem.h
@@ -15,20 +15,12 @@ public:
 	DivePixmapItem(QGraphicsItem *parent = 0);
 };
 
-class DiveButtonItem : public DivePixmapItem {
+class CloseButtonItem : public DivePixmapItem {
 	Q_OBJECT
 public:
-	DiveButtonItem(QGraphicsItem *parent = 0);
+	CloseButtonItem(QGraphicsItem *parent = 0);
 protected:
 	virtual void mousePressEvent(QGraphicsSceneMouseEvent *event);
-signals:
-	void clicked();
-};
-
-class CloseButtonItem : public DiveButtonItem {
-	Q_OBJECT
-public:
-	CloseButtonItem(QGraphicsItem *parent = 0);
 public slots:
 	void hide();
 	void show();
@@ -42,9 +34,7 @@ public:
 	void setPixmap(const QPixmap& pix);
 public slots:
 	void settingsChanged();
-#ifndef SUBSURFACE_MOBILE
 	void removePicture();
-#endif
 	void setFileUrl(const QString& s);
 protected:
 	void hoverEnterEvent(QGraphicsSceneHoverEvent *event);
@@ -54,7 +44,7 @@ private:
 	QString fileUrl;
 	QGraphicsRectItem *canvas;
 	QGraphicsRectItem *shadow;
-	DiveButtonItem *button;
+	CloseButtonItem *button;
 };
 
 #endif // DIVEPIXMAPITEM_H