search

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

Remove #Dim line from `Wrap in Try/Catch` refactor #305

Closed isc-bsaviano closed 7 months ago

isc-bsaviano commented 7 months ago

@isc-solon:

However, students have noticed that Refactor > Wrap in Try/Catch adds the "#dim ex" above the wrapping. And they ask "what's the #dim for?" And we've always explained exactly what it does and doesn't do. But then they see that it doesn't work! Do you think we should just remove the "#dim ex" from that refactor option?

This is a symptom of https://github.com/intersystems/language-server/issues/242#issuecomment-1222452436. I agree with Joel that we should remove this line because of the underlying IRIS issue.