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