IDevice: Provide default implementation for signalOperation()

Don't repeat implementation returning nullptr in derived classes.

Change-Id: I437f5d206464bf92706cd135d793438f46455ad6
Reviewed-by: hjk <hjk@qt.io>
Reviewed-by: <github-actions-qt-creator@cristianadam.eu>

src/plugins/baremetal/baremetaldevice.cpp

@@ -86,11 +86,6 @@ QVariantMap BareMetalDevice::toMap() const
     return map;
 }
 
-DeviceProcessSignalOperation::Ptr BareMetalDevice::signalOperation() const
-{
-    return DeviceProcessSignalOperation::Ptr();
-}
-
 IDeviceWidget *BareMetalDevice::createWidget()
 {
     return new BareMetalDeviceConfigurationWidget(sharedFromThis());

src/plugins/baremetal/baremetaldevice.h

@@ -26,8 +26,6 @@ public:
 
     ProjectExplorer::IDeviceWidget *createWidget() final;
 
-    ProjectExplorer::DeviceProcessSignalOperation::Ptr signalOperation() const final;
-
     QString debugServerProviderId() const;
     void setDebugServerProviderId(const QString &id);
     void unregisterDebugServerProvider(IDebugServerProvider *provider);

src/plugins/docker/dockerdevice.cpp

@@ -756,16 +756,6 @@ DeviceTester *DockerDevice::createDeviceTester() const
     return nullptr;
 }
 
-DeviceProcessSignalOperation::Ptr DockerDevice::signalOperation() const
-{
-    return DeviceProcessSignalOperation::Ptr();
-}
-
-DeviceEnvironmentFetcher::Ptr DockerDevice::environmentFetcher() const
-{
-    return DeviceEnvironmentFetcher::Ptr();
-}
-
 bool DockerDevice::usableAsBuildDevice() const
 {
     return true;

src/plugins/docker/dockerdevice.h

@@ -76,8 +76,6 @@ public:
     ProjectExplorer::DeviceProcessList *createProcessListModel(QObject *parent) const override;
     bool hasDeviceTester() const override { return false; }
     ProjectExplorer::DeviceTester *createDeviceTester() const override;
-    ProjectExplorer::DeviceProcessSignalOperation::Ptr signalOperation() const override;
-    ProjectExplorer::DeviceEnvironmentFetcher::Ptr environmentFetcher() const override;
     bool usableAsBuildDevice() const override;
 
     Utils::FilePath mapToGlobalPath(const Utils::FilePath &pathOnDevice) const override;

src/plugins/ios/iosdevice.cpp

@@ -123,11 +123,6 @@ IDeviceWidget *IosDevice::createWidget()
     return new IosDeviceInfoWidget(sharedFromThis());
 }
 
-DeviceProcessSignalOperation::Ptr IosDevice::signalOperation() const
-{
-    return DeviceProcessSignalOperation::Ptr();
-}
-
 void IosDevice::fromMap(const QVariantMap &map)
 {
     IDevice::fromMap(map);

src/plugins/ios/iosdevice.h

@@ -31,7 +31,6 @@ public:
 
     ProjectExplorer::IDevice::DeviceInfo deviceInformation() const override;
     ProjectExplorer::IDeviceWidget *createWidget() override;
-    ProjectExplorer::DeviceProcessSignalOperation::Ptr signalOperation() const override;
 
     QString deviceName() const;
     QString uniqueDeviceID() const;

src/plugins/ios/iossimulator.cpp

@@ -47,11 +47,6 @@ IDeviceWidget *IosSimulator::createWidget()
     return nullptr;
 }
 
-DeviceProcessSignalOperation::Ptr IosSimulator::signalOperation() const
-{
-    return DeviceProcessSignalOperation::Ptr();
-}
-
 Utils::Port IosSimulator::nextPort() const
 {
     for (int i = 0; i < 100; ++i) {

src/plugins/ios/iossimulator.h

@@ -50,7 +50,6 @@ public:
     ProjectExplorer::IDevice::DeviceInfo deviceInformation() const override;
 
     ProjectExplorer::IDeviceWidget *createWidget() override;
-    ProjectExplorer::DeviceProcessSignalOperation::Ptr signalOperation() const override;
     Utils::Port nextPort() const;
     bool canAutoDetectPorts() const override;
 

src/plugins/projectexplorer/devicesupport/devicemanager.cpp

@@ -522,10 +522,6 @@ public:
     static Utils::Id testTypeId() { return "TestType"; }
 private:
     IDeviceWidget *createWidget() override { return nullptr; }
-    DeviceProcessSignalOperation::Ptr signalOperation() const override
-    {
-        return DeviceProcessSignalOperation::Ptr();
-    }
 };
 
 class TestDeviceFactory final : public IDeviceFactory

src/plugins/projectexplorer/devicesupport/idevice.cpp

@@ -396,9 +396,14 @@ OsType IDevice::osType() const
     return d->osType;
 }
 
+DeviceProcessSignalOperation::Ptr IDevice::signalOperation() const
+{
+    return {};
+}
+
 DeviceEnvironmentFetcher::Ptr IDevice::environmentFetcher() const
 {
-    return DeviceEnvironmentFetcher::Ptr();
+    return {};
 }
 
 IDevice::DeviceState IDevice::deviceState() const

src/plugins/projectexplorer/devicesupport/idevice.h

@@ -162,7 +162,7 @@ public:
     virtual bool hasDeviceTester() const { return false; }
     virtual DeviceTester *createDeviceTester() const;
 
-    virtual DeviceProcessSignalOperation::Ptr signalOperation() const = 0;
+    virtual DeviceProcessSignalOperation::Ptr signalOperation() const;
     virtual DeviceEnvironmentFetcher::Ptr environmentFetcher() const;
 
     enum DeviceState { DeviceReadyToUse, DeviceConnected, DeviceDisconnected, DeviceStateUnknown };