diff options
author | Katja Marttila <katja.marttila@qt.io> | 2022-11-28 13:30:21 +0200 |
---|---|---|
committer | Katja Marttila <katja.marttila@qt.io> | 2022-11-30 15:03:46 +0200 |
commit | 51e02b08abb44131929dba2d86860d10ca17cf41 (patch) | |
tree | c74ea513a2385f430c8a2491fcdebf08e39c2937 /src/libs/installer/uninstallercalculator.h | |
parent | f02c3a2f2a6a5db84a947ca6d4f4ab5f5d867d41 (diff) |
Cleanup installer calculator classes
As there are several optimizations done, the single click of component
in component selection tree is not slow anymore. Cleaning the code so
it is easier to maintain.
Basically this revers commit a28cf55b5a5007c0dd952b3012c076d9da329f0f
but as there are bug fixes made after that so pure revert could not
be done.
Task-number: QTIFW-2885
Change-Id: Id486d5dc68c42c31b4848cd19a1761bcfe242db6
Reviewed-by: Arttu Tarkiainen <arttu.tarkiainen@qt.io>
Diffstat (limited to 'src/libs/installer/uninstallercalculator.h')
-rw-r--r-- | src/libs/installer/uninstallercalculator.h | 14 |
1 files changed, 6 insertions, 8 deletions
diff --git a/src/libs/installer/uninstallercalculator.h b/src/libs/installer/uninstallercalculator.h index 57f2f926a..74207e005 100644 --- a/src/libs/installer/uninstallercalculator.h +++ b/src/libs/installer/uninstallercalculator.h @@ -53,28 +53,26 @@ public: AutoDependent // "Removed as autodependency component is removed" }; - UninstallerCalculator(const QList<Component *> &installedComponents, PackageManagerCore *core, + UninstallerCalculator(PackageManagerCore *core, const AutoDependencyHash &autoDependencyComponentHash, const LocalDependencyHash &localDependencyComponentHash, const QStringList &localVirtualComponents); QSet<Component*> componentsToUninstall() const; - void appendComponentsToUninstall(const QList<Component*> &components, const bool reverse = false); - void removeComponentsFromUnInstall(const QList<Component*> &components); + void appendComponentsToUninstall(const QList<Component*> &components); void insertUninstallReason(Component *component, const UninstallReasonType uninstallReason, const QString &referencedComponentName = QString()); QString uninstallReason(Component *component) const; UninstallerCalculator::UninstallReasonType uninstallReasonType(Component *c) const; - QString uninstallReasonReferencedComponent(Component *component) const; - bool isRequiredVirtualPackage(Component *component); - void appendVirtualComponentsToUninstall(const bool reverse); private: - void appendComponentToUninstall(Component *component, const bool reverse); + QString uninstallReasonReferencedComponent(Component *component) const; + bool isRequiredVirtualPackage(Component *component); + void appendComponentToUninstall(Component *component); + void appendVirtualComponentsToUninstall(); - QList<Component *> m_installedComponents; QSet<Component *> m_componentsToUninstall; PackageManagerCore *m_core; QHash<QString, QPair<UninstallReasonType, QString> > m_toUninstallComponentIdReasonHash; |