[Lldb-commits] [lldb] Make BreakpointName::SetEnable() changes propagate to breakpoints (PR #176651)

[via lldb-commits]

llvmbot wrote:


<!--LLVM PR SUMMARY COMMENT-->

@llvm/pr-subscribers-lldb

Author: Teodor Spæren (rHermes)

<details>
<summary>Changes</summary>

This commit adds a missing "UpdateName()" call to the `BreakpointName::SetEnabled` method, to make sure changes are synced.

I don't know if this was left out on purpose, if so, feel free to close this, though I think it should be documented in that case, that the this behavior does not act like the others.

---
Full diff: https://github.com/llvm/llvm-project/pull/176651.diff


1 Files Affected:

- (modified) lldb/source/API/SBBreakpointName.cpp (+1) 


``````````diff
diff --git a/lldb/source/API/SBBreakpointName.cpp b/lldb/source/API/SBBreakpointName.cpp
index 0b588c38d5114..cf045e87b338a 100644
--- a/lldb/source/API/SBBreakpointName.cpp
+++ b/lldb/source/API/SBBreakpointName.cpp
@@ -213,6 +213,7 @@ void SBBreakpointName::SetEnabled(bool enable) {
         m_impl_up->GetTarget()->GetAPIMutex());
 
   bp_name->GetOptions().SetEnabled(enable);
+  UpdateName(*bp_name);
 }
 
 void SBBreakpointName::UpdateName(BreakpointName &bp_name) {

``````````

</details>


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