-
Notifications
You must be signed in to change notification settings - Fork 1.1k
Scala 2 regression: free-form names in backticks crash at runtime #12729
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
Comments
Huh, name encoding kicks in for |
perhaps it's similar to an xml literal? |
Looks like a poor attempt at dealing with |
sjrd
pushed a commit
to dotty-staging/dotty
that referenced
this issue
Jun 22, 2021
In addition, reject `<init>` and `<clinit>` in the parser, so that users don't write them in source code.
mlachkar
added a commit
to mlachkar/dotty
that referenced
this issue
Jun 24, 2021
In addition, reject `<init>` and `<clinit>` in the parser, so that users don't write them in source code.
mlachkar
added a commit
to mlachkar/dotty
that referenced
this issue
Jun 24, 2021
In addition, reject `<init>` and `<clinit>` in the parser, so that users don't write them in source code.
mlachkar
added a commit
to mlachkar/dotty
that referenced
this issue
Jun 24, 2021
In addition, reject `<init>` and `<clinit>` in the parser, so that users don't write them in source code.
bishabosha
added a commit
that referenced
this issue
Jun 25, 2021
Fix #12729: Don't encode <init> and <clinit> only.
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Compiler version
With 3.0.0 (or 3.0.1-RC1 on scastie)
Minimized code
Output
In scastie this gives:
I've also seen it give something like:
Expectation
Since (fun) names like this worked fine in 2.13, I'd expect them to work fine in 3 too... perhaps they used to be name mangled?
This form in particular using
<name>
is useful for giving names to grammar rules in parsers.The text was updated successfully, but these errors were encountered: