Changeset 16833

Timestamp:

06/14/13 10:52:43 (11 years ago)

Author:

tbretz

Message:

The disposed handle fInterruptCallback must also be cleared, otherwise it might still be accessible. Removed a stray debug output.

File:

• trunk/FACT++/src/InterpreterV8.cc (modified) (4 diffs)

trunk/FACT++/src/InterpreterV8.cc

@@ -86 +86 @@
 // script termination.
 //
-
+// Here is another good reference for v8, also containing some
+// good explanations for the meaning of handles, persistent handles
+// and weak handles: http://create.tpsitulsa.com/wiki/V8_Cookbook
+//
 // ==========================================================================
 //                            Simple interface
@@ -1570 +1573 @@
     {
         fInterruptCallback.Dispose();
+        fInterruptCallback.Clear();
         return Undefined();
     }
@@ -1599 +1603 @@
 
     for (auto it=data.begin(); it!=data.end(); it++)
-    {
-        cout << "|" << it->first << "=" << it->second << "|" << endl;
         arr->Set(String::New(it->first.c_str()), String::New(it->second.c_str()));
-    }
 
     Handle<Value> args[] = { irq_str, arr, date, usr_str };
@@ -2663 +2664 @@
     // Now we can dispose the persistent interrupt handler
     fInterruptCallback.Dispose();
+    fInterruptCallback.Clear();
 
     // Now we can dispose all persistent handles from reverse maps