Michael137 wrote:

> > Allows API tests to pass SBExpressionOptions when testing a successful 
> > expression evaluation with expect_expr.
> 
> Which tests and for what reasons?
> 
> Change is fine code wise, but I don't immediately see what I'd do with this.

I have an upcoming patch that adds an option to `SBExpressionOptions`. I would 
like to use `expect_expr` in those tests. Not super critical, but it makes the 
tests slightly nicer to read.

I'll wait on merging this PR once all the follow-up PRs are up, so there's 
examples of what it looks like

https://github.com/llvm/llvm-project/pull/177920
_______________________________________________
lldb-commits mailing list
[email protected]
https://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits

Reply via email to