RFR: 8265605: Cannot call BootLoader::loadClassOrNull before initPhase2 [v2]

Wed May 12 21:03:28 UTC 2021

On Wed, 12 May 2021 18:46:20 GMT, Alan Bateman <alanb at openjdk.org> wrote:

>> Ioi Lam has updated the pull request incrementally with one additional commit since the last revision:
>> 
>>   @AlanBateman comments
>
> src/java.base/share/classes/jdk/internal/loader/ClassLoaders.java line 60:
> 
>> 58:     // Sets the ServicesCatalog for the specified loader using archived objects.
>> 59:     private static void setArchivedServicesCatalog(
>> 60:             ArchivedClassLoaders archivedClassLoaders, ClassLoader loader)
> 
> Thanks for the update. The formatting on L60 is messed now, can't the ArchivedClassLoaders go on the previous line? I don't have any other comments, it's worked out well.

How about the latest version (4cd981c)? I removed the `archivedClassLoaders` argument; this also makes the callers simpler.

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

PR: https://git.openjdk.java.net/jdk/pull/3992