diff options
| author | Thiago Macieira <thiago.macieira@intel.com> | 2016-06-16 16:57:41 -0700 |
|---|---|---|
| committer | Thiago Macieira <thiago.macieira@intel.com> | 2016-06-17 16:27:26 +0000 |
| commit | 6da32c780622385e5844e82e5ce6c09f7203b7bf (patch) | |
| tree | 6ceb527a42e5ae2f0bd0ad3fad35fca8a61e7c28 /src | |
| parent | 691b8678954c64b1b371d782703fc6ebdfc37a68 (diff) | |
Fix compiler warning about missing return statement in non-void function
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ø <tor.arne.vestbo@theqtcompany.com>
Diffstat (limited to 'src')
| -rw-r--r-- | src/gui/kernel/qshortcutmap.cpp | 4 |
1 files changed, 2 insertions, 2 deletions
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 |