On Fri, 12 Jun 2026 15:15:27 GMT, Alan Bateman <[email protected]> wrote:

>> Dusan Balek has updated the pull request incrementally with one additional 
>> commit since the last revision:
>> 
>>   Probe lsof availability modified
>
> src/jdk.jdeps/share/classes/com/sun/tools/jdeps/JdepsConfiguration.java line 
> 330:
> 
>> 328:                 closeables.add(fileSystem);
>> 329:                 ClassLoader cl = 
>> fileSystem.provider().getClass().getClassLoader();
>> 330:                 if (cl != JdepsConfiguration.class.getClassLoader()
> 
> The jdk.jdeps module is mapped to the application class loader so this will 
> always be true. I think should be okay to test if cl is a URLClassLoader.

Updated.

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

PR Review Comment: https://git.openjdk.org/jdk/pull/31498#discussion_r3414567012

Reply via email to