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.

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

Restrict visibility of copy and apply #5472

Merged
merged 4 commits into from Nov 21, 2018
Merged

Restrict visibility of copy and apply #5472

merged 4 commits into from Nov 21, 2018

Commits on Nov 19, 2018

  1. Restrict visibility of copy and apply 

    Make the synthesized copy and apply methods of a case class have the
same visibility as its private constructor.
    @odersky
    odersky committed Nov 19, 2018
    Configuration menu
    Copy the full SHA
    019dccb View commit details
    Browse the repository at this point in the history

  2. Fix tests and bootstrapped code 

    There were two occurrences where we exploited that a case class with a private
constructor still had a public apply method. I believe both were mistakes.
    @odersky
    odersky committed Nov 19, 2018
    Configuration menu
    Copy the full SHA
    cb5c76e View commit details
    Browse the repository at this point in the history

Commits on Nov 21, 2018

  1. Drop unused definition

    @odersky
    odersky committed Nov 21, 2018
    Configuration menu
    Copy the full SHA
    d657bd7 View commit details
    Browse the repository at this point in the history

  2. Keep old scheme if option -language:Scala2 is set.

    @odersky
    odersky committed Nov 21, 2018
    Configuration menu
    Copy the full SHA
    6b840fc View commit details
    Browse the repository at this point in the history