================
@@ -904,11 +928,14 @@ bool StartDebuggingRequestHandler::DoExecute(
"startDebugging",
llvm::json::Object{{"request", request},
{"configuration", std::move(*configuration)}},
- [](llvm::Expected<llvm::json::Value> value) {
+ [](auto dap, auto value) {
----------------
labath wrote:
[llvm
policy](https://llvm.org/docs/CodingStandards.html#use-auto-type-deduction-to-make-code-more-readable)
is approximately to use `auto` only when the type is obvious from the context.
That definitely isn't the case here.
I'm pretty sure this file was written by someone unaware of that policy, but
let's not make it worse. And this is definitely making it worse. llvm::Expected
can look very similar to std::optional, but it has a very important difference
(the error must be checked), so it's very important to make the type obvious.
https://github.com/llvm/llvm-project/pull/116392
_______________________________________________
lldb-commits mailing list
lldb-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits
