Give the ArtifactVisitor functions unique names.

Overloading and polymorphism don't mix well.
Also remove the visit() functions taking lists. They are overkill.

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

1 files changed, 3 insertions, 3 deletions

diff --git a/src/lib/buildgraph/artifactcleaner.cpp b/src/lib/buildgraph/artifactcleaner.cpp
index a4ded0a62..d19fb7c37 100644
--- a/src/lib/buildgraph/artifactcleaner.cpp
+++ b/src/lib/buildgraph/artifactcleaner.cpp
@@ -94,10 +94,10 @@ public:
     {
     }
 
-    void visit(const BuildProduct::ConstPtr &product)
+    void visitProduct(const BuildProduct::ConstPtr &product)
     {
         m_product = product;
-        ArtifactVisitor::visit(product);
+        ArtifactVisitor::visitProduct(product);
     }
 
     const QSet<QString> &directories() const { return m_directories; }
@@ -128,7 +128,7 @@ void ArtifactCleaner::cleanup(const QList<BuildProduct::ConstPtr> &products, boo
     QSet<QString> directories;
     foreach (const BuildProduct::ConstPtr &product, products) {
         CleanupVisitor visitor(!buildOptions.keepGoing, buildOptions.dryRun, removeAll);
-        visitor.visit(product);
+        visitor.visitProduct(product);
         directories.unite(visitor.directories());
     }