C++: fix include/framework path handling.

Instead of having two lists of paths, now only one list is used where
both include paths and framework paths can be mixed. This reflects the
way the compiler is invoked, and retains the (correct) search order.

Task-number: QTCREATORBUG-11599
Change-Id: I373953e3e305df5b7a0d10920e12d146584adf9f
Reviewed-by: Nikolai Kosjar <nikolai.kosjar@digia.com>

1 files changed, 6 insertions, 14 deletions

diff --git a/src/plugins/cpptools/cppmodelmanager.h b/src/plugins/cpptools/cppmodelmanager.h
index bbdb2d6e79..7936019a14 100644
--- a/src/plugins/cpptools/cppmodelmanager.h
+++ b/src/plugins/cpptools/cppmodelmanager.h
@@ -128,22 +128,16 @@ public:
         return m_projectFiles;
     }
 
-    QStringList includePaths()
+    ProjectPart::HeaderPaths headerPaths()
     {
         ensureUpdated();
-        return m_includePaths;
+        return m_headerPaths;
     }
 
     // Use this *only* for auto tests
-    void setIncludePaths(const QStringList &includePaths)
+    void setHeaderPaths(const ProjectPart::HeaderPaths &headerPaths)
     {
-        m_includePaths = includePaths;
-    }
-
-    QStringList frameworkPaths()
-    {
-        ensureUpdated();
-        return m_frameworkPaths;
+        m_headerPaths = headerPaths;
     }
 
     QByteArray definedMacros()
@@ -187,8 +181,7 @@ private:
 
     void ensureUpdated();
     QStringList internalProjectFiles() const;
-    QStringList internalIncludePaths() const;
-    QStringList internalFrameworkPaths() const;
+    ProjectPart::HeaderPaths internalHeaderPaths() const;
     QByteArray internalDefinedMacros() const;
 
     void dumpModelManagerConfiguration(const QString &logFileId);
@@ -210,8 +203,7 @@ private:
     // The members below are cached/(re)calculated from the projects and/or their project parts
     bool m_dirty;
     QStringList m_projectFiles;
-    QStringList m_includePaths;
-    QStringList m_frameworkPaths;
+    ProjectPart::HeaderPaths m_headerPaths;
     QByteArray m_definedMacros;
 
     // Editor integration