[Lldb-commits] [lldb] [lldb][lldb-dap] show modules pane if supported by the adapter (PR #140603)

[Ebuka Ezike via lldb-commits]

================
@@ -787,7 +787,7 @@
         {
           "id": "lldb-dap.modules",
           "name": "Modules",
-          "when": "inDebugMode && debugType == 'lldb-dap'",
+          "when": "inDebugMode && debugType == 'lldb-dap' && lldb-dap.showModules",
----------------
da-viper wrote:

I don't think we should because, each target has at least one module. This applies when launching, attaching and loading a core file. 

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