[Lldb-commits] [lldb] Fix command escape bug in lldb-dap (PR #72902)

via lldb-commits lldb-commits at lists.llvm.org
Mon Nov 20 15:57:10 PST 2023


jeffreytan81 wrote:

> LGTM. It may be a good idea to move these APIs over to use std::optional instead of taking a default value so we can use 'value_or' instead of having to write logic to handle the default value through a parameter.

My goal is fixing the regression in this diff. I will let @walter-erquinigo to comment/follow-up on this suggestion.

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


More information about the lldb-commits mailing list