Fixing formatting and addressing how we check for extra arguments.

ashgti · ashgti · commit fcd79e1dbaeb · 2024-10-16T12:15:06.000-07:00
diff --git a/lldb/test/API/tools/lldb-dap/send-event/TestDAP_sendEvent.py b/lldb/test/API/tools/lldb-dap/send-event/TestDAP_sendEvent.py
@@ -62,6 +62,6 @@ def test_send_internal_event(self):
             "`lldb-dap send-event stopped", context="repl"
         )
         self.assertRegex(
-            resp['body']['result'],
-            r"Invalid use of lldb-dap send-event, event \"stopped\" should be handled by lldb-dap internally."
+            resp["body"]["result"],
+            r"Invalid use of lldb-dap send-event, event \"stopped\" should be handled by lldb-dap internally.",
         )
diff --git a/lldb/tools/lldb-dap/DAP.cpp b/lldb/tools/lldb-dap/DAP.cpp
@@ -997,7 +997,7 @@ bool SendEventRequestHandler::DoExecute(lldb::SBDebugger debugger,
 
   if (command[1] && !llvm::StringRef(command[1]).empty()) {
     // See if we have to unused arguments.
-    if (command[2] && !llvm::StringRef(command[2]).empty()) {
+    if (command[2]) {
       result.SetError(
           "Additional arguments found, expected `lldb-dap send-event "
           "<name> <body>?`.");

Original file line number	Diff line number	Diff line change
`@@ -62,6 +62,6 @@ def test_send_internal_event(self):`
`62`	`62`	"`lldb-dap send-event stopped", context="repl"
`63`	`63`	`)`
`64`	`64`	`self.assertRegex(`
`65`		`- resp['body']['result'],`
`66`		`- r"Invalid use of lldb-dap send-event, event \"stopped\" should be handled by lldb-dap internally."`
	`65`	`+ resp["body"]["result"],`
	`66`	`+ r"Invalid use of lldb-dap send-event, event \"stopped\" should be handled by lldb-dap internally.",`
`67`	`67`	`)`