Skip to content

SI-7187 deprecate eta-expansion of arity 0 #152

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
Closed
adriaanm opened this issue Jun 6, 2016 · 5 comments
Closed

SI-7187 deprecate eta-expansion of arity 0 #152

adriaanm opened this issue Jun 6, 2016 · 5 comments
Assignees
@adriaanm
Labels
t:sam
Milestone
2.12.0-RC1

Comments

@adriaanm
Copy link
Contributor

adriaanm commented Jun 6, 2016 •
edited
Loading

Lukas says in scala/scala#4822 (comment):

object Test {
  def foo(): () => String = () => ""
  val f: () => Any = foo

  def main(args: Array[String]): Unit = {
    println(f()) // <function0>
  }
}

Should we emit a deprecation in this case?
@adriaanm adriaanm added this to the 2.12.0-RC1 milestone Jun 6, 2016
@lrytz lrytz added the t:sam label Jun 10, 2016
@retronym
Copy link
Member

Dup? #179

@adriaanm adriaanm self-assigned this Jul 12, 2016
@adriaanm
Copy link
Contributor Author

adriaanm commented Jul 12, 2016 •
edited
Loading

see scala/bug#7187

@adriaanm adriaanm changed the title deprecation warning for eta-expansion of arity 0? eta-expansion of arity 0 doest not make sense: move away Jul 12, 2016
@adriaanm
Copy link
Contributor Author

to decide: how to handle migration/deprecation

@adriaanm adriaanm changed the title eta-expansion of arity 0 doest not make sense: move away eta-expansion of arity 0 does not make sense: move away Jul 12, 2016
@adriaanm adriaanm changed the title eta-expansion of arity 0 does not make sense: move away SI-7187 eta-expansion of arity 0 does not make sense: move away Jul 26, 2016
@adriaanm adriaanm changed the title SI-7187 eta-expansion of arity 0 does not make sense: move away SI-7187 deprecate eta-expansion of arity 0 Jul 28, 2016
@adriaanm
Copy link
Contributor Author

scala/scala#5309

@adriaanm
Copy link
Contributor Author

scala/scala#5327

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@adriaanm adriaanm

Labels
t:sam
Projects
None yet
Milestone
2.12.0-RC1
Development

No branches or pull requests
3 participants
@retronym @adriaanm @lrytz