ProjectExplorer: Replace RunConfiguration::doAdditionalSetup

... by an explicit update() call. This is what effectively done in
most cases, and should be harmless and reasonably cheap in most other
cases.

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

src/plugins/boot2qt/qdbrunconfiguration.cpp

@@ -102,11 +102,6 @@ Tasks QdbRunConfiguration::checkForIssues() const
     return tasks;
 }
 
-void QdbRunConfiguration::doAdditionalSetup(const RunConfigurationCreationInfo &)
-{
-    updateTargetInformation();
-}
-
 void QdbRunConfiguration::updateTargetInformation()
 {
     const BuildTargetInfo bti = buildTargetInfo();

src/plugins/boot2qt/qdbrunconfiguration.h

@@ -47,7 +47,6 @@ public:
 
 private:
     ProjectExplorer::Tasks checkForIssues() const override;
-    void doAdditionalSetup(const ProjectExplorer::RunConfigurationCreationInfo &) override;
     void updateTargetInformation();
     QString defaultDisplayName() const;
 };

src/plugins/ios/iosrunconfiguration.cpp

@@ -303,11 +303,6 @@ void IosDeviceTypeAspect::setDeviceType(const IosDeviceType &deviceType)
     m_deviceType = deviceType;
 }
 
-void IosRunConfiguration::doAdditionalSetup(const RunConfigurationCreationInfo &)
-{
-    update();
-}
-
 IosDeviceTypeAspect::IosDeviceTypeAspect(IosRunConfiguration *runConfiguration)
     : m_runConfiguration(runConfiguration)
 {

src/plugins/ios/iosrunconfiguration.h

@@ -50,8 +50,6 @@ public:
     QString disabledReason() const override;
     IosDeviceType deviceType() const;
 
-    void doAdditionalSetup(const ProjectExplorer::RunConfigurationCreationInfo &) override;
-
 private:
     bool isEnabled() const final;
 

src/plugins/projectexplorer/desktoprunconfiguration.cpp

@@ -149,11 +149,6 @@ bool DesktopRunConfiguration::fromMap(const QVariantMap &map)
     return true;
 }
 
-void DesktopRunConfiguration::doAdditionalSetup(const RunConfigurationCreationInfo &)
-{
-    updateTargetInformation();
-}
-
 Utils::FilePath DesktopRunConfiguration::executableToRun(const BuildTargetInfo &targetInfo) const
 {
     const FilePath appInBuildDir = targetInfo.targetFilePath;

src/plugins/projectexplorer/desktoprunconfiguration.h

@@ -41,7 +41,6 @@ protected:
     DesktopRunConfiguration(Target *target, Core::Id id, Kind kind);
 
 private:
-    void doAdditionalSetup(const RunConfigurationCreationInfo &info) final;
     bool fromMap(const QVariantMap &map) final;
 
     void updateTargetInformation();

src/plugins/projectexplorer/runconfiguration.cpp

@@ -550,7 +550,7 @@ RunConfiguration *RunConfigurationCreationInfo::create(Target *target) const
         return nullptr;
 
     rc->m_buildKey = buildKey;
-    rc->doAdditionalSetup(*this);
+    rc->update();
     rc->setDisplayName(displayName);
 
     return rc;

src/plugins/projectexplorer/runconfiguration.h

@@ -186,8 +186,6 @@ protected:
     using Updater = std::function<void()>;
     void setUpdater(const Updater &updater);
 
-    virtual void doAdditionalSetup(const RunConfigurationCreationInfo &) {}
-
     Task createConfigurationIssue(const QString &description) const;
 
 private:

src/plugins/python/pythonrunconfiguration.cpp

@@ -289,11 +289,6 @@ PythonRunConfiguration::PythonRunConfiguration(Target *target, Core::Id id)
     connect(target, &Target::buildSystemUpdated, this, &RunConfiguration::update);
 }
 
-void PythonRunConfiguration::doAdditionalSetup(const RunConfigurationCreationInfo &)
-{
-    update();
-}
-
 void PythonRunConfiguration::updateLanguageServer()
 {
     using namespace LanguageClient;

src/plugins/python/pythonrunconfiguration.h

@@ -45,7 +45,6 @@ public:
     QString interpreter() const;
 
 private:
-    void doAdditionalSetup(const ProjectExplorer::RunConfigurationCreationInfo &) final;
     void updateLanguageServer();
 
     bool supportsDebugger() const;

src/plugins/qnx/qnxrunconfiguration.cpp

@@ -88,11 +88,6 @@ Runnable QnxRunConfiguration::runnable() const
     return r;
 }
 
-void QnxRunConfiguration::doAdditionalSetup(const RunConfigurationCreationInfo &)
-{
-    updateTargetInformation();
-}
-
 void QnxRunConfiguration::updateTargetInformation()
 {
     const BuildTargetInfo bti = buildTargetInfo();

src/plugins/qnx/qnxrunconfiguration.h

@@ -48,7 +48,6 @@ public:
 
 private:
     ProjectExplorer::Runnable runnable() const override;
-    void doAdditionalSetup(const ProjectExplorer::RunConfigurationCreationInfo &) override;
 
     void updateTargetInformation();
 };

src/plugins/remotelinux/remotelinuxrunconfiguration.cpp

@@ -75,11 +75,6 @@ RemoteLinuxRunConfiguration::RemoteLinuxRunConfiguration(Target *target, Core::I
     connect(target, &Target::kitChanged, this, &RunConfiguration::update);
 }
 
-void RemoteLinuxRunConfiguration::doAdditionalSetup(const RunConfigurationCreationInfo &)
-{
-    updateTargetInformation();
-}
-
 Runnable RemoteLinuxRunConfiguration::runnable() const
 {
     Runnable r = RunConfiguration::runnable();

src/plugins/remotelinux/remotelinuxrunconfiguration.h

@@ -40,7 +40,6 @@ public:
     RemoteLinuxRunConfiguration(ProjectExplorer::Target *target, Core::Id id);
 
 private:
-    void doAdditionalSetup(const ProjectExplorer::RunConfigurationCreationInfo &) override;
     ProjectExplorer::Runnable runnable() const override;
     void updateTargetInformation();
 };