From 5b0fc95354140c714b6933a811a6561b69d4bcda Mon Sep 17 00:00:00 2001 From: hjk Date: Wed, 6 Jul 2011 11:57:33 +0200 Subject: [PATCH] analyzer: remove clutter Change-Id: I195b92107e90519d456dbd12a28e9ad236e4b298 Reviewed-on: http://codereview.qt.nokia.com/1241 Reviewed-by: Qt Sanity Bot Reviewed-by: hjk --- .../analyzerbase/analyzerruncontrol.cpp | 33 +++++++++++++------ src/plugins/analyzerbase/analyzerruncontrol.h | 5 ++- 2 files changed, 25 insertions(+), 13 deletions(-) diff --git a/src/plugins/analyzerbase/analyzerruncontrol.cpp b/src/plugins/analyzerbase/analyzerruncontrol.cpp index ce87636400c..889b2a911b6 100644 --- a/src/plugins/analyzerbase/analyzerruncontrol.cpp +++ b/src/plugins/analyzerbase/analyzerruncontrol.cpp @@ -47,11 +47,16 @@ #include #include -#include -using namespace Analyzer; +using namespace ProjectExplorer; -// AnalyzerRunControl::Private /////////////////////////////////////////// +////////////////////////////////////////////////////////////////////////// +// +// AnalyzerRunControl::Private +// +////////////////////////////////////////////////////////////////////////// + +namespace Analyzer { class AnalyzerRunControl::Private { @@ -67,7 +72,12 @@ AnalyzerRunControl::Private::Private() {} -// AnalyzerRunControl //////////////////////////////////////////////////// +////////////////////////////////////////////////////////////////////////// +// +// AnalyzerRunControl +// +////////////////////////////////////////////////////////////////////////// + AnalyzerRunControl::AnalyzerRunControl(IAnalyzerTool *tool, const AnalyzerStartParameters &sp, RunConfiguration *runConfiguration) : RunControl(runConfiguration, tool->id()), @@ -94,6 +104,7 @@ AnalyzerRunControl::~AnalyzerRunControl() delete d->m_engine; d->m_engine = 0; + delete d; } void AnalyzerRunControl::start() @@ -105,7 +116,7 @@ void AnalyzerRunControl::start() // clear about-to-be-outdated tasks ExtensionSystem::PluginManager *pm = ExtensionSystem::PluginManager::instance(); - ProjectExplorer::TaskHub *hub = pm->getObject(); + TaskHub *hub = pm->getObject(); hub->clearTasks(Constants::ANALYZERTASK_ID); d->m_isRunning = true; @@ -113,7 +124,7 @@ void AnalyzerRunControl::start() d->m_engine->start(); } -ProjectExplorer::RunControl::StopResult AnalyzerRunControl::stop() +RunControl::StopResult AnalyzerRunControl::stop() { if (!d->m_engine || !d->m_isRunning) return StoppedSynchronously; @@ -125,7 +136,7 @@ ProjectExplorer::RunControl::StopResult AnalyzerRunControl::stop() void AnalyzerRunControl::stopIt() { - if (stop() == ProjectExplorer::RunControl::StoppedSynchronously) + if (stop() == RunControl::StoppedSynchronously) AnalyzerManager::handleToolFinished(); } @@ -162,12 +173,12 @@ void AnalyzerRunControl::receiveOutput(const QString &text, Utils::OutputFormat appendMessage(text, format); } -void AnalyzerRunControl::addTask(ProjectExplorer::Task::TaskType type, const QString &description, +void AnalyzerRunControl::addTask(Task::TaskType type, const QString &description, const QString &file, int line) { ExtensionSystem::PluginManager *pm = ExtensionSystem::PluginManager::instance(); - ProjectExplorer::TaskHub *hub = pm->getObject(); - hub->addTask(ProjectExplorer::Task(type, description, file, line, Constants::ANALYZERTASK_ID)); + TaskHub *hub = pm->getObject(); + hub->addTask(Task(type, description, file, line, Constants::ANALYZERTASK_ID)); ///FIXME: get a better API for this into Qt Creator QList panes = pm->getObjects(); @@ -178,3 +189,5 @@ void AnalyzerRunControl::addTask(ProjectExplorer::Task::TaskType type, const QSt } } } + +} // namespace Analyzer diff --git a/src/plugins/analyzerbase/analyzerruncontrol.h b/src/plugins/analyzerbase/analyzerruncontrol.h index 046aa847ec3..ff320a4e2b6 100644 --- a/src/plugins/analyzerbase/analyzerruncontrol.h +++ b/src/plugins/analyzerbase/analyzerruncontrol.h @@ -52,9 +52,8 @@ class ANALYZER_EXPORT AnalyzerRunControl : public ProjectExplorer::RunControl Q_OBJECT public: - typedef ProjectExplorer::RunConfiguration RunConfiguration; AnalyzerRunControl(IAnalyzerTool *tool, const AnalyzerStartParameters &sp, - RunConfiguration *runConfiguration); + ProjectExplorer::RunConfiguration *runConfiguration); ~AnalyzerRunControl(); // ProjectExplorer::RunControl @@ -76,7 +75,7 @@ private slots: private: class Private; - QScopedPointer d; + Private *d; }; } // namespace Analyzer