[Lldb-commits] [lldb] draft: [lldb] Upgrade ValueObject::GetData to return llvm::Expected (PR #130516)

[Julius Alexandre via lldb-commits]

wizardengineer wrote:

Hey @adrian-prantl, if you can and when you have time. I was wondering if it'd be okay if you can review this and just so i can make sure it aligns with the standards for this PR.

I was also curious to know if I'd have to change any test code too, I've tried to unit test but got error relating to "expected" values. Would I need to change something in `DumpValueObjectOptionsTests.cpp`?

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