[Lldb-commits] [lldb] [lldb-dap] Add module symbol table viewer to VS Code extension #140626 (PR #153836)

[Ebuka Ezike via lldb-commits]

================
@@ -594,6 +594,20 @@ class CancelRequestHandler : public RequestHandler<protocol::CancelArguments,
   llvm::Error Run(const protocol::CancelArguments &args) const override;
 };
 
+class ModuleSymbolsRequestHandler
+    : public RequestHandler<
+          protocol::ModuleSymbolsArguments,
+          llvm::Expected<protocol::ModuleSymbolsResponseBody>> {
+public:
+  using RequestHandler::RequestHandler;
+  static llvm::StringLiteral GetCommand() { return "moduleSymbols"; }
----------------
da-viper wrote:

I think we could put the command behind some kind of namespace, to prevent name collisions in future dap-capabilities. something along `lldb_moduleSymbols` or `__moduleSymbols` 

https://github.com/llvm/llvm-project/pull/153836