Changeset 34402 in webkit for trunk/JavaScriptCore/profiler/Profile.cpp

Timestamp:

Jun 6, 2008, 9:48:54 AM (17 years ago)

Author:

[email protected]

Message:

2008-06-06 Kevin McCullough <​[email protected]>

Reviewed by Adam.

<rdar://problem/5969992> JSProfiler: Remove the recursion limit in the
profiler.

• Change the remaining functions that do not take arguments, from using recursion to using iteration.

• JavaScriptCore.exp:
• profiler/Profile.cpp: (KJS::stopProfiling): (KJS::restoreAll): (KJS::Profile::stopProfiling): Use foreach instead of recursion. (KJS::Profile::restoreAll): Ditto.
• profiler/Profile.h:
• profiler/ProfileNode.cpp: Remove recursion. (KJS::ProfileNode::stopProfiling): (KJS::ProfileNode::restore):
• profiler/ProfileNode.h:

File:

• trunk/JavaScriptCore/profiler/Profile.cpp (modified) (4 diffs)

trunk/JavaScriptCore/profiler/Profile.cpp

@@ -38 +38 @@
 const unsigned DEPTH_LIMIT = 1000;
 
+static void stopProfiling(ProfileNode* n) { n->stopProfiling(); }
+static void restoreAll(ProfileNode* n) { n->restore(); }
+
 Profile::Profile(const UString& title, ExecState* originatingGlobalExec, unsigned pageGroupIdentifier)
     : m_title(title)
@@ -54 +57 @@
     m_currentNode = 0;
     m_originatingGlobalExec = 0;
-    m_head->stopProfiling();
+    forEach(KJS::stopProfiling);
     m_depth = 0;
 }
@@ -87 +90 @@
 }
 
-void Profile::forEach(UnaryFunction function) {
+void Profile::forEach(UnaryFunction function)
+{
 
     ProfileNode* currentNode = m_head->firstChild();
@@ -99 +103 @@
     } 
 }
+
+void Profile::restoreAll()
+{
+    forEach(KJS::restoreAll);
+}
+
 
 #ifndef NDEBUG