If a test run finishes fine, I'll push this version (no Unsafe*LongFieldFieldAccessorImpl.java files included):
http://cr.openjdk.java.net/~prappo/8048267/webrev.04/ -Pavel On 30 Jun 2014, at 11:31, Andrej Golovnin <andrej.golov...@gmail.com> wrote: > Hi Pavel, > > http://cr.openjdk.java.net/~prappo/8048267/webrev.03/src/share/classes/sun/reflect/UnsafeLongFieldAccessorImpl.java.sdiff.html > http://cr.openjdk.java.net/~prappo/8048267/webrev.03/src/share/classes/sun/reflect/UnsafeQualifiedLongFieldAccessorImpl.java.sdiff.html > http://cr.openjdk.java.net/~prappo/8048267/webrev.03/src/share/classes/sun/reflect/UnsafeQualifiedStaticLongFieldAccessorImpl.java.sdiff.html > http://cr.openjdk.java.net/~prappo/8048267/webrev.03/src/share/classes/sun/reflect/UnsafeStaticLongFieldAccessorImpl.java.sdiff.html > > Please don't change the UnsafeXXXFieldAccessorImpl classes. > This classes should be changed as a part of the fix of the issue 5043030. > The patch for this issue has been already submitted, see here: > > http://mail.openjdk.java.net/pipermail/core-libs-dev/2014-June/027311.html > > When Joel is back, he will continue to work on it. > > Otherwise it looks good to me. > > Best regards, > Andrej Golovnin