Refactor BuildGraphLoader::trackProjectChanges().

It's becoming hard to keep an overview of which checks are being done to
find out whether we need to re-resolve, so introduce one function per
check.
In addition, skip checks if we already know we need to re-resolve.

Change-Id: I6abeb41c6089e7fb7a0bfa2e69361ab8e58f477f
Reviewed-by: Joerg Bornemann <joerg.bornemann@digia.com>

1 files changed, 11 insertions, 0 deletions

diff --git a/src/lib/buildgraph/buildgraphloader.h b/src/lib/buildgraph/buildgraphloader.h
index 999d1f205..cd769583b 100644
--- a/src/lib/buildgraph/buildgraphloader.h
+++ b/src/lib/buildgraph/buildgraphloader.h
@@ -41,6 +41,7 @@ class SetupProjectParameters;
 
 namespace Internal {
 class ArtifactList;
+class FileTime;
 
 class BuildGraphLoadResult
 {
@@ -65,6 +66,16 @@ private:
     void trackProjectChanges(const SetupProjectParameters &parameters,
                              const QString &buildGraphFilePath,
                              const TopLevelProjectPtr &restoredProject);
+    bool hasEnvironmentChanged(const TopLevelProjectConstPtr &restoredProject) const;
+    bool hasProjectFileChanged(const QList<ResolvedProjectPtr> &restoredProjects,
+                               const FileTime &referenceTime,
+                               QSet<QString> &remainingBuildSystemFiles) const;
+    bool hasProductFileChanged(const QList<ResolvedProductPtr> &restoredProducts,
+                               const FileTime &referenceTime,
+                               QSet<QString> &remainingBuildSystemFiles,
+                               QList<ResolvedProductPtr> &changedProducts);
+    bool hasBuildSystemFileChanged(const QSet<QString> &buildSystemFiles,
+                                   const FileTime &referenceTime);
     void onProductRemoved(const ResolvedProductPtr &product, ProjectBuildData *projectBuildData);
     void onProductChanged(const ResolvedProductPtr &product,
                           const ResolvedProductPtr &changedProduct);