From 0f470982de37d492cdab345a7cfaf916fbc512cf Mon Sep 17 00:00:00 2001
From: hjk <hjk@qt.io>
Date: Mon, 30 May 2022 17:22:21 +0200
Subject: [PATCH] PerfProfiler: Remove now-useless check for process success

The check nowadays always succeeds, the potential failure is handled
in the normal done() handling.

Change-Id: I858633871a6b66817c795e7057964afebf79eb85
Reviewed-by: Jarek Kobus <jaroslaw.kobus@qt.io>
---
 src/plugins/perfprofiler/perfprofilerruncontrol.cpp | 8 ++------
 1 file changed, 2 insertions(+), 6 deletions(-)

diff --git a/src/plugins/perfprofiler/perfprofilerruncontrol.cpp b/src/plugins/perfprofiler/perfprofilerruncontrol.cpp
index e0bf209e274..691d5a3aff0 100644
--- a/src/plugins/perfprofiler/perfprofilerruncontrol.cpp
+++ b/src/plugins/perfprofiler/perfprofilerruncontrol.cpp
@@ -23,9 +23,9 @@
 **
 ****************************************************************************/
 
-#include "perfdatareader.h"
-#include "perfprofilerconstants.h"
 #include "perfprofilerruncontrol.h"
+
+#include "perfdatareader.h"
 #include "perfprofilertool.h"
 #include "perfrunconfigurationaspect.h"
 #include "perfsettings.h"
@@ -124,10 +124,6 @@ public:
     void start() override
     {
         m_process = new QtcProcess(this);
-        if (!m_process) {
-            reportFailure(tr("Could not start device process."));
-            return;
-        }
 
         connect(m_process, &QtcProcess::started, this, &RunWorker::reportStarted);
         connect(m_process, &QtcProcess::done, this, [this] {