Apply suggestions from code review

Co-authored-by: Jonas Devlieghere <[email protected]>

Author: ashgti

lldb/tools/lldb-dap/Handler/LaunchRequestHandler.cpp

@@ -26,8 +26,7 @@ Error LaunchRequestHandler::Run(const LaunchRequestArguments &arguments) const {
   dap.stop_at_entry = arguments.stopOnEntry;
 
   if (!arguments.launchCommands.empty() && arguments.runInTerminal) {
-    return make_error<DAPError>("launchCommands and runInTerminal cannot "
-                                "both be set, use one or the other.");
+    return make_error<DAPError>("launchCommands and runInTerminal are mutually exclusive");
   }
 
   PrintWelcomeMessage();

lldb/tools/lldb-dap/Handler/RestartRequestHandler.cpp

@@ -90,8 +90,7 @@ void RestartRequestHandler::operator()(
   if (arguments) {
     // The optional `arguments` field in RestartRequest can contain an updated
     // version of the launch arguments. If there's one, use it.
-    const llvm::json::Value *restart_arguments = arguments->get("arguments");
-    if (restart_arguments) {
+    if (const llvm::json::Value *restart_arguments = arguments->get("arguments")) {
       protocol::LaunchRequestArguments updated_arguments;
       llvm::json::Path::Root root;
       if (!fromJSON(*restart_arguments, updated_arguments, root)) {