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

Exception copy is buggy #41

Open
azdavis opened this issue Jul 1, 2023 · 1 comment
Open

Exception copy is buggy #41

azdavis opened this issue Jul 1, 2023 · 1 comment
Labels
bug Something isn't working

Comments

@azdavis
Copy link
Owner

azdavis commented Jul 1, 2023

Environment

  • Millet version: v0.12.2

Steps to reproduce

Given

(*!
 * Doc for A
 *)
exception A
exception B = A

Try to e.g.:

Expected behavior

It works.

Actual behavior

It doesn't.

@azdavis azdavis added the bug Something isn't working label Jul 1, 2023
@azdavis
Copy link
Owner Author

azdavis commented Jul 2, 2023

I believe this affects more things than just exception copy. Basically whenever in the grammar we have a name or path that is a usage site that is not directly inside exp or pat, we don’t provide jump to def for that.

This is because the current way we do jump to def is by storing information for the exp or pat idx. We don’t have a way to store jump to def info for an an arbitrary path nested in some gramma structure, such as exception copy or where type.

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

No branches or pull requests

1 participant