[Lldb-commits] [lldb] [lldb-dap] Adding a modules explorer to lldb-dap ext. (PR #138977)

[Ebuka Ezike via lldb-commits]

da-viper wrote:

LGTM, 

Except the load address seems to be in decimal. It does not specify in the dap specification if it is decimal or hex. I assume it is implementation based. 

if it is the case we should show it as hex as that is default format for address in lldb. 


We could later extend it to show children instead of using the tool-tip.  

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