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.

Sign up for GitHub

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

Jump to bottom

Binary operator parsing precedence bug #258

Closed
wolfadex opened this issue Sep 10, 2024 · 1 comment
Closed

Binary operator parsing precedence bug #258

wolfadex opened this issue Sep 10, 2024 · 1 comment
Labels
bug

Comments

@wolfadex
Copy link

Describe the bug
a + b /= 0 is marked as invalid but the Elm compiler and elm-test don't have an issue with it. By wrapping the math in parens, like (a + b) /= 0, then it parses fine. This also applies to a + b + c /= 0 where wrapping like (a + b +c ) /= 0 fixes it.

Additional context
This was originally filed in elm-review jfmengels/elm-review#177 as that's where the bug was found. Elm Slack thread for discussion reference https://elmlang.slack.com/archives/C010RT4D1PT/p1725977809992319.
@wolfadex wolfadex mentioned this issue Sep 10, 2024
Closed
@jfmengels jfmengels mentioned this issue Sep 10, 2024
Closed
@jfmengels jfmengels added the bug label Sep 10, 2024
@jfmengels jfmengels changed the title binary operator parsing precendence bug Binary operator parsing precedence bug Sep 10, 2024
@lue-bird lue-bird mentioned this issue Sep 10, 2024
Merged
lue-bird added a commit to lue-bird/elm-syntax that referenced this issue Sep 10, 2024
@lue-bird
apply tests for stil4m#258 by @jfmengels
d604554
@jfmengels
Copy link
Collaborator

Fixed in 7.3.6

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
bug
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Successfully parse 'a + b /= c' stil4m/elm-syntax
2 participants
@wolfadex @jfmengels