Make ProductBuildData a proper class with accessors and modifiers

We will need to keep track of some mutating accesses in a follow-up
change.

Change-Id: I8b3c92f6cca0f5dceb35ef7559cc1357cdf38656
Reviewed-by: Joerg Bornemann <joerg.bornemann@qt.io>

1 files changed, 4 insertions, 3 deletions

diff --git a/src/lib/corelib/buildgraph/artifactcleaner.cpp b/src/lib/corelib/buildgraph/artifactcleaner.cpp
index 4266899a9..280005e02 100644
--- a/src/lib/corelib/buildgraph/artifactcleaner.cpp
+++ b/src/lib/corelib/buildgraph/artifactcleaner.cpp
@@ -112,14 +112,15 @@ public:
     {
         m_product = product;
         ArtifactVisitor::visitProduct(product);
-        auto it = product->buildData->rescuableArtifactData.begin();
-        while (it != product->buildData->rescuableArtifactData.end()) {
+        const AllRescuableArtifactData rescuableArtifactData
+                = product->buildData->rescuableArtifactData();
+        for (auto it = rescuableArtifactData.begin(); it != rescuableArtifactData.end(); ++it) {
             Artifact tmp;
             tmp.product = product;
             tmp.setFilePath(it.key());
             tmp.setTimestamp(it.value().timeStamp);
             removeArtifactFromDisk(&tmp, m_options.dryRun(), m_logger);
-            it = product->buildData->rescuableArtifactData.erase(it);
+            product->buildData->removeFromRescuableArtifactData(it.key());
         }
     }