jmx-dev RFR: 8313854: Some tests in serviceability area fail on localized Windows platform [v3]

Kimura Yukihiro duke at openjdk.org
Tue Aug 8 23:43:01 UTC 2023


On Tue, 8 Aug 2023 23:17:27 GMT, Chris Plummer <cjplummer at openjdk.org> wrote:

>> Kimura Yukihiro has updated the pull request with a new target base due to a merge or a rebase. The pull request now contains three commits:
>> 
>>  - Merge master
>>  - 8313854: Some tests in serviceability area fail on localized Windows platform
>>  - 8313854: Some tests in serviceability area fail on localized Windows platform
>
> test/jdk/sun/tools/jhsdb/JShellHeapDumpTest.java line 162:
> 
>> 160:             ProcessBuilder pb = new ProcessBuilder(launcher.getCommand());
>> 161:             pb.command().add("-J-Duser.language=en");
>> 162:             pb.command().add("-J-Duser.country=US");
> 
> How about a comment:
> 
> /* Needed so we can properly parse the "Welcome to JShell" output. */

Thank you for your comment.
I added a comment to the code.

Thanks,
Kimura Yukihiro

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

PR Review Comment: https://git.openjdk.org/jdk/pull/15172#discussion_r1287785421


More information about the jmx-dev mailing list