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
_______________________________________________
lldb-commits mailing list
lldb-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits

Reply via email to