ProfProfiler: Simplify setting dependency

Change-Id: I75feca18c52454515c1004e3f82d10fb1aa87d9a
Reviewed-by: hjk <hjk@qt.io>
This commit is contained in:
Jarek Kobus
2025-01-02 16:28:53 +01:00
parent da475f400d
commit 2ddfd1f273

View File

@@ -96,10 +96,9 @@ public:
// If the parser is gone, there is no point in going on.
m_perfParserWorker->setEssential(true);
m_perfRecordWorker = qobject_cast<ProcessRunner *>(runControl->createWorker("PerfRecorder"));
if ((m_perfRecordWorker = qobject_cast<ProcessRunner *>(runControl->createWorker("PerfRecorder")))) {
m_perfParserWorker->addStartDependency(m_perfRecordWorker);
} else {
if (!m_perfRecordWorker) {
m_perfRecordWorker = new ProcessRunner(runControl);
m_perfRecordWorker->suppressDefaultStdOutHandling();
@@ -118,14 +117,13 @@ public:
appendMessage("Starting Perf: " + cmd.toUserOutput(), NormalMessageFormat);
});
m_perfRecordWorker->addStartDependency(m_perfParserWorker);
// In the local case, the parser won't automatically stop when the recorder does. So we need
// to mark the recorder as essential, too.
m_perfRecordWorker->setEssential(true);
}
addStartDependency(m_perfRecordWorker);
m_perfParserWorker->addStartDependency(m_perfRecordWorker);
m_perfParserWorker->addStopDependency(m_perfRecordWorker);
PerfProfilerTool::instance()->onWorkerCreation(runControl);
}