RFR: 8254566: Clarify the spec of ClassLoader::getClassLoadingLock for non-parallel capable loader [v2]

Mandy Chung mchung at openjdk.org
Fri Jun 30 00:22:14 UTC 2023


On Fri, 30 Jun 2023 00:06:08 GMT, David Holmes <dholmes at openjdk.org> wrote:

>> Mandy Chung has updated the pull request incrementally with one additional commit since the last revision:
>> 
>>   review comment
>
> src/java.base/share/classes/java/lang/ClassLoader.java line 658:
> 
>> 656:      * finer-grained locking scheme such that multiple threads may load classes
>> 657:      * concurrently without deadlocks.  For non-parallel-capable class loaders,
>> 658:      * the {@code ClassLoader} object is held during the class loading operations.
> 
> s/is held/is synchronized on/  - as that is the terminology loadClass uses.

Updated per your suggestion.

-------------

PR Review Comment: https://git.openjdk.org/jdk/pull/14720#discussion_r1247275274


More information about the core-libs-dev mailing list