IdentifierSearch: stop as soon as possible

By checking the stop condition in Visitor::preVisit we can stop the
AST traversal much earlier.

Change-Id: Ic9169c95f5e11c4ba040239e4349389a94bc335b
Reviewed-by: Christian Kandeler <christian.kandeler@digia.com>

2 files changed, 6 insertions, 0 deletions

diff --git a/src/lib/language/identifiersearch.cpp b/src/lib/language/identifiersearch.cpp
index 04f3cb1a5..7baa9aafc 100644
--- a/src/lib/language/identifiersearch.cpp
+++ b/src/lib/language/identifiersearch.cpp
@@ -50,6 +50,11 @@ void IdentifierSearch::add(const QString &name, bool *found)
     m_requests.insert(name, found);
 }
 
+bool IdentifierSearch::preVisit(QmlJS::AST::Node *)
+{
+    return m_numberOfFoundIds < m_requests.count();
+}
+
 bool IdentifierSearch::visit(QmlJS::AST::IdentifierExpression *e)
 {
     bool *found = m_requests.value(e->name.toString());
diff --git a/src/lib/language/identifiersearch.h b/src/lib/language/identifiersearch.h
index dbeffefa1..b291ab05f 100644
--- a/src/lib/language/identifiersearch.h
+++ b/src/lib/language/identifiersearch.h
@@ -46,6 +46,7 @@ public:
     void add(const QString &name, bool *found);
 
 private:
+    bool preVisit(QmlJS::AST::Node *);
     bool visit(QmlJS::AST::IdentifierExpression *e);
 
     QMap<QString, bool *> m_requests;