diff options
-rw-r--r-- | src/plugins/terminal/shortcutmap.cpp | 13 | ||||
-rw-r--r-- | src/plugins/terminal/terminalwidget.cpp | 10 |
2 files changed, 14 insertions, 9 deletions
diff --git a/src/plugins/terminal/shortcutmap.cpp b/src/plugins/terminal/shortcutmap.cpp index 7a63a087f69..72023e808a0 100644 --- a/src/plugins/terminal/shortcutmap.cpp +++ b/src/plugins/terminal/shortcutmap.cpp @@ -5,6 +5,8 @@ #include "shortcutmap.h" +#include <utils/qtcassert.h> + #include <algorithm> #include <QGuiApplication> @@ -125,8 +127,9 @@ int ShortcutMap::addShortcut(QObject *owner, Qt::ShortcutContext context, ContextMatcher matcher) { - Q_ASSERT_X(owner, "ShortcutMap::addShortcut", "All shortcuts need an owner"); - Q_ASSERT_X(!key.isEmpty(), "ShortcutMap::addShortcut", "Cannot add keyless shortcuts to map"); + QTC_ASSERT(owner, return 0); // "ShortcutMap::addShortcut", "All shortcuts need an owner"); + QTC_ASSERT(!key.isEmpty(), + return 0); // "ShortcutMap::addShortcut", "Cannot add keyless shortcuts to map"); Q_D(ShortcutMap); ShortcutEntry newEntry(owner, key, context, --(d->currentId), true, matcher); @@ -327,9 +330,9 @@ QKeySequence::SequenceMatch ShortcutMap::find(QKeyEvent *e, int ignoredModifiers // Should never happen if (d->newEntries == d->currentSequences) { - Q_ASSERT_X(e->key() != Qt::Key_unknown || e->text().size(), - "ShortcutMap::find", - "New sequence to find identical to previous"); + QTC_ASSERT(e->key() != Qt::Key_unknown || e->text().size(), return QKeySequence::NoMatch); + //"ShortcutMap::find", + // "New sequence to find identical to previous"); return QKeySequence::NoMatch; } diff --git a/src/plugins/terminal/terminalwidget.cpp b/src/plugins/terminal/terminalwidget.cpp index 91905c99d40..67530473697 100644 --- a/src/plugins/terminal/terminalwidget.cpp +++ b/src/plugins/terminal/terminalwidget.cpp @@ -262,10 +262,12 @@ void TerminalWidget::registerShortcut(Command *cmd) QTC_ASSERT(cmd, return); auto addShortCut = [this, cmd] { for (const auto &keySequence : cmd->keySequences()) { - m_shortcutMap.addShortcut(cmd->action(), - keySequence, - Qt::ShortcutContext::WindowShortcut, - contextMatcher); + if (!keySequence.isEmpty()) { + m_shortcutMap.addShortcut(cmd->action(), + keySequence, + Qt::ShortcutContext::WindowShortcut, + contextMatcher); + } } }; auto removeShortCut = [this, cmd] { m_shortcutMap.removeShortcut(0, cmd->action()); }; |