From e8837674931b7f59c154083614eb182b121767f0 Mon Sep 17 00:00:00 2001 From: Anton Kudryavtsev Date: Thu, 21 Jan 2016 02:02:17 +0300 Subject: QQuickMenu: replace if-else chain with switch Change-Id: Ie0a858714b7c56129e0a877bd6abce68ad4323f8 Reviewed-by: J-P Nurmi --- src/templates/qquickmenu.cpp | 12 +++++++++--- 1 file changed, 9 insertions(+), 3 deletions(-) (limited to 'src') diff --git a/src/templates/qquickmenu.cpp b/src/templates/qquickmenu.cpp index 585d758d..d8783cfe 100644 --- a/src/templates/qquickmenu.cpp +++ b/src/templates/qquickmenu.cpp @@ -460,17 +460,23 @@ bool QQuickMenu::eventFilter(QObject *object, QEvent *event) // only allow flicking with the mouse when there are too many menu items to be // shown at once. QKeyEvent *keyEvent = static_cast(event); - if (keyEvent->key() == Qt::Key_Up) { + switch (keyEvent->key()) { + case Qt::Key_Up: if (d->contentItem->metaObject()->indexOfMethod("decrementCurrentIndex()") != -1) QMetaObject::invokeMethod(d->contentItem, "decrementCurrentIndex"); return true; - } else if (keyEvent->key() == Qt::Key_Down) { + + case Qt::Key_Down: if (d->contentItem->metaObject()->indexOfMethod("incrementCurrentIndex()") != -1) QMetaObject::invokeMethod(d->contentItem, "incrementCurrentIndex"); return true; - } else if (keyEvent->key() == Qt::Key_Escape) { + + case Qt::Key_Escape: close(); return true; + + default: + break; } return false; -- cgit v1.2.3