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

Infinite loop in scala.reflect.internal.Types$TypeVar.isGround(Types.scala:3024) #6499

Closed
scabug opened this issue Oct 10, 2012 · 7 comments
Closed
Assignees
Labels
Milestone

Comments

@scabug
Copy link

@scabug scabug commented Oct 10, 2012

The following snippet crashes in Typers with a Stackoverflow:

object Test {
  trait Matcher[X] {
    val	x: X
    def	foo(): X = { println("foo"); x }
  }
  val m = new Matcher {
    val x = Map():(scala.collection.Map[_, Int] with scala.collection.Map[K,Int] forSome { type K })
  }
  m.foo()
}

I discovered this while compiling scalatest. The file that crashes the compiler is: http://scalatest.googlecode.com/svn/branches/cmty2.10/src/test/scala/org/scalatest/matchers/ShouldContainValueSpec.scala

@scabug

This comment has been minimized.

Copy link
Author

@scabug scabug commented Oct 10, 2012

Imported From: https://issues.scala-lang.org/browse/SI-6499?orig=1
Reporter: @namin
Affected Versions: 2.10.0
Other Milestones: 2.10.0

@scabug

This comment has been minimized.

Copy link
Author

@scabug scabug commented Oct 10, 2012

@xeno-by said:
I'm taking a look, but for now I promote this issue to a blocker. Looks like really serious to me.

@scabug

This comment has been minimized.

Copy link
Author

@scabug scabug commented Oct 10, 2012

@paulp said:
Minimized:

object Test {
  Map(): Map[_, Int] with Map[_, Int]
}
@scabug

This comment has been minimized.

Copy link
Author

@scabug scabug commented Oct 10, 2012

@xeno-by said:
From Nada's report I understood that's a regression, hence the fuss. If it's not let's demote it

@scabug

This comment has been minimized.

Copy link
Author

@scabug scabug commented Oct 10, 2012

@namin said:
It is a regression, and a recent one. Paul's minimized snippet compiles fine with 2.9.2 AND 2.10.0-M7 but not with 2.10.0-wip.

@scabug

This comment has been minimized.

Copy link
Author

@scabug scabug commented Oct 10, 2012

@scabug

This comment has been minimized.

Copy link
Author

@scabug scabug commented Oct 12, 2012

@jsuereth said:
Paul's commit was merged in scala/scala@ee7f1e8

@scabug scabug closed this Oct 12, 2012
@scabug scabug added the blocker label Apr 7, 2017
@scabug scabug added this to the 2.10.0-M7 milestone Apr 7, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Linked pull requests

Successfully merging a pull request may close this issue.

None yet
2 participants
You can’t perform that action at this time.