Make the IsValid() operator for SBLineEntry "do the right thing" by returning true if the line entry is valid, not just if the opaque pointer to lldb_private::LineEntry is non NULL.

Greg Clayton · Greg Clayton · commit 3a70d5d2f090 · 2012-06-27T20:18:01.000Z
git-svn-id: https://llvm.org/svn/llvm-project/lldb/trunk@159287 91177308-0d34-0410-b5e6-96231b3b80d8
diff --git a/source/API/SBLineEntry.cpp b/source/API/SBLineEntry.cpp
@@ -110,7 +110,7 @@ SBLineEntry::GetEndAddress () const
 bool
 SBLineEntry::IsValid () const
 {
-    return m_opaque_ap.get() != NULL;
+    return m_opaque_ap.get() && m_opaque_ap->IsValid();
 }
 
 

Original file line number	Diff line number	Diff line change
`@@ -110,7 +110,7 @@ SBLineEntry::GetEndAddress () const`
`110`	`110`	`bool`
`111`	`111`	`SBLineEntry::IsValid () const`
`112`	`112`	`{`
`113`		`- return m_opaque_ap.get() != NULL;`
	`113`	`+ return m_opaque_ap.get() && m_opaque_ap->IsValid();`
`114`	`114`	`}`
`115`	`115`
`116`	`116`