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