diff options
author | Nikolai Kosjar <nikolai.kosjar@digia.com> | 2013-03-05 13:47:29 +0100 |
---|---|---|
committer | hjk <hjk121@nokiamail.com> | 2013-03-06 12:01:10 +0100 |
commit | c3b6d1501c2bb2087274e41f7486d1bfc9721b87 (patch) | |
tree | a81ca28d0b3a7f4d1e5023ddde5167fad8dc5cd7 | |
parent | 7b7058391f6b945e87782bef33f21eaf1029c0a4 (diff) |
C++: Add some QTC_ASSERT guards in CodeAssistPrivate
This should prevent a not reproducible crash.
Change-Id: I8ebdec02799e1eccd37e918d5ab02a0a7fa5afac
Reviewed-by: hjk <hjk121@nokiamail.com>
Reviewed-by: Christian Kandeler <christian.kandeler@digia.com>
Reviewed-by: Eike Ziller <eike.ziller@digia.com>
-rw-r--r-- | src/plugins/texteditor/codeassist/codeassistant.cpp | 6 |
1 files changed, 5 insertions, 1 deletions
diff --git a/src/plugins/texteditor/codeassist/codeassistant.cpp b/src/plugins/texteditor/codeassist/codeassistant.cpp index 2616924ffc..878d838e28 100644 --- a/src/plugins/texteditor/codeassist/codeassistant.cpp +++ b/src/plugins/texteditor/codeassist/codeassistant.cpp @@ -41,6 +41,7 @@ #include <texteditor/texteditorsettings.h> #include <texteditor/completionsettings.h> #include <extensionsystem/pluginmanager.h> +#include <utils/qtcassert.h> #include <QObject> #include <QList> @@ -221,7 +222,7 @@ void CodeAssistantPrivate::requestProposal(AssistReason reason, AssistKind kind, IAssistProvider *provider) { - Q_ASSERT(!isWaitingForProposal()); + QTC_ASSERT(!isWaitingForProposal(), return); if (!provider) { if (kind == Completion) { @@ -328,6 +329,7 @@ void CodeAssistantPrivate::displayProposal(IAssistProposal *newProposal, AssistR void CodeAssistantPrivate::processProposalItem(IAssistProposalItem *proposalItem) { + QTC_ASSERT(m_proposal, return); proposalItem->apply(m_textEditor, m_proposal->basePosition()); destroyContext(); process(); @@ -335,6 +337,7 @@ void CodeAssistantPrivate::processProposalItem(IAssistProposalItem *proposalItem void CodeAssistantPrivate::handlePrefixExpansion(const QString &newPrefix) { + QTC_ASSERT(m_proposal, return); const int currentPosition = m_textEditor->position(); m_textEditor->setCursorPosition(m_proposal->basePosition()); m_textEditor->replace(currentPosition - m_proposal->basePosition(), newPrefix); @@ -397,6 +400,7 @@ void CodeAssistantPrivate::notifyChange() stopAutomaticProposalTimer(); if (isDisplayingProposal()) { + QTC_ASSERT(m_proposal, return); if (m_textEditor->position() < m_proposal->basePosition()) { destroyContext(); } else { |