On Sat, 29 Apr 2023 00:09:55 GMT, Naoto Sato <na...@openjdk.org> wrote:
>> Joe Wang has updated the pull request incrementally with one additional
>> commit since the last revision:
>>
>> removing the line that changed locale. It's not necessary as the concern
>> was the number of errors reported
>
> test/jaxp/javax/xml/jaxp/unittest/validation/ErrorHandlingTest.java line 86:
>
>> 84: @Test
>> 85: public void test() throws Exception {
>> 86: Locale.setDefault(ENGLISH);
>
> I believe this test is run in `othervm` mode, but still, it would be
> preferred to restore the default locale after the test
Thanks! The line is removed.
-------------
PR Review Comment: https://git.openjdk.org/jdk/pull/13722#discussion_r1180900119
