RFR: 8325525: Create jtreg test case for JDK-8325203 [v3]

Vanitha B P duke at openjdk.org
Tue Jul 9 15:09:36 UTC 2024


On Tue, 9 Jul 2024 14:26:11 GMT, Alexey Semenyuk <asemenyuk at openjdk.org> wrote:

>> Vanitha B P has updated the pull request incrementally with one additional commit since the last revision:
>> 
>>   Addressed the review comments based on the inputs
>
> test/jdk/tools/jpackage/apps/ChildProcessAppLauncher.java line 39:
> 
>> 37:             System.out.println("Calc id=" + process.pid());
>> 38:             System.exit(0);
>> 39:         }
> 
> The better alternative would be:
> ```String calcPath = Path.of(System.getenv("SystemRoot"), "system32", "calc.exe").toString();```
> 
> This way NPE will be thrown if `SystemRoot` env variable is not set instead of silent exit, and `FS` field is not needed.

Agree, i will make the changes.

-------------

PR Review Comment: https://git.openjdk.org/jdk/pull/19536#discussion_r1670697556


More information about the core-libs-dev mailing list