Re: RFR: JDK-8164547: Make java.lang.reflect.ClassLoaderValue public for internal use

2016-08-22 Thread Peter Levart
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 propos

Re: RFR: JDK-8164547: Make java.lang.reflect.ClassLoaderValue public for internal use

2016-08-22 Thread Alan Bateman
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

Re: RFR: JDK-8164547: Make java.lang.reflect.ClassLoaderValue public for internal use

2016-08-22 Thread Aleksey Shipilev
On 08/22/2016 01:15 PM, Peter Levart wrote: > Webrev: > http://cr.openjdk.java.net/~plevart/jdk9-dev/ClassLoaderValue.public/webrev.01/ Looks good to me. AFAIU, making the classes public under the non-exported jdk.internal.loader package is fine. Thanks, -Aleksey

RFR: JDK-8164547: Make java.lang.reflect.ClassLoaderValue public for internal use

2016-08-22 Thread Peter Levart
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