tst_Gestures: fix -Wdangling-else GCC warning

... and do a local code cleanup:

- add override
- port from foreach to C++11 ranged for loop
- remove dead code (`gestureEvent` (née event) was already
  dereferenced when we check it for null'ness, so it cannot be
  nullptr, so the condition is always true).

Pick-to: 5.15
Change-Id: Ica5e34fbe65c95f6573630f188582b90be15c8f7
Reviewed-by: Friedemann Kleint <Friedemann.Kleint@qt.io>

1 files changed, 4 insertions, 4 deletions

diff --git a/tests/auto/other/gestures/tst_gestures.cpp b/tests/auto/other/gestures/tst_gestures.cpp
index c1d39d8b72..278e1d3070 100644
--- a/tests/auto/other/gestures/tst_gestures.cpp
+++ b/tests/auto/other/gestures/tst_gestures.cpp
@@ -1401,13 +1401,13 @@ void tst_Gestures::ungrabGesture() // a method on QWidget
 
         QSet<QGesture*> gestures;
     protected:
-        bool event(QEvent *event)
+        bool event(QEvent *event) override
         {
             if (event->type() == QEvent::Gesture) {
                 QGestureEvent *gestureEvent = static_cast<QGestureEvent*>(event);
-                if (gestureEvent)
-                    foreach (QGesture *g, gestureEvent->gestures())
-                        gestures.insert(g);
+                const auto eventGestures = gestureEvent->gestures();
+                for (QGesture *g : eventGestures)
+                    gestures.insert(g);
             }
             return GestureWidget::event(event);
         }