-
Notifications
You must be signed in to change notification settings - Fork 55
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
Parser gets stuck #392
Comments
Maybe try binary search to minimize the example, and/or git bisect to find out when this has occurred. |
Good points! I can confirm |
@ObserverOfTime Thanks! |
eed3si9n
added a commit
to eed3si9n/tree-sitter-scala
that referenced
this issue
Apr 13, 2024
**Problem** 350 introduced a dynamic precedence to resolve the conflict of Scala 2 and Scala 3 grammar for if-then. **Solution** This removes the dynamic precedence and replaces it with prec.right around `then` side. **Note** There's a report of parser getting stuck tree-sitter#392 and and I was hoping that removing dynamic precedence would fix that, but it didn't seem to.
eed3si9n
added a commit
to eed3si9n/tree-sitter-scala
that referenced
this issue
Jun 30, 2024
**Problem** 350 introduced a dynamic precedence to resolve the conflict of Scala 2 and Scala 3 grammar for if-then. **Solution** This removes the dynamic precedence and replaces it with prec.right around `then` side. **Note** There's a report of parser getting stuck tree-sitter#392 and and I was hoping that removing dynamic precedence would fix that, but it didn't seem to.
eed3si9n
added a commit
to eed3si9n/tree-sitter-scala
that referenced
this issue
Jun 30, 2024
**Problem** 350 introduced a dynamic precedence to resolve the conflict of Scala 2 and Scala 3 grammar for if-then. **Solution** This removes the dynamic precedence and replaces it with prec.right around `then` side. **Note** There's a report of parser getting stuck tree-sitter#392 and and I was hoping that removing dynamic precedence would fix that, but it didn't seem to.
eed3si9n
added a commit
to eed3si9n/tree-sitter-scala
that referenced
this issue
Jun 30, 2024
**Problem** 350 introduced a dynamic precedence to resolve the conflict of Scala 2 and Scala 3 grammar for if-then. **Solution** This removes the dynamic precedence and replaces it with prec.right around `then` side. **Note** There's a report of parser getting stuck tree-sitter#392 and and I was hoping that removing dynamic precedence would fix that, but it didn't seem to.
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Commit of tree-sitter-scala you tested this on
70b4fe6
A code sample showing the error
Hi,
It looks like the the grammar on commit
70b4fe63c4
is causing an infinite loop e.g. running withintree-sitter-scala
:Please see attached the sample.scala:
sample.scala.txt
Could someone help explaining how this can be debugged?
Thanks for your time and suggestions
Show the error node
No error message as execution of
tree-sitter
is stuck.What do you expect the tree to look like
No response
Where are you experiencing this error?
Running tree-sitter tool in CLI.
The text was updated successfully, but these errors were encountered: