RFR: JDK-8210031: implementation for JVM Constants API

Mandy Chung mandy.chung at oracle.com
Fri Oct 19 01:55:41 UTC 2018

On 10/15/18 11:12 AM, Vicente Romero wrote:
> [1] http://cr.openjdk.java.net/~vromero/8210031/webrev.01

I reviewed java.lang.invoke change in details.  I have skimmed through 
the new classes.
I will look at the new tests next.

@since 12 is missing in the new APIs


1887 public final String toString() {
1888 // @@@ defer to concrete type for additional description
1889 // see https://bugs.openjdk.java.net/browse/JDK-8199149 You may 
want to take out this comment or L1889 as we can refer back to JDK-8199149. VarHandles.java

  169         // @@@ This is a little fragile assuming the base is the class

Maybe FieldStaticReadOnly and FieldStaticReadWrite constructor and
getStaticFieldFromBaseAndOffset method should take Class<?> refc
rather than Object base. FieldStaticReadXXX will do the cast when
calling getStaticFieldFromBaseAndOffset.

java.base module-info.java
    It'd be good to keep the exported APIs in alphabetical order.

    copyright header is missing

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.openjdk.java.net/pipermail/compiler-dev/attachments/20181018/182962bf/attachment.html>

More information about the compiler-dev mailing list