Skip to content
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

Positions of locally bound variables in binder #696

Open
denismerigoux opened this issue Sep 25, 2024 · 0 comments
Open

Positions of locally bound variables in binder #696

denismerigoux opened this issue Sep 25, 2024 · 0 comments
Labels
🔧 compiler Issue concerns the compiler 🤩 ide IDE plugins and tooling ❓ invalid This doesn't seem right

Comments

@denismerigoux
Copy link
Contributor

Right now, EVar nodes in the AST are accompanied with the source code position of the use of the variable in the source code. However, there is no way to link the Evar back to the position of its binder. This is annoying for the LSP as we cannot implement the "Go to definition" feature for locally bound variables (let ... in, aggregation operators, etc.). This is not high-priority for the moment.

Solutions to this problem include adding the binder position directly in the EVar node, or (more daunting) revisit the polymorphic bindlib encapsulation of variables to add the binder position inside it.

@denismerigoux denismerigoux added ❓ invalid This doesn't seem right 🔧 compiler Issue concerns the compiler 🤩 ide IDE plugins and tooling labels Sep 25, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
🔧 compiler Issue concerns the compiler 🤩 ide IDE plugins and tooling ❓ invalid This doesn't seem right
Projects
Status: Todo
Development

No branches or pull requests

1 participant