On Sun, 20 Apr 2025 03:05:20 GMT, SendaoYan <s...@openjdk.org> wrote:
>> Joe Wang has updated the pull request incrementally with one additional >> commit since the last revision: >> >> remove unused variable pName; remove unused imports > > test/jaxp/javax/xml/jaxp/unittest/common/PropertiesTest.java line 47: > >> 45: * @summary Verifies JAXP API Properties as specified in the java.xml >> module. >> 46: * @library /javax/xml/jaxp/libs /javax/xml/jaxp/unittest /test/lib >> 47: * @run junit/othervm common.PropertiesTest > > `/othervm` seems do not needed It's necessary as the test may set System Properties. ------------- PR Review Comment: https://git.openjdk.org/jdk/pull/24732#discussion_r2051639843
