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: invalid qualified modifier + default args = crash #6962

Closed
scabug opened this issue Jan 11, 2013 · 7 comments
Closed

regression: invalid qualified modifier + default args = crash #6962

scabug opened this issue Jan 11, 2013 · 7 comments
Assignees
@retronym
Labels
compiler crash usability
Milestone
2.12.2

Comments

@scabug
Copy link

scabug commented Jan 11, 2013

Works in 2.9.x fails in 2.10 and master.

trait Foo {
  def sth() {
    crashes()
  }
  protected[Bar] def crashes(withDefaultParam: Boolean = true) { }
}

Most definitely has to do with error reporting so I will take it unless someone is quicker.
@scabug
Copy link
Author

scabug commented Jan 11, 2013

Imported From: https://issues.scala-lang.org/browse/SI-6962?orig=1
Reporter: @hubertp
Affected Versions: 2.10.0, 2.11.0

@scabug
Copy link
Author

scabug commented May 16, 2014

@adriaanm said:
calling your bluff, hubert :)

@scabug
Copy link
Author

scabug commented Apr 6, 2015

@retronym said:
Additional test case from [~kenji yoshida] https://gist.github.com/xuwei-k/5ba3bcc8dc014f374777

@scabug
Copy link
Author

scabug commented Apr 6, 2015

@retronym said:
WIP: https://github.com/retronym/scala/tree/ticket/6962

@scabug scabug added this to the Backlog milestone Apr 7, 2017
@diesalbla
Copy link

@retronym This already works in Scala 2.12.x (REPL) and Scala 2.13.x (Neg test).

@diesalbla diesalbla mentioned this issue Dec 31, 2018
Merged
@som-snytt
Copy link

som-snytt commented Jan 5, 2019 •
edited

Stopped crashing in 2.12.2, emits only one error (not the default) in 2.13.0-M5.

Compare #10748

@som-snytt
Copy link

The progression was by the end of scala/scala#5816

I also tested that the linked fix from 2.13 applied on 2.12.1 doesn't crash.

@SethTisue SethTisue modified the milestones: Backlog, 2.12.2 Aug 27, 2019
@som-snytt som-snytt mentioned this issue Dec 18, 2020
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@retronym retronym

Labels
compiler crash usability
Projects
None yet
Milestone
2.12.2
Development

No branches or pull requests
5 participants
@retronym @SethTisue @som-snytt @diesalbla @scabug