[Lldb-commits] [lldb] [lldb][Test] Add C++ tests for DumpValueObjectOptions and enums (PR #93158)

via lldb-commits lldb-commits at lists.llvm.org
Thu May 23 11:45:07 PDT 2024


jimingham wrote:

LGTM.  

We do offer SBValue::GetDescription, which uses a default DumpValueObjectOptions.  So it would be entirely reasonable to make an SBValueObjectDescriptionOptions (or maybe a shorter name?) and add SBValue::GetDescription(SBValueObjectDescriptionOptions) API.

But people can also cons up an equivalent Dump on their own using the other SBValue API's, so leaving the GetDescription as it is and doing the testing on the side like this is also fine.

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


More information about the lldb-commits mailing list