[Lldb-commits] [lldb] r313371 - Remove a couple of warnings pointed out by Ted Woodward.

[Jim Ingham via lldb-commits]

Author: jingham
Date: Fri Sep 15 10:54:37 2017
New Revision: 313371

URL: http://llvm.org/viewvc/llvm-project?rev=313371&view=rev
Log:
Remove a couple of warnings pointed out by Ted Woodward.

Modified:
    lldb/trunk/source/API/SBBreakpointName.cpp

Modified: lldb/trunk/source/API/SBBreakpointName.cpp
URL: http://llvm.org/viewvc/llvm-project/lldb/trunk/source/API/SBBreakpointName.cpp?rev=313371&r1=313370&r2=313371&view=diff
==============================================================================
--- lldb/trunk/source/API/SBBreakpointName.cpp (original)
+++ lldb/trunk/source/API/SBBreakpointName.cpp Fri Sep 15 10:54:37 2017
@@ -77,15 +77,15 @@ public:
   // For now we take a simple approach and only keep the name, and relook
   // up the location when we need it.
   
-  TargetSP GetTarget() {
+  TargetSP GetTarget() const {
     return m_target_wp.lock();
   }
   
-  const char *GetName() {
+  const char *GetName() const {
     return m_name.c_str();
   }
   
-  bool IsValid() {
+  bool IsValid() const {
     return !m_name.empty() && m_target_wp.lock();
   }
   
@@ -102,7 +102,13 @@ public:
   
   const lldb_private::BreakpointName *GetBreakpointName() const
   {
-    return GetBreakpointName();
+    if (!IsValid())
+      return nullptr;
+    TargetSP target_sp = GetTarget();
+    if (!target_sp)
+      return nullptr;
+    Status error;
+    return target_sp->FindBreakpointName(ConstString(m_name), true, error);    
   }
   
 private:
@@ -339,7 +345,7 @@ bool SBBreakpointName::GetAutoContinue()
   
   BreakpointName *bp_name = GetBreakpointName();
   if (!bp_name)
-    return nullptr;
+    return false;
  
   LLDB_LOG(log, "Name: {0}\n", bp_name->GetName());
   std::lock_guard<std::recursive_mutex> guard(