eclipse-cdt-cloud / vscode-memory-inspector

vscode memory inspector
https://open-vsx.org/extension/eclipse-cdt/memory-inspector
Eclipse Public License 2.0
8 stars 12 forks source link

Memory Inspector display enhancements #48

Open jreineckearm opened 11 months ago

jreineckearm commented 11 months ago

Description

As discussed during the last CDT Cloud meeting, I'd like to share some thoughts on potential display enhancements which could make the Memory Inspector even more powerful than it is already today.

Additional information

colin-grant-work commented 9 months ago

At Ericsson, we're currently working on reintroducing the hover system that (among other things) displayed values in different formats, so we'd be happy to take on

Display of different data representations (decimal, octal, binary, float, double, signed vs unsigned, etc)

@gbodeen, re: some possible format additions.

colin-grant-work commented 9 months ago

Configure word size in bytes. As I understand this is currently fixed to 1-byte words.

It would be nice for this one if it were included as an extension point for debuggers that could answer a query about word size for a given session.

jreineckearm commented 9 months ago

At Ericsson, we're currently working on reintroducing the hover system that (among other things) displayed values in different formats, so we'd be happy to take on

That would be awesome, thank you @colin-grant-work ! Would this agree with your plans, @planger ?

jreineckearm commented 9 months ago

It would be nice for this one if it were included as an extension point for debuggers that could answer a query about word size for a given session.

I fully agree. Ideally, this knowledge would be transported via the MS-DAP protocol. Need to investigate which would be the best type or response. It certainly will have its charm also for other windows and debug functionality.

A short-term/intermediate alternative could be to add it as another setting. That could later pick up knowledge from the debugger through MS-DAP with an auto setting. Finding a good default for this setting may become a challenge. Maybe it could be steered by an agreed debug launch configuration item like it's used for description files for the Peripheral Inspector extension?

planger commented 9 months ago

At Ericsson, we're currently working on reintroducing the hover system that (among other things) displayed values in different formats, so we'd be happy to take on

That would be awesome, thank you @colin-grant-work ! Would this agree with your plans, @planger ?

Sure, that sounds great! I'll comment on https://github.com/eclipse-cdt-cloud/vscode-memory-inspector/issues/66 with our initial thoughts to keep the discussion in the right place.