Re: RFR 8191173 : (cl) Clarify or remove "for delegation" in ClassLoader spec

2017-11-23 Thread mandy chung
On 11/21/17 12:41 PM, Brent Christian wrote: Hi, Please review my change to this small bit of ClassLoader spec that can be tidied up, as noticed by Martin. Issue: https://bugs.openjdk.java.net/browse/JDK-8191173 Webrev: http://cr.openjdk.java.net/~bchristi/8191173/webrev.00/ In

Re: RFR 8191173 : (cl) Clarify or remove "for delegation" in ClassLoader spec

2017-11-21 Thread Alan Bateman
On 21/11/2017 20:41, Brent Christian wrote: Hi, Please review my change to this small bit of ClassLoader spec that can be tidied up, as noticed by Martin. Issue: https://bugs.openjdk.java.net/browse/JDK-8191173 Webrev: http://cr.openjdk.java.net/~bchristi/8191173/webrev.00/ This looks

Re: RFR 8191173 : (cl) Clarify or remove "for delegation" in ClassLoader spec

2017-11-21 Thread David Holmes
Seems fine. Thanks, David On 22/11/2017 6:41 AM, Brent Christian wrote: Hi, Please review my change to this small bit of ClassLoader spec that can be tidied up, as noticed by Martin. Issue: https://bugs.openjdk.java.net/browse/JDK-8191173 Webrev:

Re: RFR 8191173 : (cl) Clarify or remove "for delegation" in ClassLoader spec

2017-11-21 Thread Martin Buchholz
Looks good! On Tue, Nov 21, 2017 at 12:41 PM, Brent Christian < brent.christ...@oracle.com> wrote: > Hi, > > Please review my change to this small bit of ClassLoader spec that can be > tidied up, as noticed by Martin. > > Issue: > https://bugs.openjdk.java.net/browse/JDK-8191173 > Webrev: >

RFR 8191173 : (cl) Clarify or remove "for delegation" in ClassLoader spec

2017-11-21 Thread Brent Christian
Hi, Please review my change to this small bit of ClassLoader spec that can be tidied up, as noticed by Martin. Issue: https://bugs.openjdk.java.net/browse/JDK-8191173 Webrev: http://cr.openjdk.java.net/~bchristi/8191173/webrev.00/ In java.lang.ClassLoader these methods: getParent()