AbstractRemoteLinuxDeployStep: Make some overrides final

Move some methods into protected section.
Do some cleanup.

Change-Id: Ica6f6fd181334c450666049c10d9ecc1ea16ea5c
Reviewed-by: Christian Kandeler <christian.kandeler@qt.io>

src/plugins/remotelinux/abstractremotelinuxdeploystep.cpp

@@ -9,15 +9,12 @@
 #include <projectexplorer/deployablefile.h>
 #include <projectexplorer/devicesupport/idevice.h>
 #include <projectexplorer/kitinformation.h>
-#include <projectexplorer/projectexplorerconstants.h>
-#include <projectexplorer/target.h>
 
 #include <solutions/tasking/tasktree.h>
 
 #include <utils/qtcassert.h>
 
 #include <QDateTime>
-#include <QPointer>
 
 using namespace ProjectExplorer;
 using namespace Tasking;
@@ -55,7 +52,7 @@ IDevice::ConstPtr AbstractRemoteLinuxDeployStep::deviceConfiguration() const
 }
 
 void AbstractRemoteLinuxDeployStep::saveDeploymentTimeStamp(const DeployableFile &deployableFile,
-                                                               const QDateTime &remoteTimestamp)
+                                                            const QDateTime &remoteTimestamp)
 {
     d->deployTimes.saveDeploymentTimeStamp(deployableFile, kit(), remoteTimestamp);
 }

src/plugins/remotelinux/abstractremotelinuxdeploystep.h

@@ -8,8 +8,6 @@
 #include <projectexplorer/buildstep.h>
 #include <projectexplorer/devicesupport/idevicefwd.h>
 
-#include <QObject>
-
 namespace ProjectExplorer { class DeployableFile; }
 namespace Tasking { class GroupItem; }
 
@@ -23,19 +21,17 @@ public:
     explicit AbstractRemoteLinuxDeployStep(ProjectExplorer::BuildStepList *bsl, Utils::Id id);
     ~AbstractRemoteLinuxDeployStep() override;
 
+protected:
     ProjectExplorer::IDeviceConstPtr deviceConfiguration() const;
-
     virtual Utils::expected_str<void> isDeploymentPossible() const;
-
     void handleStdOutData(const QString &data);
     void handleStdErrData(const QString &data);
 
-protected:
+    bool fromMap(const QVariantMap &map) final;
-    bool fromMap(const QVariantMap &map) override;
+    QVariantMap toMap() const final;
-    QVariantMap toMap() const override;
+    bool init() final;
-    bool init() override;
     void doRun() final;
-    void doCancel() override;
+    void doCancel() final;
 
     void setInternalInitializer(const std::function<Utils::expected_str<void>()> &init);
 

src/plugins/remotelinux/rsyncdeploystep.cpp

@@ -176,8 +176,7 @@ GroupItem RsyncDeployStep::transferTask()
 
         transfer.setRsyncFlags(flags());
         transfer.setFilesToTransfer(m_files);
-        connect(&transfer, &FileTransfer::progress,
+        connect(&transfer, &FileTransfer::progress, this, &RsyncDeployStep::handleStdOutData);
-                this, &AbstractRemoteLinuxDeployStep::handleStdOutData);
     };
     const auto errorHandler = [this](const FileTransfer &transfer) {
         const ProcessResultData result = transfer.resultData();