diff options
-rw-r--r-- | src/plugins/qtsupport/exampleslistmodel.cpp | 5 | ||||
-rw-r--r-- | src/plugins/qtsupport/qtprojectimporter.cpp | 7 | ||||
-rw-r--r-- | src/plugins/qtsupport/qtversionmanager.cpp | 11 | ||||
-rw-r--r-- | src/plugins/qtsupport/qtversionmanager.h | 1 | ||||
-rw-r--r-- | src/plugins/winrt/winrtdevicefactory.cpp | 5 |
5 files changed, 9 insertions, 20 deletions
diff --git a/src/plugins/qtsupport/exampleslistmodel.cpp b/src/plugins/qtsupport/exampleslistmodel.cpp index ad88fe6358..0b9dba32a9 100644 --- a/src/plugins/qtsupport/exampleslistmodel.cpp +++ b/src/plugins/qtsupport/exampleslistmodel.cpp @@ -487,9 +487,10 @@ void ExamplesListModel::updateExamples() void ExamplesListModel::updateQtVersions() { - QList<BaseQtVersion*> versions = QtVersionManager::validVersions([](const BaseQtVersion *v) { + QList<BaseQtVersion*> versions + = QtVersionManager::sortVersions(QtVersionManager::validVersions([](const BaseQtVersion *v) { return v->hasExamples() || v->hasDemos(); - }); + })); // prioritize default qt version ProjectExplorer::Kit *defaultKit = ProjectExplorer::KitManager::defaultKit(); diff --git a/src/plugins/qtsupport/qtprojectimporter.cpp b/src/plugins/qtsupport/qtprojectimporter.cpp index ab18e380f1..e21dc1ef3b 100644 --- a/src/plugins/qtsupport/qtprojectimporter.cpp +++ b/src/plugins/qtsupport/qtprojectimporter.cpp @@ -381,11 +381,8 @@ void QtSupportPlugin::testQtProjectImporter_oneProject() setupQmake(defaultQt, tempDir1.path()), setupQmake(defaultQt, tempDir2.path()) }; - for (int i = 1; i < qmakePaths.count(); ++i) { - const Utils::FileName qp = qmakePaths.at(i); - QVERIFY(!Utils::contains(QtVersionManager::versions(), - [qp](BaseQtVersion *qt) { return qt->qmakeCommand() == qp; })); - } + for (int i = 1; i < qmakePaths.count(); ++i) + QVERIFY(!QtVersionManager::version(Utils::equal(&BaseQtVersion::qmakeCommand, qmakePaths.at(i)))); QList<DirectoryData *> testData; diff --git a/src/plugins/qtsupport/qtversionmanager.cpp b/src/plugins/qtsupport/qtversionmanager.cpp index 0da7d448c7..18ec5f116c 100644 --- a/src/plugins/qtsupport/qtversionmanager.cpp +++ b/src/plugins/qtsupport/qtversionmanager.cpp @@ -511,15 +511,6 @@ QList<BaseQtVersion *> QtVersionManager::unsortedVersions(const BaseQtVersion::P return m_versions.values(); } -QList<BaseQtVersion *> QtVersionManager::versions(const BaseQtVersion::Predicate &predicate) -{ - QList<BaseQtVersion *> versions; - QTC_ASSERT(isLoaded(), return versions); - versions = unsortedVersions(predicate); - Utils::sort(versions, qtVersionNumberCompare); - return versions; -} - QList<BaseQtVersion *> QtVersionManager::sortVersions(const QList<BaseQtVersion *> &input) { QList<BaseQtVersion *> result = input; @@ -532,7 +523,7 @@ QList<BaseQtVersion *> QtVersionManager::validVersions(const BaseQtVersion::Pred QTC_ASSERT(isLoaded(), return { }); auto superPredicate = [predicate](const BaseQtVersion *v) { return v->isValid() && (!predicate || predicate(v)); }; - return versions(superPredicate); + return unsortedVersions(superPredicate); } bool QtVersionManager::isValidId(int id) diff --git a/src/plugins/qtsupport/qtversionmanager.h b/src/plugins/qtsupport/qtversionmanager.h index 38d26b3075..66fe047fbb 100644 --- a/src/plugins/qtsupport/qtversionmanager.h +++ b/src/plugins/qtsupport/qtversionmanager.h @@ -47,7 +47,6 @@ public: // This will *always* return at least one (Qt in Path), even if that is // unconfigured. // Sorting is slow due to needing to potentially run qmake --query for each version - static QList<BaseQtVersion *> versions(const BaseQtVersion::Predicate &predicate = BaseQtVersion::Predicate()); static QList<BaseQtVersion *> validVersions(const BaseQtVersion::Predicate &predicate = BaseQtVersion::Predicate()); static QList<BaseQtVersion *> unsortedVersions(const BaseQtVersion::Predicate &predicate = BaseQtVersion::Predicate()); diff --git a/src/plugins/winrt/winrtdevicefactory.cpp b/src/plugins/winrt/winrtdevicefactory.cpp index f4f676cd42..67727fa5ef 100644 --- a/src/plugins/winrt/winrtdevicefactory.cpp +++ b/src/plugins/winrt/winrtdevicefactory.cpp @@ -172,10 +172,11 @@ QString WinRtDeviceFactory::findRunnerFilePath() const { const QString winRtRunnerExe = QStringLiteral("/winrtrunner.exe"); const QList<BaseQtVersion *> winrtVersions - = QtVersionManager::validVersions([](const BaseQtVersion *v) { + = QtVersionManager::sortVersions( + QtVersionManager::validVersions([](const BaseQtVersion *v) { return v->type() == QLatin1String(Constants::WINRT_WINRTQT) || v->type() == QLatin1String(Constants::WINRT_WINPHONEQT); - }); + })); QString filePath; BaseQtVersion *qt = nullptr; for (BaseQtVersion *v : winrtVersions) { |