RFR: 8343040: Clean up references to JRE in the launcher code [v2]
Jaikiran Pai
jpai at openjdk.org
Wed Oct 30 01:52:43 UTC 2024
On Tue, 29 Oct 2024 17:40:35 GMT, Alan Bateman <alanb at openjdk.org> wrote:
>> Jaikiran Pai has updated the pull request incrementally with one additional commit since the last revision:
>>
>> Alan's review - fix code comment about Microsoft C Runtime Library
>
> src/java.base/windows/native/libjli/java_md.c line 227:
>
>> 225: /*
>> 226: * The Microsoft C Runtime Library needs to be loaded first. A copy is
>> 227: * assumed to be present in the JDK installation root directory.
>
> This comment isn't quite right, the C runtime library is in bin directory, not the root directory.
Hello Alan, I've now updated these 2 places where the code comments were explaining this library loading.
-------------
PR Review Comment: https://git.openjdk.org/jdk/pull/21704#discussion_r1821743648
More information about the core-libs-dev
mailing list