[Lldb-commits] [lldb] [lldb][ConstString] Prevent GetString from constructing a std::string with a nullptr (PR #95175)

Alex Langford via lldb-commits lldb-commits at lists.llvm.org
Tue Jun 11 16:56:29 PDT 2024


https://github.com/bulbazord approved this pull request.

LGTM to avoid the UB, I wouldn't block this because of a complex existential question.

In general though, I don't really see the point in being able to convert a `ConstString` into a `std::string`? The whole point is that it uniques the string and manages its lifetime (which is effectively until the end of LLDB). I guess if you have a function/method that takes a `std::string` argument (or a reference to one) then this is useful... But why not just change the function (if it's in our control)?

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


More information about the lldb-commits mailing list