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

(regression) StackOverflowError with erroneous code #8890

Closed
scabug opened this issue Oct 7, 2014 · 7 comments
Closed

(regression) StackOverflowError with erroneous code #8890

scabug opened this issue Oct 7, 2014 · 7 comments
Assignees
@adriaanm
Labels
blocker typer
Milestone
2.11.3

Comments

@scabug
Copy link

scabug commented Oct 7, 2014 •
edited by retronym

The following code fails with SOF in 2.11.3-SNAPSHOT (it works fine with 2.11.2). This is particularly annoying in the IDE, when the error happens while typing.

Note that if I fix the error (complete Str to String), there is no crash.

I tried with with 2.11.3-20141006-171745-8d25e84c91

// Foo.scala
package foo

class A {
  /** The other */
  def bar(x: Int): Unit = ???
  def bar(x: Str): Unit = ???
}

class B {
  (new A).bar(0)
}
@scabug
Copy link
Author

scabug commented Oct 7, 2014

Imported From: https://issues.scala-lang.org/browse/SI-8890?orig=1
Reporter: @dragos
Affected Versions: 2.11.3

@scabug
Copy link
Author

scabug commented Oct 7, 2014

@dragos said:
I think this is a fairly recent regression. I started seeing this quite frequently today, but I didn't see this before. I'm bumping the priority, and updated the example to an even simpler one.

@scabug
Copy link
Author

scabug commented Oct 7, 2014

@gkossakowski said:
Thanks for the report!
Adriaan found that it regressed in scala/scala@67651e2

This is indeed a showstopper for Scala 2.11.3 release.

@scabug
Copy link
Author

scabug commented Oct 7, 2014

@adriaanm said:
Thanks for reporting and minimizing! Tentative fix: https://github.com/adriaanm/scala/compare/t8890?expand=1

@scabug
Copy link
Author

scabug commented Oct 7, 2014

@dragos said:
Thanks for the quick turnaround! It seems I was off regarding how recent it is... Today was my unlucky day :)

@scabug
Copy link
Author

scabug commented Oct 7, 2014

@adriaanm said:
Very glad you caught this one. Bit embarrassing...

@scabug
Copy link
Author

scabug commented Oct 8, 2014

@adriaanm said:
scala/scala#4037

@scabug scabug closed this as completed Oct 9, 2014
@scabug scabug added this to the 2.11.3 milestone Apr 7, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@adriaanm adriaanm

Labels
blocker typer
Projects
None yet
Milestone
2.11.3
Development

No branches or pull requests
2 participants
@adriaanm @scabug