llvmbot wrote:
<!--LLVM PR SUMMARY COMMENT--> @llvm/pr-subscribers-lldb Author: Ebuka Ezike (da-viper) <details> <summary>Changes</summary> >From >https://github.com/llvm/llvm-project/pull/175930#pullrequestreview-3667183086 --- Full diff: https://github.com/llvm/llvm-project/pull/176233.diff 3 Files Affected: - (modified) lldb/tools/lldb-dap/Protocol/ProtocolRequests.cpp (-1) - (modified) lldb/tools/lldb-dap/Protocol/ProtocolRequests.h (+3-3) - (modified) lldb/unittests/DAP/ProtocolTypesTest.cpp (+1-1) ``````````diff diff --git a/lldb/tools/lldb-dap/Protocol/ProtocolRequests.cpp b/lldb/tools/lldb-dap/Protocol/ProtocolRequests.cpp index 23789f6d46f90..a72802a33fc9c 100644 --- a/lldb/tools/lldb-dap/Protocol/ProtocolRequests.cpp +++ b/lldb/tools/lldb-dap/Protocol/ProtocolRequests.cpp @@ -335,7 +335,6 @@ bool fromJSON(const llvm::json::Value &Params, DAPSession &Ses, llvm::json::Path P) { json::ObjectMapper O(Params, P); - // Validate that both debuggerID and targetId are provided. return O && O.map("targetId", Ses.targetId) && O.map("debuggerId", Ses.debuggerId); } diff --git a/lldb/tools/lldb-dap/Protocol/ProtocolRequests.h b/lldb/tools/lldb-dap/Protocol/ProtocolRequests.h index 9d5bdf1efe94c..77b575bc5d0e4 100644 --- a/lldb/tools/lldb-dap/Protocol/ProtocolRequests.h +++ b/lldb/tools/lldb-dap/Protocol/ProtocolRequests.h @@ -318,10 +318,10 @@ using LaunchResponse = VoidResponse; #define LLDB_DAP_INVALID_FRAME_ID UINT64_MAX struct DAPSession { - /// Unique ID of an existing target to attach to. + /// A Unique ID of an existing target to attach to. lldb::user_id_t targetId; - /// ID of an existing debugger instance to use. + /// A Unique ID of an existing debugger instance to use. lldb::user_id_t debuggerId; }; bool fromJSON(const llvm::json::Value &, DAPSession &, llvm::json::Path); @@ -360,7 +360,7 @@ struct AttachRequestArguments { /// Path to the core file to debug. std::string coreFile; - /// An Existing session that consist of a target and debugger. + /// An existing session that consist of a target and debugger. std::optional<DAPSession> session; /// @} diff --git a/lldb/unittests/DAP/ProtocolTypesTest.cpp b/lldb/unittests/DAP/ProtocolTypesTest.cpp index 6476492813455..82753d3fe0cfd 100644 --- a/lldb/unittests/DAP/ProtocolTypesTest.cpp +++ b/lldb/unittests/DAP/ProtocolTypesTest.cpp @@ -1264,7 +1264,7 @@ TEST(ProtocolTypesTest, StackFrame) { } TEST(ProtocolTypesTest, DAPSession) { - const DAPSession session{/*targetId*/ 1000, /*debuggerId*/ 300}; + const DAPSession session{/*targetId=*/1000, /*debuggerId=*/300}; auto expected = parse<DAPSession>(R"({"targetId": 1000, "debuggerId": 300})"); ASSERT_THAT_EXPECTED(expected, llvm::Succeeded()); `````````` </details> https://github.com/llvm/llvm-project/pull/176233 _______________________________________________ lldb-commits mailing list [email protected] https://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits
