adrian-prantl wrote: >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?
Usually wrapping calls in `llvm::expectedToOptional()` is the cheapest way to just adapt the API in places where the error handling doesn't matter. You could even write a wrapper for that test that creates a Status from the error and imitates the old API, just inside the unit tests. https://github.com/llvm/llvm-project/pull/130516 _______________________________________________ lldb-commits mailing list lldb-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits
