intersystems / language-server

Repository for the VS Code Language Server
https://marketplace.visualstudio.com/items?itemName=intersystems.language-server
Other
16 stars 6 forks source link

hovertext incorrect for methods with default values with $lb() #311

Closed mburstinisc closed 3 months ago

mburstinisc commented 3 months ago

If I have a method defined as: Method runBigVector(colCount As %Integer, vecsize As %Integer, rowcount As %Integer = 50, types As %List = {$lb("int", "double", "decimal", "string")}, skipLevel As %Integer = 0)

When I start typing in to calling that method, the types %List causes the hover text for filling in the rest of the signature: image

Note that from that image, you can see that I typed in 25 as the rowcount value, then 2 commas. The first comma advanced the hovertext to types, but the next comma (to accept the default value for types) advanced the hovertext to "double" being highlighted instead of skipLevel.

isc-bsaviano commented 3 months ago

Thanks for the screenshot.

isc-bsaviano commented 3 months ago

~@mburstinisc I just tried to reproduce this using the latest version (2.4.3) and I can't. Are you on the latest version? If so, can you send me the exact class you're using so I can test with it?~

EDIT: I can reproduce the issue using your exact method spec above. I'll investigate.