[Lldb-commits] [lldb] [lldb] Step over non-lldb breakpoints (PR #174348)

[David Spickett via lldb-commits]

================
@@ -1159,6 +1195,13 @@ class StopInfoUnixSignal : public StopInfo {
     return false;
   }
 
+  void PerformAction([[maybe_unused]] Event *event_ptr) override {
+    // A signal of SIGTRAP indicates that a trap instruction has been hit.
+    if (m_value != SIGTRAP)
----------------
DavidSpickett wrote:

Personal taste, but I prefer positive checks to negative ones. So:
```
if (m_value == SIGTRAP)
  SkipOverTrapInstruction()
```
And it saves writing `return`.

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