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

@volatile var cannot be defined in case class with confusing error message #8873

Closed
scabug opened this issue Oct 1, 2014 · 4 comments
Closed

@volatile var cannot be defined in case class with confusing error message #8873

scabug opened this issue Oct 1, 2014 · 4 comments
Assignees
@adriaanm
Labels
annotations backend has PR
Milestone
2.12.0-RC1

Comments

@scabug
Copy link

scabug commented Oct 1, 2014 •
edited by retronym 

scala> case class X(@volatile var x:Int)
<console>:7: error: values cannot be volatile
       case class X(@volatile var x:Int)
                                  ^

// where compiler see values ?
@scabug
Copy link
Author

scabug commented Oct 1, 2014

Imported From: https://issues.scala-lang.org/browse/SI-8873?orig=1
Reporter: ruslan shevchenko (rssh)
Affected Versions: 2.11.2
Duplicates #8826

@scabug
Copy link
Author

scabug commented Oct 2, 2014

@gourlaysama said:
Hmm, that's a bug in case class synthetic methods. The compiler wrongly keeps the @volative annotation in the generated apply constructor:

object X extends AbstractFunction1[Int,X] with Serializable {
...
def apply(@volatile x: Int): X = new X(x) // that's wrong
}

@scabug
Copy link
Author

scabug commented Nov 4, 2014

@retronym said:
Reassinging to 2.11.5, 2.11.4 has already been released.

@scabug
Copy link
Author

scabug commented Aug 12, 2016

@adriaanm said:
picked this one up in scala/scala#5294

@scabug scabug closed this as completed Sep 6, 2016
@scabug scabug added this to the 2.12.0-RC1 milestone Apr 7, 2017
@ijuma ijuma mentioned this issue Aug 1, 2017
Closed
@srowen srowen mentioned this issue Feb 21, 2019
Closed
@alexandrfox alexandrfox mentioned this issue Dec 1, 2019
Merged
3 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@adriaanm adriaanm

Labels
annotations backend has PR
Projects
None yet
Milestone
2.12.0-RC1
Development

No branches or pull requests
2 participants
@adriaanm @scabug