Fix another memory leak.

Now we should only leak objects that are VM::Values, and directly
related objects like the PropertyTable. The directly related ones will
disappear through destructor calls when the VM::Values are GC-ed.

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

1 files changed, 7 insertions, 0 deletions

diff --git a/qmljs_engine.cpp b/qmljs_engine.cpp
index 6c5edd16e2..4af900549d 100644
--- a/qmljs_engine.cpp
+++ b/qmljs_engine.cpp
@@ -187,6 +187,13 @@ ExecutionEngine::ExecutionEngine(EvalISelFactory *factory)
     glo->__put__(rootContext, identifier(QStringLiteral("isFinite")), Value::fromObject(new IsFiniteFunction(rootContext))); // isFinite [15.1.2.5]
 }
 
+ExecutionEngine::~ExecutionEngine()
+{
+    delete globalObject.asObject();
+    delete rootContext;
+    delete stringPool; // the String pointers should get GC-ed.
+}
+
 ExecutionContext *ExecutionEngine::newContext()
 {
     return new ExecutionContext();