ProjectExplorer: Remove IDevice::createProcessListModel

All re-implementations are the same nowadays.

Change-Id: I954b3c9ca296967a99ea548ade6282fba188249f
Reviewed-by: <github-actions-qt-creator@cristianadam.eu>
Reviewed-by: Qt CI Bot <qt_ci_bot@qt-project.org>
Reviewed-by: Christian Kandeler <christian.kandeler@qt.io>
This commit is contained in:
hjk
2023-06-21 17:20:55 +02:00
parent d045d8dc60
commit a62fcb73ce
9 changed files with 3 additions and 30 deletions

View File

@@ -828,11 +828,6 @@ ProcessInterface *DockerDevice::createProcessInterface() const
return new DockerProcessImpl(this->sharedFromThis(), d); return new DockerProcessImpl(this->sharedFromThis(), d);
} }
ProcessList *DockerDevice::createProcessListModel(QObject *parent) const
{
return new ProcessList(sharedFromThis(), parent);
}
DeviceTester *DockerDevice::createDeviceTester() const DeviceTester *DockerDevice::createDeviceTester() const
{ {
return nullptr; return nullptr;

View File

@@ -74,7 +74,6 @@ public:
Utils::ProcessInterface *createProcessInterface() const override; Utils::ProcessInterface *createProcessInterface() const override;
bool canCreateProcessModel() const override { return true; } bool canCreateProcessModel() const override { return true; }
ProjectExplorer::ProcessList *createProcessListModel(QObject *parent) const override;
bool hasDeviceTester() const override { return false; } bool hasDeviceTester() const override { return false; }
ProjectExplorer::DeviceTester *createDeviceTester() const override; ProjectExplorer::DeviceTester *createDeviceTester() const override;
bool usableAsBuildDevice() const override; bool usableAsBuildDevice() const override;

View File

@@ -89,11 +89,6 @@ bool DesktopDevice::canCreateProcessModel() const
return true; return true;
} }
ProcessList *DesktopDevice::createProcessListModel(QObject *parent) const
{
return new ProcessList(sharedFromThis(), parent);
}
DeviceProcessSignalOperation::Ptr DesktopDevice::signalOperation() const DeviceProcessSignalOperation::Ptr DesktopDevice::signalOperation() const
{ {
return DeviceProcessSignalOperation::Ptr(new DesktopProcessSignalOperation()); return DeviceProcessSignalOperation::Ptr(new DesktopProcessSignalOperation());

View File

@@ -26,7 +26,6 @@ public:
IDeviceWidget *createWidget() override; IDeviceWidget *createWidget() override;
bool canCreateProcessModel() const override; bool canCreateProcessModel() const override;
ProcessList *createProcessListModel(QObject *parent) const override;
DeviceProcessSignalOperation::Ptr signalOperation() const override; DeviceProcessSignalOperation::Ptr signalOperation() const override;
QUrl toolControlChannel(const ControlChannelHint &) const override; QUrl toolControlChannel(const ControlChannelHint &) const override;
bool usableAsBuildDevice() const override; bool usableAsBuildDevice() const override;

View File

@@ -187,7 +187,8 @@ void DeviceProcessesDialogPrivate::setDevice(const IDevice::ConstPtr &device)
if (!device) if (!device)
return; return;
processList.reset(device->createProcessListModel()); processList.reset(new ProcessList(device->sharedFromThis(), this));
QTC_ASSERT(processList, return); QTC_ASSERT(processList, return);
proxyModel.setSourceModel(processList->model()); proxyModel.setSourceModel(processList->model());

View File

@@ -391,13 +391,6 @@ PortsGatheringMethod IDevice::portsGatheringMethod() const
return {filePath("netstat"), {"-a", "-n"}}; return {filePath("netstat"), {"-a", "-n"}};
}, },
&Port::parseFromCommandOutput}; &Port::parseFromCommandOutput};
};
ProcessList *IDevice::createProcessListModel(QObject *parent) const
{
Q_UNUSED(parent)
QTC_ASSERT(false, qDebug("This should not have been called..."); return nullptr);
return nullptr;
} }
DeviceTester *IDevice::createDeviceTester() const DeviceTester *IDevice::createDeviceTester() const

View File

@@ -32,16 +32,14 @@ namespace Utils {
class CommandLine; class CommandLine;
class DeviceFileAccess; class DeviceFileAccess;
class Environment; class Environment;
class Icon;
class PortList; class PortList;
class Port; class Port;
class ProcessInterface;
class Process; class Process;
class ProcessInterface;
} // Utils } // Utils
namespace ProjectExplorer { namespace ProjectExplorer {
class ProcessList;
class FileTransferInterface; class FileTransferInterface;
class FileTransferSetupData; class FileTransferSetupData;
class Kit; class Kit;
@@ -142,7 +140,6 @@ public:
virtual PortsGatheringMethod portsGatheringMethod() const; virtual PortsGatheringMethod portsGatheringMethod() const;
virtual bool canCreateProcessModel() const { return false; } virtual bool canCreateProcessModel() const { return false; }
virtual ProcessList *createProcessListModel(QObject *parent = nullptr) const;
virtual bool hasDeviceTester() const { return false; } virtual bool hasDeviceTester() const { return false; }
virtual DeviceTester *createDeviceTester() const; virtual DeviceTester *createDeviceTester() const;

View File

@@ -995,11 +995,6 @@ IDeviceWidget *LinuxDevice::createWidget()
return new Internal::GenericLinuxDeviceConfigurationWidget(sharedFromThis()); return new Internal::GenericLinuxDeviceConfigurationWidget(sharedFromThis());
} }
ProcessList *LinuxDevice::createProcessListModel(QObject *parent) const
{
return new ProcessList(sharedFromThis(), parent);
}
DeviceTester *LinuxDevice::createDeviceTester() const DeviceTester *LinuxDevice::createDeviceTester() const
{ {
return new GenericLinuxDeviceTester; return new GenericLinuxDeviceTester;

View File

@@ -23,7 +23,6 @@ public:
ProjectExplorer::IDeviceWidget *createWidget() override; ProjectExplorer::IDeviceWidget *createWidget() override;
bool canCreateProcessModel() const override { return true; } bool canCreateProcessModel() const override { return true; }
ProjectExplorer::ProcessList *createProcessListModel(QObject *parent) const override;
bool hasDeviceTester() const override { return true; } bool hasDeviceTester() const override { return true; }
ProjectExplorer::DeviceTester *createDeviceTester() const override; ProjectExplorer::DeviceTester *createDeviceTester() const override;
ProjectExplorer::DeviceProcessSignalOperation::Ptr signalOperation() const override; ProjectExplorer::DeviceProcessSignalOperation::Ptr signalOperation() const override;