From d96f0682e54cb496d49ebc4f7caded01bbc0c7c0 Mon Sep 17 00:00:00 2001
From: Enrico Granata <egranata@apple.com>
Date: Thu, 7 Feb 2013 22:22:27 +0000
Subject: [PATCH] Correct logging for function calls that return SBValueList

llvm-svn: 174670
---
 lldb/include/lldb/API/SBValueList.h | 9 +++++++--
 lldb/source/API/SBFrame.cpp         | 4 ++--
 lldb/source/API/SBValueList.cpp     | 2 +-
 3 files changed, 10 insertions(+), 5 deletions(-)

diff --git a/lldb/include/lldb/API/SBValueList.h b/lldb/include/lldb/API/SBValueList.h
index 76654f3..5d046e4 100644
--- a/lldb/include/lldb/API/SBValueList.h
+++ b/lldb/include/lldb/API/SBValueList.h
@@ -52,7 +52,14 @@ public:
     const lldb::SBValueList &
     operator = (const lldb::SBValueList &rhs);
 
+protected:
+    
+    void *
+    get ();
+
 private:
+    friend class SBFrame;
+    
     SBValueList (const ValueListImpl *lldb_object_ptr);
 
     void
@@ -73,8 +80,6 @@ private:
     const ValueListImpl &
     operator* () const;
     
-    ValueListImpl *
-    get ();
     
     ValueListImpl &
     ref ();
diff --git a/lldb/source/API/SBFrame.cpp b/lldb/source/API/SBFrame.cpp
index 1283758..f339c90 100644
--- a/lldb/source/API/SBFrame.cpp
+++ b/lldb/source/API/SBFrame.cpp
@@ -1159,7 +1159,7 @@ SBFrame::GetVariables (bool arguments,
 
     if (log)
     {
-        log->Printf ("SBFrame(%p)::GetVariables (...) => SBValueList", frame);
+        log->Printf ("SBFrame(%p)::GetVariables (...) => SBValueList(%p)", frame, value_list.get());
     }
 
     return value_list;
@@ -1209,7 +1209,7 @@ SBFrame::GetRegisters ()
     }
 
     if (log)
-        log->Printf ("SBFrame(%p)::GetRegisters () => SBValueList", frame);
+        log->Printf ("SBFrame(%p)::GetRegisters () => SBValueList(%p)", frame, value_list.get());
 
     return value_list;
 }
diff --git a/lldb/source/API/SBValueList.cpp b/lldb/source/API/SBValueList.cpp
index 2ec4a7b..32b60f4 100644
--- a/lldb/source/API/SBValueList.cpp
+++ b/lldb/source/API/SBValueList.cpp
@@ -262,7 +262,7 @@ SBValueList::FindValueObjectByUID (lldb::user_id_t uid)
     return sb_value;
 }
 
-ValueListImpl *
+void *
 SBValueList::get ()
 {
     return m_opaque_ap.get();
-- 
2.7.4