debugger: don't store active languages twice

Change-Id: I6517b42b4cfd0d2d919fa2971c3ea0995443cbf3
Reviewed-by: hjk <qthjk@ovi.com>
This commit is contained in:
hjk
2012-08-14 17:54:01 +02:00
parent e481b1ad27
commit 776fd73e30
10 changed files with 11 additions and 27 deletions

View File

@@ -428,7 +428,7 @@ static inline Utils::SavedAction *theAssemblerAction()
CdbEngine::CdbEngine(const DebuggerStartParameters &sp, CdbEngine::CdbEngine(const DebuggerStartParameters &sp,
DebuggerEngine *masterEngine, const OptionsPtr &options) : DebuggerEngine *masterEngine, const OptionsPtr &options) :
DebuggerEngine(sp, CppLanguage, masterEngine), DebuggerEngine(sp, masterEngine),
m_creatorExtPrefix("<qtcreatorcdbext>|"), m_creatorExtPrefix("<qtcreatorcdbext>|"),
m_tokenPrefix("<token>"), m_tokenPrefix("<token>"),
m_options(options), m_options(options),

View File

@@ -155,13 +155,11 @@ class DebuggerEnginePrivate : public QObject
public: public:
DebuggerEnginePrivate(DebuggerEngine *engine, DebuggerEnginePrivate(DebuggerEngine *engine,
DebuggerEngine *masterEngine, DebuggerEngine *masterEngine,
DebuggerLanguages languages,
const DebuggerStartParameters &sp) const DebuggerStartParameters &sp)
: m_engine(engine), : m_engine(engine),
m_masterEngine(masterEngine), m_masterEngine(masterEngine),
m_runControl(0), m_runControl(0),
m_startParameters(sp), m_startParameters(sp),
m_languages(languages),
m_state(DebuggerNotReady), m_state(DebuggerNotReady),
m_lastGoodState(DebuggerNotReady), m_lastGoodState(DebuggerNotReady),
m_targetState(DebuggerNotReady), m_targetState(DebuggerNotReady),
@@ -183,8 +181,6 @@ public:
connect(&m_locationTimer, SIGNAL(timeout()), SLOT(resetLocation())); connect(&m_locationTimer, SIGNAL(timeout()), SLOT(resetLocation()));
} }
~DebuggerEnginePrivate() {}
public slots: public slots:
void doSetupEngine(); void doSetupEngine();
void doSetupInferior(); void doSetupInferior();
@@ -282,7 +278,6 @@ public:
DebuggerRunControl *m_runControl; // Not owned. DebuggerRunControl *m_runControl; // Not owned.
DebuggerStartParameters m_startParameters; DebuggerStartParameters m_startParameters;
DebuggerLanguages m_languages;
// The current state. // The current state.
DebuggerState m_state; DebuggerState m_state;
@@ -336,12 +331,9 @@ public:
////////////////////////////////////////////////////////////////////// //////////////////////////////////////////////////////////////////////
DebuggerEngine::DebuggerEngine(const DebuggerStartParameters &startParameters, DebuggerEngine::DebuggerEngine(const DebuggerStartParameters &startParameters,
DebuggerLanguages languages,
DebuggerEngine *parentEngine) DebuggerEngine *parentEngine)
: d(new DebuggerEnginePrivate(this, parentEngine, languages, startParameters)) : d(new DebuggerEnginePrivate(this, parentEngine, startParameters))
{ {}
d->m_inferiorPid = 0;
}
DebuggerEngine::~DebuggerEngine() DebuggerEngine::~DebuggerEngine()
{ {
@@ -1277,11 +1269,6 @@ DebuggerEngine *DebuggerEngine::masterEngine() const
return d->m_masterEngine; return d->m_masterEngine;
} }
DebuggerLanguages DebuggerEngine::languages() const
{
return d->m_languages;
}
QString DebuggerEngine::toFileInProject(const QUrl &fileUrl) QString DebuggerEngine::toFileInProject(const QUrl &fileUrl)
{ {
// make sure file finder is properly initialized // make sure file finder is properly initialized

View File

@@ -140,7 +140,6 @@ class DEBUGGER_EXPORT DebuggerEngine : public QObject
public: public:
explicit DebuggerEngine(const DebuggerStartParameters &sp, explicit DebuggerEngine(const DebuggerStartParameters &sp,
DebuggerLanguages languages,
DebuggerEngine *parentEngine = 0); DebuggerEngine *parentEngine = 0);
virtual ~DebuggerEngine(); virtual ~DebuggerEngine();
@@ -266,8 +265,6 @@ public:
bool isMasterEngine() const; bool isMasterEngine() const;
DebuggerEngine *masterEngine() const; DebuggerEngine *masterEngine() const;
DebuggerLanguages languages() const;
virtual bool setupQmlStep(bool /*on*/) { return false; } virtual bool setupQmlStep(bool /*on*/) { return false; }
virtual void readyToExecuteQmlStep() {} virtual void readyToExecuteQmlStep() {}

View File

@@ -479,7 +479,7 @@ class DummyEngine : public DebuggerEngine
Q_OBJECT Q_OBJECT
public: public:
DummyEngine() : DebuggerEngine(DebuggerStartParameters(), AnyLanguage) {} DummyEngine() : DebuggerEngine(DebuggerStartParameters()) {}
~DummyEngine() {} ~DummyEngine() {}
void setupEngine() {} void setupEngine() {}
@@ -2002,7 +2002,7 @@ void DebuggerPluginPrivate::connectEngine(DebuggerEngine *engine)
engine->watchHandler()->rebuildModel(); engine->watchHandler()->rebuildModel();
mainWindow()->setEngineDebugLanguages(engine->languages()); mainWindow()->setEngineDebugLanguages(engine->startParameters().languages);
mainWindow()->setCurrentEngine(engine); mainWindow()->setCurrentEngine(engine);
} }

View File

@@ -238,7 +238,7 @@ private:
GdbEngine::GdbEngine(const DebuggerStartParameters &startParameters, GdbEngine::GdbEngine(const DebuggerStartParameters &startParameters,
DebuggerEngine *masterEngine) DebuggerEngine *masterEngine)
: DebuggerEngine(startParameters, CppLanguage, masterEngine) : DebuggerEngine(startParameters, masterEngine)
{ {
setObjectName(_("GdbEngine")); setObjectName(_("GdbEngine"));

View File

@@ -64,7 +64,7 @@ namespace Debugger {
namespace Internal { namespace Internal {
IPCEngineHost::IPCEngineHost (const DebuggerStartParameters &startParameters) IPCEngineHost::IPCEngineHost (const DebuggerStartParameters &startParameters)
: DebuggerEngine(startParameters, CppLanguage) : DebuggerEngine(startParameters)
, m_localGuest(0) , m_localGuest(0)
, m_nextMessagePayloadSize(0) , m_nextMessagePayloadSize(0)
, m_cookie(1) , m_cookie(1)

View File

@@ -88,7 +88,7 @@ namespace Internal {
/////////////////////////////////////////////////////////////////////// ///////////////////////////////////////////////////////////////////////
PdbEngine::PdbEngine(const DebuggerStartParameters &startParameters) PdbEngine::PdbEngine(const DebuggerStartParameters &startParameters)
: DebuggerEngine(startParameters, AnyLanguage) : DebuggerEngine(startParameters)
{ {
setObjectName(QLatin1String("PdbEngine")); setObjectName(QLatin1String("PdbEngine"));
} }

View File

@@ -110,7 +110,7 @@ QmlCppEnginePrivate::QmlCppEnginePrivate(QmlCppEngine *parent,
QmlCppEngine::QmlCppEngine(const DebuggerStartParameters &sp, QmlCppEngine::QmlCppEngine(const DebuggerStartParameters &sp,
DebuggerEngineType slaveEngineType, DebuggerEngineType slaveEngineType,
QString *errorMessage) QString *errorMessage)
: DebuggerEngine(sp, DebuggerLanguages(CppLanguage) | QmlLanguage), d(new QmlCppEnginePrivate(this, sp)) : DebuggerEngine(sp), d(new QmlCppEnginePrivate(this, sp))
{ {
setObjectName(QLatin1String("QmlCppEngine")); setObjectName(QLatin1String("QmlCppEngine"));
d->m_cppEngine = DebuggerRunControlFactory::createEngine(slaveEngineType, sp, this, errorMessage); d->m_cppEngine = DebuggerRunControlFactory::createEngine(slaveEngineType, sp, this, errorMessage);

View File

@@ -268,7 +268,7 @@ public:
QmlEngine::QmlEngine(const DebuggerStartParameters &startParameters, QmlEngine::QmlEngine(const DebuggerStartParameters &startParameters,
DebuggerEngine *masterEngine) DebuggerEngine *masterEngine)
: DebuggerEngine(startParameters, QmlLanguage, masterEngine) : DebuggerEngine(startParameters, masterEngine)
, m_adapter(this) , m_adapter(this)
, m_inspectorAdapter(&m_adapter, this) , m_inspectorAdapter(&m_adapter, this)
, m_retryOnConnectFail(false) , m_retryOnConnectFail(false)

View File

@@ -203,7 +203,7 @@ void ScriptAgent::scriptUnload(qint64 scriptId)
/////////////////////////////////////////////////////////////////////// ///////////////////////////////////////////////////////////////////////
ScriptEngine::ScriptEngine(const DebuggerStartParameters &startParameters) ScriptEngine::ScriptEngine(const DebuggerStartParameters &startParameters)
: DebuggerEngine(startParameters, AnyLanguage) : DebuggerEngine(startParameters)
{ {
setObjectName(QLatin1String("ScriptEngine")); setObjectName(QLatin1String("ScriptEngine"));
} }