Show variable type on hover

[db4]

From VS Code 1.3.0 release notes:

Show variable type on hover: If a debug adapter returns the optional type attribute for the variable type, VS Code will show the value of this attribute when hovering over the variable's name.

Can we support this?

[hackwaly]

Sure! I'm downloading new version of VS Code now.

[hackwaly]

I can't get it work, even I set type? property. Seems variableRequest can't directly response a variable with type, but can response a variable with sub-variables that has type. Maybe it's a bug of new version of DebugProtocol. https://github.com/Microsoft/vscode-debugadapter-node/blob/master/protocol/src/debugProtocol.ts#L575

Are there any screenshots about showing variable type? Or there is an extension implemented this feature? microsoft/vscode-debugadapter-node#32

[hackwaly]

I consider close this issue because of ocamldebug only gives us the type of result variable self per display command, which is useless. Otherwise we need issue lots of display commands to retrieve type informations of result's children (or call member or property).

I implemented the simpler one in 046ee31, and it will release in v0.5.0.

[db4]

I consider close this issue because of ocamldebug only gives us the type of result variable self per display command, which is useless.

When you issue print command it responds with
<variable name> : <type> = <value>
You are parsing this anyway and can use <type> info as well.

[hackwaly]

Yes, I did it in v0.5.0 which is released.

[db4]

Yes, I did it in v0.5.0 which is released.

Hmm, I still don't see types in the debugger. E.g. for a record type a see <record> instead of the real type name. Maybe I should adjust some setting?

[hackwaly]

It's hidden by default in vscode.

Said https://code.visualstudio.com/updates/June_2016

Show Variable Type on Hover
If a debug extension supports variable types, we now show the type when hovering over the variable name.

[hackwaly]

Eh... found a bug: it shows array as list.

[hackwaly]

Maybe showing type instead of <record> being there is better.

[db4]

Ah, I see. Yes, it works when hovering over a variable in the watch window. But I expected it to work in an editor window also:

[db4]

This commit cb8d50b breaks hovering over plain variables: instead of the variable's value its type is shown now.

[hackwaly]

Sorry, my mistake. Fixed in 365f23c.

[db4]

Now it's OK. Thanks!