Skip to content

Compiler crash: is not a simple name while compiling #6626

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

Closed
biboudis opened this issue Jun 6, 2019 · 6 comments
Closed

Compiler crash: is not a simple name while compiling #6626

biboudis opened this issue Jun 6, 2019 · 6 comments

Comments

@biboudis
Copy link
Contributor

biboudis commented Jun 6, 2019

class Test[T1] {
  class A[T2]

  object B  {
     def m(): A[B.type] = ???
  }
}
java.lang.UnsupportedOperationException: B[ModuleClass] is not a simple name while compiling tests/pos/ranks.scala
Exception in thread "main" java.lang.UnsupportedOperationException: B[ModuleClass] is not a simple name
        at dotty.tools.dotc.core.Names$DerivedName.asSimpleName(Names.scala:474)
        at dotty.tools.dotc.core.Names$DerivedName.asSimpleName(Names.scala:474)
        at dotty.tools.dotc.core.Names$TypeName.asSimpleName(Names.scala:441)
        at dotty.tools.dotc.transform.GenericSignatures$.classSig$2(GenericSignatures.scala:167)
        at dotty.tools.dotc.transform.GenericSignatures$.jsig$1(GenericSignatures.scala:228)
        at dotty.tools.dotc.transform.GenericSignatures$.boxedSig$1(GenericSignatures.scala:64)
        at dotty.tools.dotc.transform.GenericSignatures$.argSig$1(GenericSignatures.scala:145)
        at dotty.tools.dotc.transform.GenericSignatures$.classSig$1$$anonfun$1(GenericSignatures.scala:174)

scalac passes

@ashwinbhaskar
Copy link
Contributor

Can I give the fix a shot?

@biboudis
Copy link
Contributor Author

Yes, much appreciated! Try tracking the error using the guide can help

https://dotty.epfl.ch/docs/contributing/debugging.html

Either send a PR with the fix, ask questions on gitter or pot your progress here when you reach a point that you couldn't progress further 🎉

@ashwinbhaskar
Copy link
Contributor

thank you:) Can you assign the issue to me?

@biboudis
Copy link
Contributor Author

biboudis commented Sep 18, 2019

Feel free to work on it but if you feel that you don't want work on it anymore or it takes a long time (let's say more than a week without progress), don't fear the "Unassign me" button. This mark this up for grabs for others as well! First step, try if its still reproducible on master.

@ashwinbhaskar
Copy link
Contributor

@biboudis okay:) Yes it is reproducible on master

ashwinbhaskar added a commit to ashwinbhaskar/dotty that referenced this issue Sep 23, 2019
ashwinbhaskar added a commit to ashwinbhaskar/dotty that referenced this issue Sep 23, 2019
ashwinbhaskar added a commit to ashwinbhaskar/dotty that referenced this issue Sep 23, 2019
ashwinbhaskar added a commit to ashwinbhaskar/dotty that referenced this issue Sep 23, 2019
ashwinbhaskar added a commit to ashwinbhaskar/dotty that referenced this issue Sep 23, 2019
ashwinbhaskar added a commit to ashwinbhaskar/dotty that referenced this issue Sep 24, 2019
@ashwinbhaskar
Copy link
Contributor

can the PR be merged?

smarter added a commit that referenced this issue Sep 26, 2019
Fixes #6626. Pass sym.name to sanitizeName instead of sym.name.asSimpleName
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants