[Lldb-commits] [lldb] Add a scripted way to re-present a stop location (PR #158128)

[Jonas Devlieghere via lldb-commits]

================
@@ -454,8 +525,13 @@ bool BreakpointLocation::SetBreakpointSite(BreakpointSiteSP &bp_site_sp) {
 }
 
 llvm::Error BreakpointLocation::ClearBreakpointSite() {
-  if (!m_bp_site_sp)
-    return llvm::createStringError("no breakpoint site to clear");
+  if (!m_bp_site_sp) {
+    // This might be a Facade Location, which don't have sites or addresses
+    if (IsFacade())
+      return llvm::Error::success();
+    else
+      return llvm::createStringError("no breakpoint site to clear");
----------------
JDevlieghere wrote:

[No else after return](https://llvm.org/docs/CodingStandards.html#don-t-use-else-after-a-return).

https://github.com/llvm/llvm-project/pull/158128