RFR: JDK-8164547: Make java.lang.reflect.ClassLoaderValue public for internal use
Hi, Several potential internal usages of currently package-private java.lang.reflect.ClassLoaderValue have been identified. Here is a proposal to move it to jdk.internal.loader package and make it public... Task: https://bugs.openjdk.java.net/browse/JDK-8164547 Webrev: http://cr.openjdk.java.net/~plevart/jdk9-dev/ClassLoaderValue.public/webrev.... Regards, Peter
On 08/22/2016 01:15 PM, Peter Levart wrote:
Webrev: http://cr.openjdk.java.net/~plevart/jdk9-dev/ClassLoaderValue.public/webrev....
Looks good to me. AFAIU, making the classes public under the non-exported jdk.internal.loader package is fine. Thanks, -Aleksey
On 22/08/2016 11:15, Peter Levart wrote:
Hi,
Several potential internal usages of currently package-private java.lang.reflect.ClassLoaderValue have been identified. Here is a proposal to move it to jdk.internal.loader package and make it public...
Task: https://bugs.openjdk.java.net/browse/JDK-8164547 Webrev: http://cr.openjdk.java.net/~plevart/jdk9-dev/ClassLoaderValue.public/webrev.... jdk.internal.loader looks right, the change looks good to me.
-Alan
Hi Aleksey, Alan, Thanks for reviewing. Pushed. Regards, Peter On 08/22/2016 12:27 PM, Alan Bateman wrote:
On 22/08/2016 11:15, Peter Levart wrote:
Hi,
Several potential internal usages of currently package-private java.lang.reflect.ClassLoaderValue have been identified. Here is a proposal to move it to jdk.internal.loader package and make it public...
Task: https://bugs.openjdk.java.net/browse/JDK-8164547 Webrev: http://cr.openjdk.java.net/~plevart/jdk9-dev/ClassLoaderValue.public/webrev.... jdk.internal.loader looks right, the change looks good to me.
-Alan
participants (3)
-
Alan Bateman
-
Aleksey Shipilev
-
Peter Levart