================
@@ -21,7 +21,7 @@
 #include <optional>
 #include <string>
 
-namespace lldb_dap::protocol {
+namespace lldb_dap::protocol::dap {
----------------
ashgti wrote:

I don't think we should need to change the existing namespace. The 
`lldb_dap::protocol` namespace already corresponds to the types used for the 
DAP protocol.

The other implementation details of lldb-dap are in the `lldb_dap` namespace.

https://github.com/llvm/llvm-project/pull/153836
_______________________________________________
lldb-commits mailing list
lldb-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits

Reply via email to