diff options
author | Arttu Tarkiainen <arttu.tarkiainen@qt.io> | 2021-11-16 17:23:30 +0200 |
---|---|---|
committer | Arttu Tarkiainen <arttu.tarkiainen@qt.io> | 2021-12-21 10:16:32 +0200 |
commit | 347d9cd8525a0baadbd4822465248905877babd3 (patch) | |
tree | a9a7aa98f32008cfdc11132842fe63eb7113be05 /src/libs | |
parent | 7ba80f53f3f493e6b66c9867a9f31cbdcf364466 (diff) |
Fix deprecation warnings from building with Qt 5.15.2
Also update build version requirements.
Task-number: QTIFW-2388
Change-Id: Iae1949548dda7a3b8d448228e27060efb5abd8eb
Reviewed-by: Katja Marttila <katja.marttila@qt.io>
Diffstat (limited to 'src/libs')
-rw-r--r-- | src/libs/ifwtools/repositorygen.cpp | 6 | ||||
-rw-r--r-- | src/libs/installer/abstractarchive.h | 1 | ||||
-rw-r--r-- | src/libs/installer/component.cpp | 10 | ||||
-rw-r--r-- | src/libs/installer/componentmodel.cpp | 4 | ||||
-rw-r--r-- | src/libs/installer/createdesktopentryoperation.cpp | 2 | ||||
-rw-r--r-- | src/libs/installer/fakestopprocessforupdateoperation.cpp | 2 | ||||
-rw-r--r-- | src/libs/installer/fileutils.cpp | 4 | ||||
-rw-r--r-- | src/libs/installer/graph.h | 2 | ||||
-rw-r--r-- | src/libs/installer/installercalculator.cpp | 3 | ||||
-rw-r--r-- | src/libs/installer/installiconsoperation.cpp | 2 | ||||
-rw-r--r-- | src/libs/installer/packagemanagercore.cpp | 13 | ||||
-rw-r--r-- | src/libs/installer/settings.cpp | 16 | ||||
-rw-r--r-- | src/libs/installer/sysinfo_win.cpp | 2 | ||||
-rw-r--r-- | src/libs/installer/uninstallercalculator.cpp | 7 | ||||
-rw-r--r-- | src/libs/kdtools/kdsysinfo_win.cpp | 2 | ||||
-rw-r--r-- | src/libs/kdtools/localpackagehub.cpp | 4 | ||||
-rw-r--r-- | src/libs/kdtools/sysinfo.cpp | 2 | ||||
-rw-r--r-- | src/libs/kdtools/sysinfo_x11.cpp | 4 | ||||
-rw-r--r-- | src/libs/kdtools/updatefinder.cpp | 2 |
19 files changed, 49 insertions, 39 deletions
diff --git a/src/libs/ifwtools/repositorygen.cpp b/src/libs/ifwtools/repositorygen.cpp index a045c5248..874d88494 100644 --- a/src/libs/ifwtools/repositorygen.cpp +++ b/src/libs/ifwtools/repositorygen.cpp @@ -525,7 +525,7 @@ PackageInfoVector QInstallerTools::createListOfPackages(const QStringList &packa .arg(QDir::toNativeSeparators(fileInfo.absoluteFilePath()), info.version)); } info.dependencies = packageElement.firstChildElement(QLatin1String("Dependencies")).text() - .split(QInstaller::commaRegExp(), QString::SkipEmptyParts); + .split(QInstaller::commaRegExp(), Qt::SkipEmptyParts); info.directory = it->filePath(); if (packagesUpdatedWithSha.contains(info.name)) { info.createContentSha1Node = true; @@ -663,10 +663,10 @@ PackageInfoVector QInstallerTools::createListOfRepositoryPackages(const QStringL const QDomElement c2Element = c2.at(j).toElement(); if (c2Element.tagName() == QInstaller::scDependencies) info.dependencies = c2Element.text() - .split(QInstaller::commaRegExp(), QString::SkipEmptyParts); + .split(QInstaller::commaRegExp(), Qt::SkipEmptyParts); else if (c2Element.tagName() == QInstaller::scDownloadableArchives) { QStringList names = c2Element.text() - .split(QInstaller::commaRegExp(), QString::SkipEmptyParts); + .split(QInstaller::commaRegExp(), Qt::SkipEmptyParts); foreach (const QString &name, names) { info.copiedFiles.append(QString::fromLatin1("%1/%3%2").arg(info.directory, name, info.version)); diff --git a/src/libs/installer/abstractarchive.h b/src/libs/installer/abstractarchive.h index 77cc35b61..55489eb14 100644 --- a/src/libs/installer/abstractarchive.h +++ b/src/libs/installer/abstractarchive.h @@ -49,7 +49,6 @@ struct INSTALLER_EXPORT ArchiveEntry , compressedSize(0) , uncompressedSize(0) , permissions_mode(0) - , permissions_enum(0) {} QString path; diff --git a/src/libs/installer/component.cpp b/src/libs/installer/component.cpp index 1a83291c1..21daa795c 100644 --- a/src/libs/installer/component.cpp +++ b/src/libs/installer/component.cpp @@ -352,12 +352,12 @@ void Component::loadDataFromPackage(const Package &package) setLocalTempPath(QInstaller::pathFromUrl(package.packageSource().url)); const QStringList uis = package.data(QLatin1String("UserInterfaces")).toString() - .split(QInstaller::commaRegExp(), QString::SkipEmptyParts); + .split(QInstaller::commaRegExp(), Qt::SkipEmptyParts); if (!uis.isEmpty()) loadUserInterfaces(QDir(QString::fromLatin1("%1/%2").arg(localTempPath(), name())), uis); #ifndef IFW_DISABLE_TRANSLATIONS const QStringList qms = package.data(QLatin1String("Translations")).toString() - .split(QInstaller::commaRegExp(), QString::SkipEmptyParts); + .split(QInstaller::commaRegExp(), Qt::SkipEmptyParts); if (!qms.isEmpty()) loadTranslations(QDir(QString::fromLatin1("%1/%2").arg(localTempPath(), name())), qms); #endif @@ -1329,7 +1329,7 @@ void Component::addDependency(const QString &newDependency) QStringList Component::dependencies() const { - return d->m_vars.value(scDependencies).split(QInstaller::commaRegExp(), QString::SkipEmptyParts); + return d->m_vars.value(scDependencies).split(QInstaller::commaRegExp(), Qt::SkipEmptyParts); } /*! @@ -1352,7 +1352,7 @@ void Component::addAutoDependOn(const QString &newDependOn) QStringList Component::autoDependencies() const { - return d->m_vars.value(scAutoDependOn).split(QInstaller::commaRegExp(), QString::SkipEmptyParts); + return d->m_vars.value(scAutoDependOn).split(QInstaller::commaRegExp(), Qt::SkipEmptyParts); } /*! @@ -1383,7 +1383,7 @@ bool Component::isAutoDependOn(const QSet<QString> &componentsToInstall) const // essential updates needs to be installed first, otherwise non-essential components // will be installed if (packageManagerCore()->foundEssentialUpdate()) { - const QSet<QString> autoDependOnSet = autoDependOnList.toSet(); + const QSet<QString> autoDependOnSet(autoDependOnList.begin(), autoDependOnList.end()); if (componentsToInstall.contains(autoDependOnSet)) { foreach (const QString &autoDep, autoDependOnSet) { Component *component = packageManagerCore()->componentByName(autoDep); diff --git a/src/libs/installer/componentmodel.cpp b/src/libs/installer/componentmodel.cpp index 56ee57fc6..3862f8dc5 100644 --- a/src/libs/installer/componentmodel.cpp +++ b/src/libs/installer/componentmodel.cpp @@ -252,7 +252,9 @@ bool ComponentModel::setData(const QModelIndex &index, const QVariant &value, in if (role == Qt::CheckStateRole) { if (index.column() != 0) return false; - ComponentSet nodes = component->childItems().toSet(); + + const QList<Component*> childItems = component->childItems(); + ComponentSet nodes(childItems.begin(), childItems.end()); Qt::CheckState newValue = Qt::CheckState(value.toInt()); if (newValue == Qt::PartiallyChecked) { const Qt::CheckState oldValue = component->checkState(); diff --git a/src/libs/installer/createdesktopentryoperation.cpp b/src/libs/installer/createdesktopentryoperation.cpp index 1a56c193c..6c92cc178 100644 --- a/src/libs/installer/createdesktopentryoperation.cpp +++ b/src/libs/installer/createdesktopentryoperation.cpp @@ -60,7 +60,7 @@ QString CreateDesktopEntryOperation::absoluteFileName() QStringList XDG_DATA_HOME = QString::fromLocal8Bit(qgetenv("XDG_DATA_HOME")) .split(QLatin1Char(':'), - QString::SkipEmptyParts); + Qt::SkipEmptyParts); XDG_DATA_HOME.push_back(QDir::home().absoluteFilePath(QLatin1String(".local/share"))); // default user-specific path diff --git a/src/libs/installer/fakestopprocessforupdateoperation.cpp b/src/libs/installer/fakestopprocessforupdateoperation.cpp index 36892eac7..67d60a92f 100644 --- a/src/libs/installer/fakestopprocessforupdateoperation.cpp +++ b/src/libs/installer/fakestopprocessforupdateoperation.cpp @@ -68,7 +68,7 @@ bool FakeStopProcessForUpdateOperation::undoOperation() return false; } - QStringList processes = arguments().at(0).split(QLatin1Char(','), QString::SkipEmptyParts); + QStringList processes = arguments().at(0).split(QLatin1Char(','), Qt::SkipEmptyParts); for (int i = processes.count() - 1; i >= 0; --i) { if (!core->isProcessRunning(processes.at(i))) processes.removeAt(i); diff --git a/src/libs/installer/fileutils.cpp b/src/libs/installer/fileutils.cpp index 8c015a1c2..d534b3651 100644 --- a/src/libs/installer/fileutils.cpp +++ b/src/libs/installer/fileutils.cpp @@ -78,7 +78,7 @@ TempDirDeleter::TempDirDeleter(const QString &path) } TempDirDeleter::TempDirDeleter(const QStringList &paths) - : m_paths(paths.toSet()) + : m_paths(QSet<QString>(paths.begin(), paths.end())) { } @@ -99,7 +99,7 @@ void TempDirDeleter::add(const QString &path) void TempDirDeleter::add(const QStringList &paths) { - m_paths += paths.toSet(); + m_paths += QSet<QString>(paths.begin(), paths.end()); } void TempDirDeleter::releaseAndDeleteAll() diff --git a/src/libs/installer/graph.h b/src/libs/installer/graph.h index 5f330e69b..49da9c2ed 100644 --- a/src/libs/installer/graph.h +++ b/src/libs/installer/graph.h @@ -63,7 +63,7 @@ public: QList<T> edges(const T &node) const { - return m_graph.value(node).toList(); + return m_graph.value(node).values(); } void addEdge(const T &node, const T &edge) diff --git a/src/libs/installer/installercalculator.cpp b/src/libs/installer/installercalculator.cpp index 363837dd1..ba8bf9db3 100644 --- a/src/libs/installer/installercalculator.cpp +++ b/src/libs/installer/installercalculator.cpp @@ -164,7 +164,8 @@ bool InstallerCalculator::appendComponentsToInstall(const QList<Component *> &co bool InstallerCalculator::appendComponentToInstall(Component *component, const QString &version) { - QSet<QString> allDependencies = component->dependencies().toSet(); + const QStringList dependenciesList = component->dependencies(); + QSet<QString> allDependencies(dependenciesList.begin(), dependenciesList.end()); QString requiredDependencyVersion = version; foreach (const QString &dependencyComponentName, allDependencies) { // PackageManagerCore::componentByName returns 0 if dependencyComponentName contains a diff --git a/src/libs/installer/installiconsoperation.cpp b/src/libs/installer/installiconsoperation.cpp index cfb5a9536..5927ebf74 100644 --- a/src/libs/installer/installiconsoperation.cpp +++ b/src/libs/installer/installiconsoperation.cpp @@ -54,7 +54,7 @@ QString InstallIconsOperation::targetDirectory() QStringList XDG_DATA_HOME = QString::fromLocal8Bit(qgetenv("XDG_DATA_HOME")) .split(QLatin1Char(':'), - QString::SkipEmptyParts); + Qt::SkipEmptyParts); XDG_DATA_HOME.push_back(QDir::home().absoluteFilePath(QLatin1String(".local/share"))); // default user-specific path if (AdminAuthorization::hasAdminRights() || RemoteClient::instance().isActive()) diff --git a/src/libs/installer/packagemanagercore.cpp b/src/libs/installer/packagemanagercore.cpp index 75cd4a4cf..e17444be6 100644 --- a/src/libs/installer/packagemanagercore.cpp +++ b/src/libs/installer/packagemanagercore.cpp @@ -1225,7 +1225,8 @@ PackageManagerCore::PackageManagerCore(qint64 magicmaker, const QList<OperationB // Sanity check to detect a broken installations with missing operations. // Every installed package should have at least one MinimalProgress operation. // - QSet<QString> installedPackages = d->m_core->localInstalledPackages().keys().toSet(); + const QStringList localPackageList = d->m_core->localInstalledPackages().keys(); + QSet<QString> installedPackages(localPackageList.begin(), localPackageList.end()); QSet<QString> operationPackages; foreach (QInstaller::Operation *operation, d->m_performedOperationsOld) { if (operation->hasValue(QLatin1String("component"))) @@ -2110,7 +2111,9 @@ bool PackageManagerCore::calculateComponentsToUninstall() const emit aboutCalculateComponentsToUninstall(); if (!isUpdater()) { // hack to avoid removing needed dependencies - QSet<Component*> componentsToInstall = d->installerCalculator()->orderedComponentsToInstall().toSet(); + const QList<Component *> componentsToInstallList + = d->installerCalculator()->orderedComponentsToInstall(); + QSet<Component*> componentsToInstall(componentsToInstallList.begin(), componentsToInstallList.end()); QList<Component*> componentsToUninstall; foreach (Component *component, components(ComponentType::All)) { @@ -3684,7 +3687,7 @@ bool PackageManagerCore::updateComponentData(struct Data &data, Component *compo // add downloadable archive from xml const QStringList downloadableArchives = data.package->data(scDownloadableArchives).toString() - .split(QInstaller::commaRegExp(), QString::SkipEmptyParts); + .split(QInstaller::commaRegExp(), Qt::SkipEmptyParts); if (component->isFromOnlineRepository()) { foreach (const QString downloadableArchive, downloadableArchives) @@ -3692,7 +3695,7 @@ bool PackageManagerCore::updateComponentData(struct Data &data, Component *compo } const QStringList componentsToReplace = data.package->data(scReplaces).toString() - .split(QInstaller::commaRegExp(), QString::SkipEmptyParts); + .split(QInstaller::commaRegExp(), Qt::SkipEmptyParts); if (!componentsToReplace.isEmpty()) { // Store the component (this is a component that replaces others) and all components that @@ -3873,7 +3876,7 @@ bool PackageManagerCore::fetchUpdaterPackages(const PackagesList &remotes, const bool isValidUpdate = locals.contains(name); if (!isValidUpdate && !replaces.isEmpty()) { const QStringList possibleNames = replaces.split(QInstaller::commaRegExp(), - QString::SkipEmptyParts); + Qt::SkipEmptyParts); foreach (const QString &possibleName, possibleNames) { if (locals.contains(possibleName)) { isValidUpdate = true; diff --git a/src/libs/installer/settings.cpp b/src/libs/installer/settings.cpp index 537f6060b..c64a33382 100644 --- a/src/libs/installer/settings.cpp +++ b/src/libs/installer/settings.cpp @@ -132,7 +132,7 @@ static QStringList readArgumentAttributes(QXmlStreamReader &reader, Settings::Pa if (reader.isWhitespace()) continue; arguments.append(reader.text().toString().split(QRegularExpression(QLatin1String("\\s+")), - QString::SkipEmptyParts)); + Qt::SkipEmptyParts)); } break; case QXmlStreamReader::EndElement: { @@ -713,7 +713,7 @@ Settings::Update Settings::updateRepositoryCategories(const RepoHash &updates) } } if (update) { - categories = categoriesList.toSet(); + categories = QSet<RepositoryCategory>(categoriesList.begin(), categoriesList.end()); setRepositoryCategories(categories); } return update ? Settings::UpdatesApplied : Settings::NoUpdatesApplied; @@ -764,8 +764,10 @@ Settings::Update Settings::updateDefaultRepositories(const RepoHash &updates) } const bool updated = apply(updates, &defaultRepos); - if (updated) - setDefaultRepositories(defaultRepos.values().toSet()); + if (updated) { + const QList<Repository> repositoriesList = defaultRepos.values(); + setDefaultRepositories(QSet<Repository>(repositoriesList.begin(), repositoriesList.end())); + } return updated ? Settings::UpdatesApplied : Settings::NoUpdatesApplied; } @@ -816,8 +818,10 @@ Settings::Update Settings::updateUserRepositories(const RepoHash &updates) } const bool updated = apply(updates, &reposToUpdate); - if (updated) - setUserRepositories(reposToUpdate.values().toSet()); + if (updated) { + const QList<Repository> repositoriesList = reposToUpdate.values(); + setUserRepositories(QSet<Repository>(repositoriesList.begin(), repositoriesList.end())); + } return updated ? Settings::UpdatesApplied : Settings::NoUpdatesApplied; } diff --git a/src/libs/installer/sysinfo_win.cpp b/src/libs/installer/sysinfo_win.cpp index 508ce7a6e..12ffbcdd7 100644 --- a/src/libs/installer/sysinfo_win.cpp +++ b/src/libs/installer/sysinfo_win.cpp @@ -109,7 +109,7 @@ QList<VolumeInfo> localVolumeInfosFromMountPoints(PTCHAR volumeGUID) TCHAR volumeNames[MAX_PATH + 1] = { 0 }; if (GetVolumePathNamesForVolumeName(volumeGUID, volumeNames, MAX_PATH, &bufferSize)) { QStringList mountedPaths = QString::fromWCharArray(volumeNames, bufferSize).split(QLatin1Char(char(0)), - QString::SkipEmptyParts); + Qt::SkipEmptyParts); foreach (const QString &mountedPath, mountedPaths) { VolumeInfo info; info.setMountPath(mountedPath); diff --git a/src/libs/installer/uninstallercalculator.cpp b/src/libs/installer/uninstallercalculator.cpp index 668ace699..10b44d80a 100644 --- a/src/libs/installer/uninstallercalculator.cpp +++ b/src/libs/installer/uninstallercalculator.cpp @@ -62,8 +62,9 @@ void UninstallerCalculator::appendComponentToUninstall(Component *component) PackageManagerCore *core = component->packageManagerCore(); // remove all already resolved dependees - QSet<Component *> dependees = core->dependees(component).toSet() - .subtract(m_componentsToUninstall); + const QList<Component *> dependeesList = core->dependees(component); + QSet<Component *> dependees = QSet<Component *>(dependeesList.begin(), + dependeesList.end()).subtract(m_componentsToUninstall); foreach (Component *dependee, dependees) appendComponentToUninstall(dependee); @@ -104,7 +105,7 @@ void UninstallerCalculator::appendComponentsToUninstall(const QList<Component*> foreach (Component *c, m_installedComponents) { const QString replaces = c->value(scReplaces); const QStringList possibleNames = replaces.split(QInstaller::commaRegExp(), - QString::SkipEmptyParts) << c->name(); + Qt::SkipEmptyParts) << c->name(); foreach (const QString &possibleName, possibleNames) { Component *cc = PackageManagerCore::componentByName(possibleName, m_installedComponents); diff --git a/src/libs/kdtools/kdsysinfo_win.cpp b/src/libs/kdtools/kdsysinfo_win.cpp index 0aa7e7aa3..465029252 100644 --- a/src/libs/kdtools/kdsysinfo_win.cpp +++ b/src/libs/kdtools/kdsysinfo_win.cpp @@ -68,7 +68,7 @@ QList<ProcessInfo> runningProcesses() char buffer[bufferSize + 1] = { 0 }; if (QSysInfo::windowsVersion() <= QSysInfo::WV_5_2) { const DWORD size = GetLogicalDriveStringsA(bufferSize, buffer); - deviceList = QString::fromLatin1(buffer, size).split(QLatin1Char(char(0)), QString::SkipEmptyParts); + deviceList = QString::fromLatin1(buffer, size).split(QLatin1Char(char(0)), Qt::SkipEmptyParts); } QLibrary kernel32(QLatin1String("Kernel32.dll")); diff --git a/src/libs/kdtools/localpackagehub.cpp b/src/libs/kdtools/localpackagehub.cpp index 2ee880e04..a1c112900 100644 --- a/src/libs/kdtools/localpackagehub.cpp +++ b/src/libs/kdtools/localpackagehub.cpp @@ -488,10 +488,10 @@ void LocalPackageHub::PackagesInfoData::addPackageFrom(const QDomElement &packag info.uncompressedSize = childNodeE.text().toULongLong(); else if (childNodeE.tagName() == QLatin1String("Dependencies")) { info.dependencies = childNodeE.text().split(QInstaller::commaRegExp(), - QString::SkipEmptyParts); + Qt::SkipEmptyParts); } else if (childNodeE.tagName() == QLatin1String("AutoDependOn")) { info.autoDependencies = childNodeE.text().split(QInstaller::commaRegExp(), - QString::SkipEmptyParts); + Qt::SkipEmptyParts); } else if (childNodeE.tagName() == QLatin1String("ForcedInstallation")) info.forcedInstallation = childNodeE.text().toLower() == QLatin1String( "true" ) ? true : false; else if (childNodeE.tagName() == QLatin1String("LastUpdateDate")) diff --git a/src/libs/kdtools/sysinfo.cpp b/src/libs/kdtools/sysinfo.cpp index df9a374f2..9512e59e3 100644 --- a/src/libs/kdtools/sysinfo.cpp +++ b/src/libs/kdtools/sysinfo.cpp @@ -71,7 +71,7 @@ VolumeInfo VolumeInfo::fromPath(const QString &path) } // the target directory does not exist yet, we need to cd up till we find the first existing dir - QStringList parts = targetPath.absolutePath().split(QDir::separator(),QString::SkipEmptyParts); + QStringList parts = targetPath.absolutePath().split(QDir::separator(), Qt::SkipEmptyParts); while (targetPath.absolutePath() != QDir::rootPath()) { if (targetPath.exists()) break; diff --git a/src/libs/kdtools/sysinfo_x11.cpp b/src/libs/kdtools/sysinfo_x11.cpp index bc55939a6..a59fd150b 100644 --- a/src/libs/kdtools/sysinfo_x11.cpp +++ b/src/libs/kdtools/sysinfo_x11.cpp @@ -60,7 +60,7 @@ quint64 installedMemory() else if (s.isEmpty()) return quint64(); - const QStringList parts = s.split(QLatin1Char(' '), QString::SkipEmptyParts); + const QStringList parts = s.split(QLatin1Char(' '), Qt::SkipEmptyParts); return quint64(parts.at(1).toInt() * 1024LL); } #else @@ -99,7 +99,7 @@ QList<VolumeInfo> mountedVolumes() if (!s.startsWith(QLatin1Char('/')) && !s.startsWith(QLatin1String("tmpfs ") + QDir::tempPath())) continue; - const QStringList parts = s.split(QLatin1Char(' '), QString::SkipEmptyParts); + const QStringList parts = s.split(QLatin1Char(' '), Qt::SkipEmptyParts); VolumeInfo v; v.setMountPath(parts.at(1)); diff --git a/src/libs/kdtools/updatefinder.cpp b/src/libs/kdtools/updatefinder.cpp index 5bf33d4a4..ed45b9797 100644 --- a/src/libs/kdtools/updatefinder.cpp +++ b/src/libs/kdtools/updatefinder.cpp @@ -375,7 +375,7 @@ QList<UpdateInfo> UpdateFinder::Private::applicableUpdates(UpdatesInfo *updatesI // Catch hold of app names contained updatesInfo->applicationName() // If the application appName isn't one of the app names, then the updates are not applicable. - const QStringList apps = appName.split(QInstaller::commaRegExp(), QString::SkipEmptyParts); + const QStringList apps = appName.split(QInstaller::commaRegExp(), Qt::SkipEmptyParts); if (apps.indexOf([&packages] { return packages->isValid() ? packages->applicationName() : QCoreApplication::applicationName(); } ()) < 0) { return dummy; |