[Lldb-commits] [lldb] [lldb-dap] Refactoring breakpoints to not use the `g_dap` reference. (PR #115208)

Pavel Labath via lldb-commits lldb-commits at lists.llvm.org
Thu Nov 7 01:13:43 PST 2024


https://github.com/labath commented:

This PR is much easier to review. I mainly focused on the stylistic parts. I'll leave it to the lldb-dap maintainers to review the functionality.

Copying polymorphic objects around is asking for trouble, and I don't think we should be doing that. You didn't come up with that, and it's unrelated to what you're trying to do, so I'm not asking you to change that. I'm just mentioning this in case you feel the urge to do something about.

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


More information about the lldb-commits mailing list