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

@volatile + final var = exception! #2418

Closed
scabug opened this issue Sep 30, 2009 · 6 comments
Closed

@volatile + final var = exception! #2418

scabug opened this issue Sep 30, 2009 · 6 comments
Labels
Milestone

Comments

@scabug
Copy link

@scabug scabug commented Sep 30, 2009

using this code on 2.8 latest gives this

class Foo{@volatile final var x=10};new Foo
Welcome to Scala version 2.8.0.r18830-b20090930020951 (Java HotSpot(TM) 64-Bit Server VM, Java 1.6.0_16).
Type in expressions to have them evaluated.
Type :help for more information.

scala> class Foo{@volatile final var x=10};new Foo
java.lang.ClassFormatError: Illegal field modifiers in class Foo: 0x52
	at java.lang.ClassLoader.defineClass1(Native Method)
	at java.lang.ClassLoader.defineClass(ClassLoader.java:621)
	at java.lang.ClassLoader.defineClass(ClassLoader.java:466)
	at scala.tools.nsc.interpreter.AbstractFileClassLoader.findClass(AbstractFileClassLoader.scala:31)
	at java.lang.ClassLoader.loadClass(ClassL...
scala> 

I noticed #2095
not sure if they are related.

@scabug
Copy link
Author

@scabug scabug commented Sep 30, 2009

Imported From: https://issues.scala-lang.org/browse/SI-2418?orig=1
Reporter: Ben Jackman (bjackman)
Assignee: @JamesIry

@scabug
Copy link
Author

@scabug scabug commented Nov 26, 2009

@dragos said:
(In r19899) Closed #2418

@scabug
Copy link
Author

@scabug scabug commented Jul 26, 2012

@scabug
Copy link
Author

@scabug scabug commented Jan 23, 2013

@adriaanm said:
reopening for potential backport to 2.10.1-RC1

@scabug
Copy link
Author

@scabug scabug commented Jan 31, 2013

@JamesIry said:
I'll take the backport just to clear it off our plate.

@scabug
Copy link
Author

@scabug scabug commented Jan 31, 2013

@JamesIry said:
backport scala/scala#2037

@scabug scabug closed this Feb 1, 2013
@scabug scabug added the backport label Apr 7, 2017
@scabug scabug added this to the 2.10.1 milestone Apr 7, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Linked pull requests

Successfully merging a pull request may close this issue.

None yet
1 participant