-
Notifications
You must be signed in to change notification settings - Fork 789
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Support F# expression in debugger eval #2544
Comments
Tagging - we're still in the process of figuring out things internal w.r.t teams and what they're working on, but this is something we care deeply about and will be looking at. Just a note, it's a considerable amount of work. A proper EE for F# will take time to complete and get right. |
@enricosada An F# aware EE would be a fantastic thing ... with the roslyn EE it may not be quite the humongous work-item it would have been in the past. We are working through what it would take to make this happen internally, or what it would take to enable the OSS community to drive it. Kevin |
Gregg's comment is promising. Looking at the API it doesn't look like it would be insanely difficult - considering some of the Roslyn features that have been written from scratch over the past few months this should be simple :p |
I just logged this as todo because the sample is really well documented. Can be useful later |
We should consider what the necessary entry point to the F# compiler service is. The signature needed will be something like this:
though some additional context may be needed to bind names, work out NameResolutionEnvironment etc. Walking |
For future reference: the C#/VB expression evaluator is here: https://github.com/dotnet/roslyn/tree/main/src/ExpressionEvaluator |
Chat with @tmat
|
To support F# expressions in debugger eval (watch/etc) need an F# expression evaluator
A sample evaluator https://github.com/Microsoft/ConcordExtensibilitySamples/wiki/Managed-Expression-Evaluator-Sample
As per info by @gregg-miskelly (debugger team) in dotnet/vscode-csharp#1257 (comment) i quote directly because good ihmo (is already language agnostic, easier to do than before, debugger team may assist a bit to start)
The text was updated successfully, but these errors were encountered: