[Lldb-commits] [lldb] [lldb] Standardize command option parsing error messages (PR #82273)

Felipe de Azevedo Piovezan via lldb-commits lldb-commits at lists.llvm.org
Mon Feb 19 18:09:51 PST 2024


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

This LGTM!

I don't think I can see far enough ahead on what you are planning here, but I was just wondering if the ultimate goal is to  have the `option_arg.getAsT` return an `Expected<T>`. In this case, wouldn't all these arguments (short option, long option, additional context) have to be part of the interface of `getAsT`?  I suspect this is not your goal, but I can't see a way around that 

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


More information about the lldb-commits mailing list