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

SI-7605 Deprecate procedure syntax #3076

Merged
merged 1 commit into from Oct 29, 2013

Conversation

Projects
None yet
3 participants
@soc
Copy link
Member

soc commented Oct 23, 2013

This commit covers three cases:

  • constructor definitions (def this {...})
  • concrete method definitions (def foo {...})
  • abstract method declarations (def foo)

The deprecation is currently hidden behind -Xfuture pending IDE support
for migrating users from procedures to methods.

SI-7605 Deprecate procedure syntax
This commit covers three cases:
- constructor definitions      (def this {...})
- concrete method definitions  (def foo {...})
- abstract method declarations (def foo)

The deprecation is currently hidden behind -Xfuture pending IDE support
for migrating users from procedures to methods.
@soc

This comment has been minimized.

Copy link
Member Author

soc commented Oct 23, 2013

Review by @odersky / @jsuereth, please.

@JamesIry

This comment has been minimized.

Copy link
Contributor

JamesIry commented Oct 25, 2013

Would it make sense to remove the use of procedure syntax in the compiler and library in the same PR (separate commit) or is the job too big for one pass?

@soc

This comment has been minimized.

Copy link
Member Author

soc commented Oct 25, 2013

@JamesIry My plan was to first do the minimal thing to get the warning so that I can start looking into IDE support.
With IDE support working, I would have migrated our code. In some sense, it's eating our own dog food. If the IDE handles all migration cases from the standard library fine, I'd be optimistic that it is ready for prime-time.

So, yes, I could do it now manually, but I'd prefer using the IDE to verify that things work as expected.

@JamesIry

This comment has been minimized.

Copy link
Contributor

JamesIry commented Oct 25, 2013

LGTM, but would like some comment from the reviewers or @adriaanm

@ghost ghost assigned adriaanm Oct 25, 2013

@adriaanm

This comment has been minimized.

Copy link
Member

adriaanm commented Oct 25, 2013

LGTM2 -- thanks, Simon!

@adriaanm

This comment has been minimized.

Copy link
Member

adriaanm commented Oct 25, 2013

In this area, we should also deprecate implicit definitions without return type. They cause much more confusion (and supposed type checker slowdown) than it's worth (not to mention binary/behaviorally incompatible changes due to changes in type inference...)

@soc

This comment has been minimized.

Copy link
Member Author

soc commented Oct 29, 2013

@adriaanm Is this good to go or was your comment about adding a deprecation for implicit defs targeted at this commit?

@adriaanm

This comment has been minimized.

Copy link
Member

adriaanm commented Oct 29, 2013

this is good to go, just meant to say "while you're at it -- here's some more stuff I think we should deprecate"

@adriaanm

This comment has been minimized.

Copy link
Member

adriaanm commented Oct 29, 2013

Thanks!

JamesIry added a commit that referenced this pull request Oct 29, 2013

@JamesIry JamesIry merged commit 9b8d108 into scala:master Oct 29, 2013

1 check passed

default pr-scala Took 77 min.
Details

@t-mrt t-mrt referenced this pull request May 12, 2016

Merged

Deprecate procedure syntax #16

@scabug scabug referenced this pull request Apr 7, 2017

Closed

Deprecate procedure syntax #7605

eed3si9n added a commit to eed3si9n/scala that referenced this pull request Feb 14, 2018

Deprecate procedure syntax without -Xfuture
Ref scala/bug#7605

scala#3076 deprecated the procedure syntax, but only under -Xfuture flag. This deprecates it without it, and drops it under -Xsource:2.14.

eed3si9n added a commit to eed3si9n/scala that referenced this pull request Mar 4, 2018

Deprecate procedure syntax without -Xfuture
Ref scala/bug#7605

scala#3076 deprecated the procedure syntax, but only under -Xfuture flag. This deprecates it without it, and drops it under -Xsource:2.14.

adriaanm added a commit to eed3si9n/scala that referenced this pull request Apr 10, 2018

Deprecate procedure syntax unconditionally
Procedure syntax was deprecated under -Xfuture flag in scala#3076.
This deprecates it unconditionally, and drops it under -Xsource:2.14.

See scala/bug#7605

adriaanm added a commit to eed3si9n/scala that referenced this pull request Apr 10, 2018

Deprecate procedure syntax unconditionally
Procedure syntax was deprecated under -Xfuture flag in scala#3076.
This deprecates it unconditionally, and drops it under -Xsource:2.14.

See scala/bug#7605

adriaanm added a commit to eed3si9n/scala that referenced this pull request Apr 10, 2018

Deprecate procedure syntax unconditionally
Procedure syntax was deprecated under -Xfuture flag in scala#3076.
This deprecates it unconditionally, and drops it under -Xsource:2.14.

See scala/bug#7605

adriaanm added a commit to eed3si9n/scala that referenced this pull request Apr 11, 2018

Deprecate procedure syntax unconditionally
Procedure syntax was deprecated under -Xfuture flag in scala#3076.
This deprecates it unconditionally, and drops it under -Xsource:2.14.

See scala/bug#7605

eed3si9n added a commit to eed3si9n/scala that referenced this pull request Apr 24, 2018

Deprecate procedure syntax unconditionally
Procedure syntax was deprecated under -Xfuture flag in scala#3076.
This deprecates it unconditionally, and drops it under -Xsource:2.14.

See scala/bug#7605

eed3si9n added a commit to eed3si9n/scala that referenced this pull request Apr 25, 2018

Deprecate procedure syntax unconditionally
Procedure syntax was deprecated under -Xfuture flag in scala#3076.
This deprecates it unconditionally, and drops it under -Xsource:2.14.
See scala/bug#7605

To update the tests, this drops procedure syntax from test/ using ScalaFix

```
$ coursier launch ch.epfl.scala:scalafix-cli_2.12.3:0.5.3 -- -r ProcedureSyntax test
$ coursier launch ch.epfl.scala:scalafix-cli_2.12.4:0.5.10 -- -r ExplicitUnit test
```

@joan38 joan38 referenced this pull request May 4, 2018

Open

Remove Scala procedure syntax #4964

0 of 3 tasks complete
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment