Skip to content

let's deprecate view bounds by default (not just under -Xfuture) #10719

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

Closed
SethTisue opened this issue Feb 13, 2018 · 4 comments
Closed

let's deprecate view bounds by default (not just under -Xfuture) #10719

SethTisue opened this issue Feb 13, 2018 · 4 comments
Assignees
Milestone

Comments

@SethTisue
Copy link
Member

SethTisue commented Feb 13, 2018

scala/scala#2909 deprecated view bounds under -Xfuture, but nobody ever came back and removed the if (settings.future) check in Parsers.scala. as far as I know, that was just an oversight

there is a question of whether this change should target 2.12 or 2.13. I suggest we target 2.12.5 for the deprecation, and then actually remove it from the language in 2.13.

@SethTisue
Copy link
Member Author

someone want to tackle this...?

@SethTisue
Copy link
Member Author

anyone interested in tackling this might also like to tackle the slightly broader scala/scala-dev#471 and other pieces of scala/scala-dev#430

eed3si9n added a commit to eed3si9n/scala that referenced this issue Apr 8, 2018
Fixes scala/bug#10719

scala#2909 deprecated view bounds under -Xfuture. This deprecates it without it, and drops it under -Xsource:2.14.
@eed3si9n
Copy link
Member

eed3si9n commented Apr 8, 2018

I didn't see a PR yet on this, so I took it.

@eed3si9n eed3si9n self-assigned this Apr 8, 2018
eed3si9n added a commit to eed3si9n/scala that referenced this issue Apr 9, 2018
adriaanm added a commit to scala/scala that referenced this issue Apr 10, 2018
@SethTisue
Copy link
Member Author

I suggest we target 2.12.5 for the deprecation, and then actually remove it from the language in 2.13.

we went with a slower timetable: deprecated in 2.13, slated for removal in 2.14.

@SethTisue SethTisue added this to the 2.13.0-M4 milestone Apr 10, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants