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.

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

Decide what to do about Unapply in the short term #1491

Closed
DavidGregory084 opened this issue Dec 10, 2016 · 2 comments
Closed

Decide what to do about Unapply in the short term #1491

DavidGregory084 opened this issue Dec 10, 2016 · 2 comments

Comments

@DavidGregory084
Copy link
Member

To deprecate our CartesianBuilders (part of #1363) we will be creating tuple apply syntax instances for Tuple1..22.

However, these instances cannot be resolved without -Ypartial-unification enabled.

It has been suggested that we extend our Unapply machinery to do this and I have prototyped this for Tuple2 in #1487 based on @julienrf's work.

I also had a quick look at templating Unapply2..N as part of our generated code but this was a bit of a horrible process and I'm not sure whether to continue it.

Should we put #1363 and similar issues on hold until Scala 2.11.9 is available and we can remove Unapply entirely?

@peterneyens
Copy link
Collaborator

This is related to #1073.

@peterneyens
Copy link
Collaborator

We removed Unapply in #1679.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants