Long valueOf instead of new Long

Pavel Rappo pavel.rappo at oracle.com
Mon Jun 30 12:04:13 UTC 2014


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.golovnin at 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




More information about the core-libs-dev mailing list