diff options
author | hjk <hjk@qt.io> | 2017-06-27 17:30:32 +0200 |
---|---|---|
committer | hjk <hjk@qt.io> | 2017-06-28 09:05:43 +0000 |
commit | 682cd36b1598c29a17b993ba94fa006a012aa796 (patch) | |
tree | 85eab1aad935bd44331069f6fddb6d80603c16a5 /src/plugins/valgrind/memchecktool.cpp | |
parent | 228e4a7542f03acde52c697dcbd3d4d93efa3cf5 (diff) |
Valgrind: Remove starting() signal
It's only used internally as one-time communication between
worker and the *Tool singleton that handles toolbar button
states.
We basically start immediately after creation of the worker, so
any pre-start tool bar specific operations can be done at
RunWorker creation time.
Medium term, the runworker should proably steer "their" buttons
more directly.
Change-Id: Id6df703746ece5eebc23507739cd2a92ec55d11d
Reviewed-by: Christian Kandeler <christian.kandeler@qt.io>
Diffstat (limited to 'src/plugins/valgrind/memchecktool.cpp')
-rw-r--r-- | src/plugins/valgrind/memchecktool.cpp | 10 |
1 files changed, 2 insertions, 8 deletions
diff --git a/src/plugins/valgrind/memchecktool.cpp b/src/plugins/valgrind/memchecktool.cpp index ae272f6ad3..16977250a6 100644 --- a/src/plugins/valgrind/memchecktool.cpp +++ b/src/plugins/valgrind/memchecktool.cpp @@ -250,7 +250,6 @@ private: void settingsDestroyed(QObject *settings); void maybeActiveRunConfigurationChanged(); - void engineStarting(const MemcheckToolRunner *engine); void engineFinished(); void loadingExternalXmlLogFileFinished(); @@ -565,8 +564,6 @@ RunWorker *MemcheckTool::createRunWorker(RunControl *runControl) auto runTool = new MemcheckToolRunner(runControl, runControl->runMode() == MEMCHECK_WITH_GDB_RUN_MODE); - connect(runTool, &MemcheckToolRunner::starting, - this, [this, runTool] { engineStarting(runTool); }); connect(runTool, &MemcheckToolRunner::parserError, this, &MemcheckTool::parserError); connect(runTool, &MemcheckToolRunner::internalParserError, this, &MemcheckTool::internalParserError); connect(runTool, &MemcheckToolRunner::stopped, this, &MemcheckTool::engineFinished); @@ -576,11 +573,6 @@ RunWorker *MemcheckTool::createRunWorker(RunControl *runControl) m_toolBusy = true; updateRunActions(); - return runTool; -} - -void MemcheckTool::engineStarting(const MemcheckToolRunner *runTool) -{ setBusyCursor(true); clearErrorView(); m_loadExternalLogFile->setDisabled(true); @@ -601,6 +593,8 @@ void MemcheckTool::engineStarting(const MemcheckToolRunner *runTool) }); m_suppressionActions.append(action); } + + return runTool; } void MemcheckTool::loadExternalXmlLogFile() |