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

Scala for loops parse error #5650

Closed
ievans opened this issue Jun 28, 2022 · 1 comment · Fixed by #5661
Closed

Scala for loops parse error #5650

ievans opened this issue Jun 28, 2022 · 1 comment · Fixed by #5661
Labels
lang:scala parsing Requires a fix in a parser, typically a tree-sitter or menhir grammar. parsing-pattern priority:low user:external requested by someone outside of r2c

Comments

@ievans
Copy link
Member

ievans commented Jun 28, 2022

I expected that for($EXPR2; $IDENT <- $GEN; $EXPR) {yield ... }
Or for(...; $IDENT <- $GEN; ...) {yield ... }

would parse as patterns but they don't as of 0.100: https://semgrep.dev/s/NoNx
@ievans ievans added parsing Requires a fix in a parser, typically a tree-sitter or menhir grammar. lang:scala parsing-pattern labels Jun 28, 2022
@r2c-demo
Copy link
Collaborator

This issue is synced in Linear at https://linear.app/r2c/issue/PA-1579/scala-for-loops-parse-error. Note: this link is for r2c use only and is not accessible publicly.

@ievans ievans added priority:low user:external requested by someone outside of r2c labels Jun 28, 2022
This was referenced Jun 29, 2022
Merged
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
lang:scala parsing Requires a fix in a parser, typically a tree-sitter or menhir grammar. parsing-pattern priority:low user:external requested by someone outside of r2c
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Scala: allow ellipsis in for loop headers semgrep/semgrep
2 participants
@ievans @r2c-demo