Re: RFR: 8225093: Special property jdk.boot.class.path.append should not default to empty string

[David Holmes]

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