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

Bug with -Ykind-projector and * term #13141

Closed
japgolly opened this issue Jul 23, 2021 · 3 comments · Fixed by #13237
Closed

Bug with -Ykind-projector and * term #13141

japgolly opened this issue Jul 23, 2021 · 3 comments · Fixed by #13237
Labels
area:parser itype:bug Spree Suitable for a future Spree
Milestone

Comments

@japgolly
Copy link
Contributor

Compiler version

  • 3.0.1
  • 3.0.2-RC1

Minimized code

object A {

  class X { type Blah = Int }

  val * = new X

  val a: *.Blah = 2
}

Output

[error] 7 |  val a: *.Blah = 2
[error]   |          ^
[error]   |          end of statement expected but '.' found
[error] one error found

Expectation

Should compile. It does in Scala 3 without the -Ykind-projector flag.

@bishabosha
Copy link
Member

@neko-kai would you like to do this?

@bishabosha bishabosha added the Spree Suitable for a future Spree label Aug 2, 2021
@neko-kai
Copy link
Contributor

neko-kai commented Aug 2, 2021

@bishabosha I'll try to have a look this evening, but if I don't push a fix today, feel free to reassign because I won't have the time to look into this later.

neko-kai added a commit to 7mind/dotty that referenced this issue Aug 2, 2021
…r (was already a valid identifier)

Add test case; add a new test case for backticks handling (not related to the bug above)
@neko-kai
Copy link
Contributor

neko-kai commented Aug 2, 2021

@bishabosha Done in #13237 👍

bishabosha added a commit that referenced this issue Aug 2, 2021
…r-star-failure

Fix #13141 remove spurious coercion of `*` to a _type_ identifier (was already a valid identifier)
tanishiking pushed a commit to tanishiking/scala3 that referenced this issue Aug 10, 2021
…r (was already a valid identifier)

Add test case; add a new test case for backticks handling (not related to the bug above)
@Kordyjan Kordyjan added this to the 3.1.0 milestone Aug 2, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area:parser itype:bug Spree Suitable for a future Spree
Projects
None yet
4 participants