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

Peter Levart peter.levart at gmail.com
Tue Oct 11 06:03:45 UTC 2016


Hi Brent,


On 10/11/2016 12:44 AM, 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/
>
> -Brent

It would be marginally more efficient to check for parallelLockMap != 
null instead of looking up the CL implementation class in the 
WeakHashMap within a synchronized block, but I guess the performance of 
this method is not very important, is it?

Regards, Peter



More information about the core-libs-dev mailing list