On Thu, 19 Dec 2024 12:38:27 GMT, liyazzi <d...@openjdk.org> wrote:

>> For two cases:
>> 
>> 1. When the ImageReaderFactory was loaded by local jdk,that means the 
>> ImageReaderFactory was loaded by boot class loader,then init the `Path 
>> BOOT_MODULES_JIMAGE` by using `sun.nio.fs.DefaultFileSystemProvider` which 
>> is obtained through reflection,due to it is in jdk internal.
>> 2. When loaded by a target jdk, such as jdk8 runtime, then use the Java 8 
>> compatible APIs: `FileSystems.getDefault()` to init the 
>> `BOOT_MODULES_JIMAGE` field.
>> Then we can avoid the circular dependencies in class loading caused by 
>> loading the defaultSystemProvider.
>
> liyazzi has updated the pull request incrementally with two additional 
> commits since the last revision:
> 
>  - Update the date of 'ImageReaderFactory'
>  - Update the date of 'ImageReaderFactory' and remove the import of 
> 'Disabled' in SetDefaultProvider test

@liyazzi 
Your change (at version 7ee321b13a386f67621a26cbe0fe87f709951efe) is now ready 
to be sponsored by a Committer.

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

PR Comment: https://git.openjdk.org/jdk/pull/22628#issuecomment-2554620271

Reply via email to