diff options
author | Thiago Macieira <thiago.macieira@intel.com> | 2018-04-27 00:00:41 -0700 |
---|---|---|
committer | Thiago Macieira <thiago.macieira@intel.com> | 2018-05-01 21:19:11 +0000 |
commit | 88b409cfdd5722ea4bd8f1620624df8f5093baca (patch) | |
tree | d79c2cd532eccc6c66bf5661feb49e1578e5a140 /src/input/frontend/qmousehandler.cpp | |
parent | 2831c495233338403b31348393e785055700541b (diff) |
Fix bad use of QEvent::Timer
QEvent::Timer == 1 == Qt::TapGesture
This is not what was intended here. I'm assuming it's the Gesture event.
Note that there are multiple gesture types possible (hence,
Qt::TapGesture), so this code probably needs to be updated.
Caught by Clang 6:
qmousehandler.cpp:92:10: warning: comparison of two values with different enumeration types
in switch statement ('QEvent::Type' and 'Qt::GestureType') [-Wenum-compare-switch]
Change-Id: I3840d727dee443318644fffd152938d633280de1
Reviewed-by: Mike Krus <mike.krus@kdab.com>
Reviewed-by: Antti Määttä <antti.maatta@qt.io>
Diffstat (limited to 'src/input/frontend/qmousehandler.cpp')
-rw-r--r-- | src/input/frontend/qmousehandler.cpp | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/src/input/frontend/qmousehandler.cpp b/src/input/frontend/qmousehandler.cpp index 5d75f82d4..824e30492 100644 --- a/src/input/frontend/qmousehandler.cpp +++ b/src/input/frontend/qmousehandler.cpp @@ -89,7 +89,7 @@ void QMouseHandlerPrivate::mouseEvent(const QMouseEventPtr &event) emit q->released(event.data()); break; #if QT_CONFIG(gestures) - case Qt::TapGesture: + case QEvent::Gesture: emit q->clicked(event.data()); break; #endif |