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/SBEvent.cpp

diff --git a/source/API/SBEvent.cpp b/source/API/SBEvent.cpp
index 70a4277..527a7d2 100644
--- a/source/API/SBEvent.cpp
+++ b/source/API/SBEvent.cpp
@@ -27,23 +27,18 @@
     m_event_sp (),
     m_opaque (NULL)
 {
-    Log *log = lldb_private::GetLogIfAllCategoriesSet (LIBLLDB_LOG_API | LIBLLDB_LOG_VERBOSE);
-
-    if (log)
-        log->Printf ("SBEvent::SBEvent () ==> this = %p", this);
 }
 
 SBEvent::SBEvent (uint32_t event_type, const char *cstr, uint32_t cstr_len) :
     m_event_sp (new Event (event_type, new EventDataBytes (cstr, cstr_len))),
     m_opaque (m_event_sp.get())
 {
-    Log *log = lldb_private::GetLogIfAllCategoriesSet (LIBLLDB_LOG_API | LIBLLDB_LOG_VERBOSE);
+    Log *log = lldb_private::GetLogIfAllCategoriesSet (LIBLLDB_LOG_API);
 
     if (log)
     {
-        log->Printf ("SBEvent::SBEvent (uint32_t event_type, const char *cstr, uint32_t cstr_len)");
-        log->Printf ("    event_type = %d, cstr = '%s', cstr_len = %d  ==> this = %p (m_opaque = %p)", event_type,
-                     cstr, cstr_len, this, m_opaque);
+        log->Printf ("SBEvent::SBEvent (event_type=%d, cstr='%s', cstr_len=%d)  => this.sp = %p", event_type,
+                     cstr, cstr_len, m_opaque);
     }
 }
 
@@ -51,10 +46,10 @@
     m_event_sp (event_sp),
     m_opaque (event_sp.get())
 {
-    Log *log = lldb_private::GetLogIfAllCategoriesSet (LIBLLDB_LOG_API | LIBLLDB_LOG_VERBOSE);
+    Log *log = lldb_private::GetLogIfAllCategoriesSet (LIBLLDB_LOG_API);
 
     if (log)
-        log->Printf ("SBEvent::SBEvent (EventSP &event_sp) event_sp.get() = %p ==> this = %p", event_sp.get(), this);
+        log->Printf ("SBEvent::SBEvent (event_sp=%p) => this.sp = %p", event_sp.get(), m_opaque);
 }
 
 SBEvent::~SBEvent()
@@ -75,8 +70,8 @@
 {
     Log *log = lldb_private::GetLogIfAllCategoriesSet (LIBLLDB_LOG_API);
 
-    if (log)
-        log->Printf ("SBEvent::GetType ()");
+    //if (log)
+    //    log->Printf ("SBEvent::GetType ()");
 
     const Event *lldb_event = get();
     uint32_t event_type = 0;
@@ -84,7 +79,7 @@
         event_type = lldb_event->GetType();
 
     if (log)
-        log->Printf ("SBEvent::GetType ==> %d", event_type);
+        log->Printf ("SBEvent::GetType (this.sp=%p) => %d", m_opaque, event_type);
 
     return event_type;
 }
@@ -125,7 +120,8 @@
         success = lldb_event->BroadcasterIs (broadcaster.get());
 
     if (log)
-        log->Printf ("SBEvent::BroadcasterMathesRef ==> %s", (success ? "true" : "false"));
+        log->Printf ("SBEvent::BroadcasterMathesRef (this.sp=%p, broadcaster.obj=%p) => %s", m_opaque, 
+                     broadcaster.get(), (success ? "true" : "false"));
 
     return success;
 }
@@ -186,7 +182,7 @@
     Log *log = lldb_private::GetLogIfAllCategoriesSet (LIBLLDB_LOG_API);
 
     if (log)
-        log->Printf ("GetCStringFromEvent ==> %s",
+        log->Printf ("GetCStringFromEvent (event.sp=%p) => %s", event.m_opaque, 
                      reinterpret_cast<const char *>(EventDataBytes::GetBytesFromEvent (event.get())));
 
     return reinterpret_cast<const char *>(EventDataBytes::GetBytesFromEvent (event.get()));