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

SI-6359 Deep prohibition of templates in value class #1300

Merged
merged 1 commit into from
Sep 13, 2012

Conversation

retronym
Copy link
Member

This seems to have been the intent of 95d532 / SI-5882.

The method local object/classes are also generated with outer pointers, so we should also prohibit them, at least for now.

Review by @paulp or @odersky

This seems to have been the intent of 95d532 / SI-5882.
@paulp
Copy link
Contributor

paulp commented Sep 13, 2012

It's a start on all the things we need to prohibit! LGTM.

@scala-jenkins
Copy link

Started jenkins job pr-rangepos at https://scala-webapps.epfl.ch/jenkins/job/pr-rangepos/439/

@scala-jenkins
Copy link

Started jenkins job pr-scala-testsuite-linux-opt at https://scala-webapps.epfl.ch/jenkins/job/pr-scala-testsuite-linux-opt/1148/

@jsuereth
Copy link
Contributor

+1

@scala-jenkins
Copy link

jenkins job pr-scala-testsuite-linux-opt: Success - https://scala-webapps.epfl.ch/jenkins/job/pr-scala-testsuite-linux-opt/1148/

@scala-jenkins
Copy link

jenkins job pr-rangepos: Success - https://scala-webapps.epfl.ch/jenkins/job/pr-rangepos/439/

jsuereth added a commit that referenced this pull request Sep 13, 2012
SI-6359 Deep prohibition of templates in value class
@jsuereth jsuereth merged commit 255e096 into scala:2.10.x Sep 13, 2012
@retronym
Copy link
Member Author

It's a start on all the things we need to prohibit! LGTM.

I'm just waiting for the Speakeasies to start popping up.

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

Successfully merging this pull request may close these issues.

5 participants