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

Find usage for match bodies #1260

Open
ice1000 opened this issue Dec 25, 2024 · 3 comments
Open

Find usage for match bodies #1260

ice1000 opened this issue Dec 25, 2024 · 3 comments
Labels
bug Something isn't working lsp

Comments

@ice1000
Copy link
Member

ice1000 commented Dec 25, 2024

image

@ice1000
Copy link
Member Author

ice1000 commented Dec 25, 2024

image

😨

Edit: this is ok, XYResolver is a visitor.

@ice1000 ice1000 added lsp bug Something isn't working labels Dec 25, 2024
@HoshinoTented
Copy link
Contributor

case Expr.Match match -> match.clauses().forEach(clause -> clause.patterns.forEach(
t -> visitPattern(t.term())));

It seems we didn't visit the body of a clause.

@ice1000
Copy link
Member Author

ice1000 commented Dec 29, 2024

case Expr.Match match -> match.clauses().forEach(clause -> clause.patterns.forEach(
t -> visitPattern(t.term())));

It seems we didn't visit the body of a clause.

The expr.forEach below should visit the clauses

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working lsp
Projects
None yet
Development

No branches or pull requests

2 participants