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

Timestamp:

May 20, 2008, 4:32:49 PM (17 years ago)

Author:

[email protected]

Message:

2008-05-20 Kevin McCullough <​[email protected]>

Reviewed by Darin.

<rdar://problem/5950796> JSProfiler: dump functions are in the code
Removed dump and logging functions from the Release version of the code
and renamed them to be obviously for debugging only.

• JavaScriptCore.exp:
• profiler/Profile.cpp: (KJS::Profile::debugPrintData): (KJS::Profile::debugPrintDataSampleStyle):
• profiler/Profile.h:
• profiler/ProfileNode.cpp: (KJS::ProfileNode::debugPrintData): (KJS::ProfileNode::debugPrintDataSampleStyle):
• profiler/ProfileNode.h:
• profiler/Profiler.cpp:
• profiler/Profiler.h:

File:

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

trunk/JavaScriptCore/profiler/Profile.cpp

@@ -73 +73 @@
 }
 
-void Profile::printDataInspectorStyle() const
+#ifndef NDEBUG
+void Profile::debugPrintData() const
 {
     printf("Call graph:\n");
-    m_callTree->printDataInspectorStyle(0);
+    m_callTree->debugPrintData(0);
 }
 
@@ -86 +87 @@
 }
 
-void Profile::printDataSampleStyle() const
+void Profile::debugPrintDataSampleStyle() const
 {
     typedef Vector<NameCountPair> NameCountPairVector;
@@ -92 +93 @@
     FunctionCallHashCount countedFunctions;
     printf("Call graph:\n");
-    m_callTree->printDataSampleStyle(0, countedFunctions);
+    m_callTree->debugPrintDataSampleStyle(0, countedFunctions);
 
     printf("\nTotal number in stack:\n");
@@ -104 +105 @@
     printf("\nSort by top of stack, same collapsed (when >= 5):\n");
 }
+#endif
 
 }   // namespace KJS