aboutsummaryrefslogtreecommitdiffstats
path: root/profile-widget
diff options
context:
space:
mode:
authorGravatar Berthold Stoeger <bstoeger@mail.tuwien.ac.at>2021-04-25 22:06:14 +0200
committerGravatar Dirk Hohndel <dirk@hohndel.org>2021-04-26 08:24:28 -0700
commit78f4d7b2b921f4c1f1f313c5d3b1ed393144a9da (patch)
treeb161c34bc1a05e7dfdb04643a8fef3d41703f91f /profile-widget
parente9dd1b150f3958ae90900456b04ce0688c93d7fe (diff)
downloadsubsurface-78f4d7b2b921f4c1f1f313c5d3b1ed393144a9da.tar.gz
profile: simplify checking for DiveTextItem
When creating the context menu on the profile, the code has to check whether the context menu is activated on the dive computer name to show a special menu (delete / split dive computer). This was done by setting a special property on the item and then checking for that property on the item that the menu is invoked on or its parents. The reason the code didn't simply check the pointer was probably that DiveTextItem uses multiple inheritance: It derives from QObject and QGraphicsItem. It has to derive from QObject first, because (the ridiculously broken) MOC needs it that way. The object added to the scene is a QGraphicsItem. Thus, we get a pointer _into_ the DiveTextItem object. However, that's all completely unnecessary. We can simply compare the pointers, as the compiler will understand that QGraphicsItem is only the second base class of DiveTextItem. Magic! Let's remove the cruft and simply compare the pointers. Signed-off-by: Berthold Stoeger <bstoeger@mail.tuwien.ac.at>
Diffstat (limited to 'profile-widget')
-rw-r--r--profile-widget/profilewidget2.cpp10
1 files changed, 1 insertions, 9 deletions
diff --git a/profile-widget/profilewidget2.cpp b/profile-widget/profilewidget2.cpp
index 3557ede83..b8b655e04 100644
--- a/profile-widget/profilewidget2.cpp
+++ b/profile-widget/profilewidget2.cpp
@@ -217,9 +217,6 @@ void ProfileWidget2::addActionShortcut(const Qt::Key shortcut, void (ProfileWidg
}
#endif // SUBSURFACE_MOBILE
-#define SUBSURFACE_OBJ_DATA 1
-#define SUBSURFACE_OBJ_DC_TEXT 0x42
-
void ProfileWidget2::addItemsToScene()
{
scene()->addItem(background);
@@ -228,11 +225,6 @@ void ProfileWidget2::addItemsToScene()
scene()->addItem(temperatureAxis);
scene()->addItem(timeAxis);
scene()->addItem(cylinderPressureAxis);
- // I cannot seem to figure out if an object that I find with itemAt() on the scene
- // is the object I am looking for - my guess is there's a simple way in Qt to do that
- // but nothing I tried worked.
- // so instead this adds a special magic key/value pair to the object to mark it
- diveComputerText->setData(SUBSURFACE_OBJ_DATA, SUBSURFACE_OBJ_DC_TEXT);
scene()->addItem(diveComputerText);
scene()->addItem(tankItem);
scene()->addItem(decoModelParameters);
@@ -1365,7 +1357,7 @@ void ProfileWidget2::contextMenuEvent(QContextMenuEvent *event)
if (sceneItem) {
QGraphicsItem *parentItem = sceneItem;
while (parentItem) {
- if (parentItem->data(SUBSURFACE_OBJ_DATA) == SUBSURFACE_OBJ_DC_TEXT) {
+ if (parentItem == diveComputerText) {
isDCName = true;
break;
}