revert extra dap namespace

Author: eronnen

lldb/tools/lldb-dap/Breakpoint.cpp

@@ -25,9 +25,9 @@
 
 using namespace lldb_dap;
 
-static std::optional<protocol::dap::PersistenceData>
+static std::optional<protocol::PersistenceData>
 GetPersistenceDataForSymbol(lldb::SBSymbol &symbol) {
-  protocol::dap::PersistenceData persistence_data;
+  protocol::PersistenceData persistence_data;
   lldb::SBModule module = symbol.GetStartAddress().GetModule();
   if (!module.IsValid())
     return std::nullopt;
@@ -105,11 +105,11 @@ protocol::Breakpoint Breakpoint::ToProtocolBreakpoint() {
 
         // Add persistent data so that the breakpoint can be resolved
         // in future sessions.
-        std::optional<protocol::dap::PersistenceData> persistence_data =
+        std::optional<protocol::PersistenceData> persistence_data =
             GetPersistenceDataForSymbol(symbol);
         if (persistence_data) {
           source->adapterData =
-              protocol::dap::SourceLLDBData{std::move(persistence_data)};
+              protocol::SourceLLDBData{std::move(persistence_data)};
         }
       }
     }

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

@@ -114,7 +114,7 @@ ModuleSymbolsRequestHandler::Run(const ModuleSymbolsArguments &args) const {
   if (!module_spec.IsValid())
     return response;
 
-  std::vector<dap::Symbol> &symbols = response.symbols;
+  std::vector<Symbol> &symbols = response.symbols;
   lldb::SBModule module = dap.target.FindModule(module_spec);
   if (!module.IsValid())
     return llvm::make_error<DAPError>("Module not found");
@@ -125,7 +125,7 @@ ModuleSymbolsRequestHandler::Run(const ModuleSymbolsArguments &args) const {
     if (!symbol.IsValid())
       continue;
 
-    dap::Symbol dap_symbol;
+    Symbol dap_symbol;
     dap_symbol.userId = symbol.GetID();
     dap_symbol.type = SymbolTypeToString(symbol.GetType());
     dap_symbol.isDebug = symbol.IsDebug();

lldb/tools/lldb-dap/Protocol/DAPTypes.cpp

@@ -2,7 +2,7 @@
 
 using namespace llvm;
 
-namespace lldb_dap::protocol::dap {
+namespace lldb_dap::protocol {
 
 bool fromJSON(const llvm::json::Value &Params, PersistenceData &PD,
               llvm::json::Path P) {
@@ -60,4 +60,4 @@ llvm::json::Value toJSON(const Symbol &DS) {
   return result;
 }
 
-} // namespace lldb_dap::protocol::dap
+} // namespace lldb_dap::protocol

lldb/tools/lldb-dap/Protocol/DAPTypes.h

@@ -21,7 +21,7 @@
 #include <optional>
 #include <string>
 
-namespace lldb_dap::protocol::dap {
+namespace lldb_dap::protocol {
 
 /// Data used to help lldb-dap resolve breakpoints persistently across different
 /// sessions. This information is especially useful for assembly breakpoints,
@@ -80,6 +80,6 @@ struct Symbol {
 bool fromJSON(const llvm::json::Value &, Symbol &, llvm::json::Path);
 llvm::json::Value toJSON(const Symbol &);
 
-} // namespace lldb_dap::protocol::dap
+} // namespace lldb_dap::protocol
 
 #endif

lldb/tools/lldb-dap/Protocol/ProtocolRequests.h

@@ -994,7 +994,7 @@ bool fromJSON(const llvm::json::Value &, ModuleSymbolsArguments &,
 /// Response to `getModuleSymbols` request.
 struct ModuleSymbolsResponseBody {
   /// The symbols for the specified module.
-  std::vector<dap::Symbol> symbols;
+  std::vector<Symbol> symbols;
 };
 llvm::json::Value toJSON(const ModuleSymbolsResponseBody &);
 

lldb/tools/lldb-dap/Protocol/ProtocolTypes.h

@@ -340,7 +340,7 @@ struct Source {
   /// Additional data that a debug adapter might want to loop through the
   /// client. The client should leave the data intact and persist it across
   /// sessions. The client should not interpret the data.
-  std::optional<dap::SourceLLDBData> adapterData;
+  std::optional<SourceLLDBData> adapterData;
 
   // unsupported keys: origin, sources, checksums
 };

lldb/tools/lldb-dap/SourceBreakpoint.cpp

@@ -120,7 +120,7 @@ llvm::Error SourceBreakpoint::CreateAssemblyBreakpointWithSourceReference(
 }
 
 llvm::Error SourceBreakpoint::CreateAssemblyBreakpointWithPersistenceData(
-    const protocol::dap::PersistenceData &persistence_data) {
+    const protocol::PersistenceData &persistence_data) {
   lldb::SBFileSpec file_spec(persistence_data.module_path.c_str());
   lldb::SBFileSpecList comp_unit_list;
   lldb::SBFileSpecList file_spec_list;

lldb/tools/lldb-dap/SourceBreakpoint.h

@@ -55,7 +55,7 @@ class SourceBreakpoint : public Breakpoint {
   llvm::Error
   CreateAssemblyBreakpointWithSourceReference(int64_t source_reference);
   llvm::Error CreateAssemblyBreakpointWithPersistenceData(
-      const protocol::dap::PersistenceData &persistence_data);
+      const protocol::PersistenceData &persistence_data);
 
   // logMessage part can be either a raw text or an expression.
   struct LogMessagePart {

lldb/unittests/DAP/DAPTypesTest.cpp

@@ -15,7 +15,7 @@
 using namespace llvm;
 using namespace lldb;
 using namespace lldb_dap;
-using namespace lldb_dap::protocol::dap;
+using namespace lldb_dap::protocol;
 using lldb_private::roundtripJSON;
 
 TEST(DAPTypesTest, SourceLLDBData) {