================
@@ -135,36 +140,26 @@ class RequestHandler : public BaseRequestHandler {
llvm::Expected<Args> arguments = parseArgs<Args>(request);
if (llvm::Error err = arguments.takeError()) {
HandleErrorResponse(std::move(err), response);
- dap.Send(response);
+ Send(response);
return;
}
if constexpr (std::is_same_v<Resp, llvm::Error>) {
- if (llvm::Error err = Run(*arguments)) {
+ if (llvm::Error err = Run(*arguments))
HandleErrorResponse(std::move(err), response);
- } else {
+ else
response.success = true;
- }
} else {
Resp body = Run(*arguments);
- if (llvm::Error err = body.takeError()) {
+ if (llvm::Error err = body.takeError())
HandleErrorResponse(std::move(err), response);
- } else {
+ else {
----------------
JDevlieghere wrote:
If one of the blocks needs braces, the other one should have braces too,
regardless of it being a single line, according to our style guide.
```suggestion
if (llvm::Error err = body.takeError()) {
HandleErrorResponse(std::move(err), response);
} else {
```
https://github.com/llvm/llvm-project/pull/171549
_______________________________________________
lldb-commits mailing list
[email protected]
https://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits
