On Fri, 9 Jun 2023 15:01:47 GMT, Alan Bateman <[email protected]> wrote:
>> Jaikiran Pai has updated the pull request incrementally with one additional
>> commit since the last revision:
>>
>> newline at end of test file
>
> src/java.base/share/classes/jdk/internal/loader/URLClassPath.java line 186:
>
>> 184: * @apiNote Used to create the application class path.
>> 185: */
>> 186: URLClassPath(String cp, URLStreamHandler jarHandler, boolean
>> skipEmptyElements) {
>
> This is a special constructor for the application class path so shouldn't
> have the jarHandle parameter. This should allow you to drop the changes to
> ClassLoaders.
Thank you Alan for the suggestion. I've updated the PR accordingly. The test
continues to pass.
-------------
PR Review Comment: https://git.openjdk.org/jdk/pull/14395#discussion_r1224470836
