On Thu, 16 Dec 2021 17:33:29 GMT, Harold Seigel <hsei...@openjdk.org> wrote:
> Please review this fix for JDK-8225093 to set the default value of property
> jdk.boot.class.path.append to NULL instead of "". This fix was tested by
> running Mach5 tiers 1-2 on Linux, Mac OS, and Windows, and Mach5 tiers 3-5 on
> Linux x64. The fix was also tested by running the agent.c example in
> JDK-8224791.
>
> Thanks! Harold
Hi Harold,
Change looks good and I've Reviewed the CSR request.
One nit in the test.
Thanks,
David
test/hotspot/jtreg/runtime/BootClassAppendProp/GetBootClassPathAppendProp.java
line 40:
> 38:
> 39: public static void main(String[] args) throws Exception {
> 40: String vm_info_jvmti = getSystemProperty();
Nit: strange name and doesn't follow Java style rules. Won't "path" suffice?
-------------
Marked as reviewed by dholmes (Reviewer).
PR: https://git.openjdk.java.net/jdk/pull/6868
