[Lldb-commits] [lldb] [lldb] Convert Breakpoint & Watchpoints structs to classes (NFC) (PR #133780)

[John Harrison via lldb-commits]

================
@@ -49,6 +41,17 @@ struct BreakpointBase {
   /// breakpoint in one of the DAP breakpoints that we should report changes
   /// for.
   static constexpr const char *kDAPBreakpointLabel = "dap";
+
+protected:
+  /// Associated DAP session.
+  DAP &dap;
+
+  /// An optional expression for conditional breakpoints.
+  std::string condition;
+
+  /// An optional expression that controls how many hits of the breakpoint are
+  /// ignored. The backend is expected to interpret the expression as needed
+  std::string hitCondition;
----------------
ashgti wrote:

nit: Should we use a `m_` prefix on any of these protected variables?

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