diff options
author | Marc Mutz <marc.mutz@kdab.com> | 2015-09-09 09:29:52 +0200 |
---|---|---|
committer | Marc Mutz <marc.mutz@kdab.com> | 2016-02-17 16:17:32 +0000 |
commit | 30b0c346ec9bfe3730aa1a5a77199a3b0e61c1f8 (patch) | |
tree | 3e5d6b3dd866b82e7966b5aaffdb0e7a951f02f4 /src | |
parent | 4262f31330ea48c384396e40dc50811b97642575 (diff) |
QShortcutMap: re-use an existing bool variable
... instead of re-evaluating its expression. More readable.
Change-Id: I18c6ab3bbc4c5a14328f9910fab991f6cad5549d
Reviewed-by: Friedemann Kleint <Friedemann.Kleint@theqtcompany.com>
Diffstat (limited to 'src')
-rw-r--r-- | src/gui/kernel/qshortcutmap.cpp | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/src/gui/kernel/qshortcutmap.cpp b/src/gui/kernel/qshortcutmap.cpp index 14c01085e7..5eb216d58e 100644 --- a/src/gui/kernel/qshortcutmap.cpp +++ b/src/gui/kernel/qshortcutmap.cpp @@ -191,7 +191,7 @@ int QShortcutMap::removeShortcut(int id, QObject *owner, const QKeySequence &key bool allIds = id == 0; // Special case, remove everything - if (allOwners && allKeys && id == 0) { + if (allOwners && allKeys && allIds) { itemsRemoved = d->sequences.size(); d->sequences.clear(); return itemsRemoved; |