From f5e9326c51e6352eed919c66d0c2a213f1e5b1dc Mon Sep 17 00:00:00 2001
From: Joerg Bornemann <joerg.bornemann@qt.io>
Date: Thu, 26 Apr 2018 10:02:25 +0200
Subject: Make ResolvedProduct::artifactProperties a vector

The value_type is not suitable for QList.

Change-Id: I73d021f0c635ed46992397c4d6e3f3394ec5a7f8
Reviewed-by: Christian Kandeler <christian.kandeler@qt.io>
---
 src/lib/corelib/language/language.cpp        | 6 +++---
 src/lib/corelib/language/language.h          | 6 +++---
 src/lib/corelib/language/projectresolver.cpp | 5 ++---
 3 files changed, 8 insertions(+), 9 deletions(-)

(limited to 'src/lib/corelib/language')

diff --git a/src/lib/corelib/language/language.cpp b/src/lib/corelib/language/language.cpp
index 6b9e5d056..fd3679f89 100644
--- a/src/lib/corelib/language/language.cpp
+++ b/src/lib/corelib/language/language.cpp
@@ -295,7 +295,7 @@ FileTags Rule::staticOutputFileTags() const
 FileTags Rule::collectedOutputFileTags() const
 {
     FileTags result = outputFileTags.empty() ? staticOutputFileTags() : outputFileTags;
-    for (const auto &ap : qAsConst(product->artifactProperties)) {
+    for (const auto &ap : product->artifactProperties) {
         if (ap->fileTagsFilter().intersects(result))
             result += ap->extraFileTags();
     }
@@ -932,8 +932,8 @@ uint qHash(const RuleArtifact::Binding &b)
     return qHash(std::make_pair(b.code, b.name.join(QLatin1Char(','))));
 }
 
-bool artifactPropertyListsAreEqual(const QList<ArtifactPropertiesPtr> &l1,
-                                   const QList<ArtifactPropertiesPtr> &l2)
+bool artifactPropertyListsAreEqual(const std::vector<ArtifactPropertiesPtr> &l1,
+                                   const std::vector<ArtifactPropertiesPtr> &l2)
 {
     return listsAreEqual(l1, l2);
 }
diff --git a/src/lib/corelib/language/language.h b/src/lib/corelib/language/language.h
index 38fbcd5f4..524c9cb27 100644
--- a/src/lib/corelib/language/language.h
+++ b/src/lib/corelib/language/language.h
@@ -578,7 +578,7 @@ public:
     std::vector<ResolvedScannerConstPtr> scanners;
     std::vector<GroupPtr> groups;
     std::vector<ProbeConstPtr> probes;
-    QList<ArtifactPropertiesPtr> artifactProperties;
+    std::vector<ArtifactPropertiesPtr> artifactProperties;
     QStringList missingSourceFiles;
     std::unique_ptr<ProductBuildData> buildData;
 
@@ -734,8 +734,8 @@ private:
     QVariantMap m_buildConfiguration;
 };
 
-bool artifactPropertyListsAreEqual(const QList<ArtifactPropertiesPtr> &l1,
-                                   const QList<ArtifactPropertiesPtr> &l2);
+bool artifactPropertyListsAreEqual(const std::vector<ArtifactPropertiesPtr> &l1,
+                                   const std::vector<ArtifactPropertiesPtr> &l2);
 
 QString multiplexIdToString(const QString &id);
 
diff --git a/src/lib/corelib/language/projectresolver.cpp b/src/lib/corelib/language/projectresolver.cpp
index 7341efdd4..c25e252bf 100644
--- a/src/lib/corelib/language/projectresolver.cpp
+++ b/src/lib/corelib/language/projectresolver.cpp
@@ -727,7 +727,7 @@ void ProjectResolver::resolveGroupFully(Item *item, ProjectResolver::ProjectCont
             }
 
             // Discard any Group with the same fileTagsFilter that was defined in a base file.
-            m_productContext->product->artifactProperties.removeAll(apinfo.first);
+            removeAll(m_productContext->product->artifactProperties, apinfo.first);
         }
         if (!isEnabled)
             return;
@@ -1389,8 +1389,7 @@ void ProjectResolver::matchArtifactProperties(const ResolvedProductPtr &product,
         const QList<SourceArtifactPtr> &artifacts)
 {
     for (const SourceArtifactPtr &artifact : artifacts) {
-        for (const ArtifactPropertiesConstPtr &artifactProperties :
-                 qAsConst(product->artifactProperties)) {
+        for (const ArtifactPropertiesConstPtr &artifactProperties : product->artifactProperties) {
             if (!artifact->isTargetOfModule()
                     && artifact->fileTags.intersects(artifactProperties->fileTagsFilter())) {
                 artifact->properties = artifactProperties->propertyMap();
-- 
cgit v1.2.3