On 08/05/2012 11:30, David Holmes wrote:
....
I was forgetting there's another major consideration here too, and that
is that this code sync's up with Doug Lea's JSR-166 CVS repository. The
changes I made are independent of the JDK used, but if I refactored
things into ReflectUtil then the code would only be valid on a JDK with
that update - which means Doug would need to maintain a different
version of this fix for older JDKs.
Right, unless there is a good reason we should try not to maintain a
separate fork here. Sync'ing is already a pain!
-Chris.
David
-----
David
-----
-Ulf
Am 08.05.2012 09:36, schrieb [email protected]:
Changeset: 48513d156965
Author: dholmes
Date: 2012-05-08 02:59 -0400
URL: http://hg.openjdk.java.net/jdk8/tl/jdk/rev/48513d156965
7103570: AtomicIntegerFieldUpdater does not work when SecurityManager
is installed
Summary: Perform class.getField inside a doPrivileged block
Reviewed-by: chegar, psandoz
!
src/share/classes/java/util/concurrent/atomic/AtomicIntegerFieldUpdater.java
!
src/share/classes/java/util/concurrent/atomic/AtomicLongFieldUpdater.java
!
src/share/classes/java/util/concurrent/atomic/AtomicReferenceFieldUpdater.java
+ test/java/util/concurrent/atomic/AtomicUpdaters.java
