From 6da32c780622385e5844e82e5ce6c09f7203b7bf Mon Sep 17 00:00:00 2001 From: Thiago Macieira Date: Thu, 16 Jun 2016 16:57:41 -0700 Subject: Fix compiler warning about missing return statement in non-void function MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit Found by ICC 17 Beta on Windows. This commit also removes the "default" cause from the switch, which will make compilers let us know if we add another enum but don't handle it in the switch. Change-Id: Ib57b52598e2f452985e9fffd1458b5b39b238db7 Reviewed-by: Tor Arne Vestbø --- src/gui/kernel/qshortcutmap.cpp | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) (limited to 'src/gui/kernel/qshortcutmap.cpp') diff --git a/src/gui/kernel/qshortcutmap.cpp b/src/gui/kernel/qshortcutmap.cpp index 3b40aba49e..6c19eab287 100644 --- a/src/gui/kernel/qshortcutmap.cpp +++ b/src/gui/kernel/qshortcutmap.cpp @@ -353,9 +353,9 @@ bool QShortcutMap::tryShortcut(QKeyEvent *e) // shouldn't say that we handled the event. return identicalMatches > 0; } - default: - Q_UNREACHABLE(); } + Q_UNREACHABLE(); + return false; } /*! \internal -- cgit v1.2.3