RFR: 8338626: ClassLoaderExt::process_jar_manifest() should allow / separator on Windows [v3]
Calvin Cheung
ccheung at openjdk.org
Thu Sep 12 15:42:43 UTC 2024
On Thu, 12 Sep 2024 06:11:40 GMT, David Holmes <dholmes at openjdk.org> wrote:
>> Calvin Cheung has updated the pull request incrementally with one additional commit since the last revision:
>>
>> @dholmes-ora comment and test update
>
> test/hotspot/jtreg/runtime/cds/appcds/ClassPathAttr.java line 87:
>
>> 85: });
>> 86:
>> 87: // Test handling of forward slash ('/') file separator for Class-Path attribute on Windows.
>
> Again this comment is wrong/misleading - the / is in the classpath entry
Updated the comment.
-------------
PR Review Comment: https://git.openjdk.org/jdk/pull/20924#discussion_r1757128171
More information about the hotspot-runtime-dev
mailing list