Remove ExecutionEngine::identifiers.

The member was a duplicate of stringPool, there is no point in caching
a cache.

Change-Id: If34c80ee120dfaff1dd94326625b02d8014806dc
Reviewed-by: Simon Hausmann <simon.hausmann@digia.com>

diff --git a/qmljs_engine.cpp b/qmljs_engine.cpp
index 7d5a8a4..1940a4f 100644 (file)
--- a/qmljs_engine.cpp
+++ b/qmljs_engine.cpp
@@ -223,10 +223,7 @@ ExecutionContext *ExecutionEngine::newContext()
 
 String *ExecutionEngine::identifier(const QString &s)
 {
-    String *&id = identifiers[s];
-    if (! id)
-        id = newString(s);
-    return id;
+    return stringPool->newString(s);
 }
 
 Function *ExecutionEngine::newFunction(const QString &name)

diff --git a/qmljs_engine.h b/qmljs_engine.h
index 16d8fc5..94093cb 100644 (file)
--- a/qmljs_engine.h
+++ b/qmljs_engine.h
@@ -130,8 +130,6 @@ struct ExecutionEngine
     TypeErrorPrototype *typeErrorPrototype;
     URIErrorPrototype *uRIErrorPrototype;
 
-    QHash<QString, String *> identifiers;
-
     String *id_length;
     String *id_prototype;
     String *id_constructor;