[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