Changeset 12512 in webkit for trunk/JavaScriptCore/kjs/nodes.cpp

Timestamp:

Jan 31, 2006, 10:24:32 PM (19 years ago)

Author:

mjs

Message:

Reviewed by Darin.

• fixed leak of hundreds of thousands of JS parser nodes on the layout tests, and added an exit counter that would catch them

• kjs/nodes.cpp: (NodeCounter::~NodeCounter): Added debug-only node counter. (Node::Node): (Node::~Node):
• kxmlcore/VectorTraits.h: Simple classes like RefPtr do in fact need destruction.

File:

• trunk/JavaScriptCore/kjs/nodes.cpp (modified) (2 diffs)

trunk/JavaScriptCore/kjs/nodes.cpp

@@ -86 +86 @@
 // ------------------------------ Node -----------------------------------------
 
+
+#ifndef NDEBUG
+struct NodeCounter { 
+    static int count; 
+    ~NodeCounter() { if (count != 0) fprintf(stderr, "LEAK: %d KJS::Node\n", count); }
+};
+int NodeCounter::count = 0;
+static NodeCounter nodeImplCounter;
+#endif NDEBUG
+
+
 Node::Node()
 {
+#ifndef NDEBUG
+    ++NodeCounter::count;
+#endif
   line = Lexer::curr()->lineNo();
   sourceURL = Lexer::curr()->sourceURL();
@@ -96 +110 @@
 Node::~Node()
 {
+#ifndef NDEBUG
+    --NodeCounter::count;
+#endif
 }