RFR 8165793 : Provide an API to query if a ClassLoader is parallel capable

Alan Bateman Alan.Bateman at oracle.com
Tue Oct 11 15:25:46 UTC 2016


On 10/10/2016 23:44, Brent Christian wrote:

> On 10/10/16 2:30 PM, Mandy Chung wrote:
>>
>> The patch looks fine.  It would be good to add @see
>> #registerAsParallelCapable in this new method.  Also the first
>> “parallel capable” occurrance in the class spec and the
>> registerAsParallelCapable method spec to @linkplain
>> #isParallelCapable.
>
> Thanks for having a look, and for the suggestions.  I also added an 
> @see #isParallelCapable in registerAsParallelCapable().
>
> Webrev updated in place:
> http://cr.openjdk.java.net/~bchristi/8165793/webrev.00/
The javadoc looks okay to me. One thing that would be good is to beef up 
the test to cover more scenarios, esp. loader L1 extends loader L2 where 
you've got 4 combination of capable/non-capable to test.

-Alan


More information about the core-libs-dev mailing list