commit 61ba7ec20f616f907473dd501341cef5b47fa3f8
author Caroline Tice <ctice@apple.com> Tue Oct 26 23:49:36 2010 +0000
committer Caroline Tice <ctice@apple.com> Tue Oct 26 23:49:36 2010 +0000
tree 8cbfb029ec3c98825ba6fe0912c49cad3ac538a7
parent 0561f3da5dd365d1f75c0a6888293eaf1695ccd1

Clean up the API logging code:
      - Try to reduce logging to one line per function call instead of tw
      - Put all arguments & their values into log for calls
      - Add 'this' parameter information to function call logging, making it show the appropriate
        internal pointer (this.obj, this.sp, this.ap...)
      - Clean up some return values
      - Remove logging of constructors that construct empty objects
      - Change '==>' to '=>'  for showing result values...
      - Fix various minor bugs
      - Add some protected 'get' functions to help getting the internal pointers for the 'this' arguments...      



git-svn-id: https://llvm.org/svn/llvm-project/llvdb/trunk@117417 91177308-0d34-0410-b5e6-96231b3b80d8

source/API/SBFunction.cpp

diff --git a/source/API/SBFunction.cpp b/source/API/SBFunction.cpp
index 459ead2..2efd37c 100644
--- a/source/API/SBFunction.cpp
+++ b/source/API/SBFunction.cpp
@@ -25,23 +25,20 @@
 SBFunction::SBFunction () :
     m_opaque_ptr (NULL)
 {
-    Log *log = lldb_private::GetLogIfAllCategoriesSet (LIBLLDB_LOG_API | LIBLLDB_LOG_VERBOSE);
-
-    if (log)
-        log->Printf ("SBFunction::SBFunction () ==> this = %p", this);
 }
 
 SBFunction::SBFunction (lldb_private::Function *lldb_object_ptr) :
     m_opaque_ptr (lldb_object_ptr)
 {
-    Log *log = lldb_private::GetLogIfAllCategoriesSet (LIBLLDB_LOG_API | LIBLLDB_LOG_VERBOSE);
+    Log *log = lldb_private::GetLogIfAllCategoriesSet (LIBLLDB_LOG_API);
 
     if (log)
     {
         SBStream sstr;
         GetDescription (sstr);
-        log->Printf ("SBFunction::SBFunction (lldb_Private::Function *lldb_object_ptr) lldb_object_ptr = %p "
-                     " ==> this = %p (%s)", lldb_object_ptr, this, sstr.GetData());
+        log->Printf ("SBFunction::SBFunction (lldb_object_ptr=%p) => this.obj = %p ('%s')", lldb_object_ptr, 
+                     m_opaque_ptr, sstr.GetData());
+                     
     }
 }
 
@@ -61,18 +58,19 @@
 {
     Log *log = lldb_private::GetLogIfAllCategoriesSet (LIBLLDB_LOG_API);
 
-    if (log)
-        log->Printf ("SBFunction::GetName ()"); 
+    //if (log)
+    //    log->Printf ("SBFunction::GetName ()"); 
 
     if (m_opaque_ptr)
     {
         if (log)
-            log->Printf ("SBFunction::GetName ==> %s", m_opaque_ptr->GetMangled().GetName().AsCString());
+            log->Printf ("SBFunction::GetName (this.obj=%p) => '%s'", m_opaque_ptr,
+                         m_opaque_ptr->GetMangled().GetName().AsCString());
         return m_opaque_ptr->GetMangled().GetName().AsCString();
     }
 
     if (log)
-        log->Printf ("SBFunction::GetName ==> NULL");
+        log->Printf ("SBFunction::GetName (this.obj=%p) => NULL", m_opaque_ptr);
     return NULL;
 }
 
@@ -136,4 +134,9 @@
     return sb_instructions;
 }
 
+lldb_private::Function *
+SBFunction::get ()
+{
+    return m_opaque_ptr;
+}