RFR: 8342642: Class loading failure due to archived map issue in ModuleLoaderMap.Mapper [v2]

Jiangli Zhou jiangli at openjdk.org
Thu Oct 31 00:53:34 UTC 2024


On Tue, 29 Oct 2024 16:22:47 GMT, Aleksey Shipilev <shade at openjdk.org> wrote:

>> Jiangli Zhou has updated the pull request incrementally with one additional commit since the last revision:
>> 
>>   Adjust comment.
>
> src/java.base/share/classes/jdk/internal/module/ModuleLoaderMap.java line 92:
> 
>> 90:             } else if (PLATFORM_LOADER_NAME.equals(loader)) {
>> 91:                 return PLATFORM_CLASSLOADER;
>> 92:             } else { // BOOT_LOADER_INDEX
> 
> Drop the `// BOOT_LOADER_INDEX`?

Looks like the trivial update also needs to be reviewed, after the previous review/approval. @shipilev can you review again? Thanks

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

PR Review Comment: https://git.openjdk.org/jdk/pull/21722#discussion_r1823640088


More information about the core-libs-dev mailing list