diff --git a/src/plugins/clangstaticanalyzer/clangstaticanalyzerruncontrol.cpp b/src/plugins/clangstaticanalyzer/clangstaticanalyzerruncontrol.cpp index 223a0ee9813..c286a22c017 100644 --- a/src/plugins/clangstaticanalyzer/clangstaticanalyzerruncontrol.cpp +++ b/src/plugins/clangstaticanalyzer/clangstaticanalyzerruncontrol.cpp @@ -30,7 +30,7 @@ #include "clangstaticanalyzersettings.h" #include "clangstaticanalyzerutils.h" -#include +#include #include @@ -419,7 +419,8 @@ void ClangStaticAnalyzerRunControl::start() const QString errorMessage = tr("Clang Static Analyzer: Invalid executable \"%1\", stop.") .arg(executable); appendMessage(errorMessage + QLatin1Char('\n'), Utils::ErrorMessageFormat); - AnalyzerUtils::logToIssuesPane(Task::Error, errorMessage); + TaskHub::addTask(Task::Error, errorMessage, Debugger::Constants::ANALYZERTASK_ID); + TaskHub::requestPopup(); emit finished(); return; } @@ -432,7 +433,8 @@ void ClangStaticAnalyzerRunControl::start() const QString errorMessage = tr("Clang Static Analyzer: Failed to create temporary dir, stop."); appendMessage(errorMessage + QLatin1Char('\n'), Utils::ErrorMessageFormat); - AnalyzerUtils::logToIssuesPane(Task::Error, errorMessage); + TaskHub::addTask(Task::Error, errorMessage, Debugger::Constants::ANALYZERTASK_ID); + TaskHub::requestPopup(); emit finished(); return; } @@ -571,8 +573,8 @@ void ClangStaticAnalyzerRunControl::onRunnerFinishedWithFailure(const QString &e + QLatin1Char('\n') , Utils::StdErrFormat); appendMessage(errorDetails, Utils::StdErrFormat); - AnalyzerUtils::logToIssuesPane(Task::Warning, errorMessage); - AnalyzerUtils::logToIssuesPane(Task::Warning, errorDetails); + TaskHub::addTask(Task::Warning, errorMessage, Debugger::Constants::ANALYZERTASK_ID); + TaskHub::addTask(Task::Warning, errorDetails, Debugger::Constants::ANALYZERTASK_ID); handleFinished(); } @@ -605,8 +607,9 @@ void ClangStaticAnalyzerRunControl::finalize() Utils::NormalMessageFormat); if (m_filesNotAnalyzed != 0) { - AnalyzerUtils::logToIssuesPane(Task::Error, - tr("Clang Static Analyzer: Not all files could be analyzed.")); + QString msg = tr("Clang Static Analyzer: Not all files could be analyzed."); + TaskHub::addTask(Task::Error, msg, Debugger::Constants::ANALYZERTASK_ID); + TaskHub::requestPopup(); } m_progress.reportFinished(); diff --git a/src/plugins/debugger/analyzer/analyzerutils.cpp b/src/plugins/debugger/analyzer/analyzerutils.cpp index c38e54bb454..21146372ecb 100644 --- a/src/plugins/debugger/analyzer/analyzerutils.cpp +++ b/src/plugins/debugger/analyzer/analyzerutils.cpp @@ -25,10 +25,7 @@ #include "analyzerutils.h" -#include "analyzerconstants.h" - #include -#include #include #include @@ -39,7 +36,6 @@ #include -using namespace Debugger; using namespace Core; using namespace ProjectExplorer; @@ -89,10 +85,3 @@ CPlusPlus::Symbol *AnalyzerUtils::findSymbolUnderCursor() const CPlusPlus::LookupItem &lookupItem = lookupItems.first(); // ### TODO: select best candidate. return lookupItem.declaration(); } - -void AnalyzerUtils::logToIssuesPane(Task::TaskType type, const QString &message) -{ - TaskHub::addTask(type, message, Debugger::Constants::ANALYZERTASK_ID); - if (type == Task::Error) - TaskHub::requestPopup(); -} diff --git a/src/plugins/debugger/analyzer/analyzerutils.h b/src/plugins/debugger/analyzer/analyzerutils.h index 1c5327d4fe9..f38fdf80af1 100644 --- a/src/plugins/debugger/analyzer/analyzerutils.h +++ b/src/plugins/debugger/analyzer/analyzerutils.h @@ -27,13 +27,9 @@ #include -#include - namespace CPlusPlus { class Symbol; } namespace AnalyzerUtils { DEBUGGER_EXPORT CPlusPlus::Symbol *findSymbolUnderCursor(); - DEBUGGER_EXPORT void logToIssuesPane(ProjectExplorer::Task::TaskType type, - const QString &message); } diff --git a/src/plugins/valgrind/callgrindtool.cpp b/src/plugins/valgrind/callgrindtool.cpp index 28a3dd52e8e..6d708997d6b 100644 --- a/src/plugins/valgrind/callgrindtool.cpp +++ b/src/plugins/valgrind/callgrindtool.cpp @@ -78,6 +78,7 @@ #include #include #include +#include #include #include @@ -909,8 +910,9 @@ void CallgrindTool::loadExternalLogFile() QFile logFile(filePath); if (!logFile.open(QIODevice::ReadOnly | QIODevice::Text)) { - AnalyzerUtils::logToIssuesPane(Task::Error, - tr("Callgrind: Failed to open file for reading: %1").arg(filePath)); + QString msg = tr("Callgrind: Failed to open file for reading: %1").arg(filePath); + TaskHub::addTask(Task::Error, msg, Debugger::Constants::ANALYZERTASK_ID); + TaskHub::requestPopup(); return; } diff --git a/src/plugins/valgrind/memchecktool.cpp b/src/plugins/valgrind/memchecktool.cpp index ded4867fcec..8e4ea368782 100644 --- a/src/plugins/valgrind/memchecktool.cpp +++ b/src/plugins/valgrind/memchecktool.cpp @@ -53,6 +53,7 @@ #include #include #include +#include #include #include @@ -592,8 +593,9 @@ void MemcheckTool::loadExternalXmlLogFile() QFile *logFile = new QFile(filePath); if (!logFile->open(QIODevice::ReadOnly | QIODevice::Text)) { delete logFile; - AnalyzerUtils::logToIssuesPane(Task::Error, - tr("Memcheck: Failed to open file for reading: %1").arg(filePath)); + QString msg = tr("Memcheck: Failed to open file for reading: %1").arg(filePath); + TaskHub::addTask(Task::Error, msg, Debugger::Constants::ANALYZERTASK_ID); + TaskHub::requestPopup(); return; } @@ -623,8 +625,9 @@ void MemcheckTool::parserError(const Error &error) void MemcheckTool::internalParserError(const QString &errorString) { - AnalyzerUtils::logToIssuesPane(Task::Error, - tr("Memcheck: Error occurred parsing Valgrind output: %1").arg(errorString)); + QString msg = tr("Memcheck: Error occurred parsing Valgrind output: %1").arg(errorString); + TaskHub::addTask(Task::Error, msg, Debugger::Constants::ANALYZERTASK_ID); + TaskHub::requestPopup(); } void MemcheckTool::clearErrorView()