On Fri, 7 Feb 2025 12:52:45 GMT, Weijun Wang <wei...@openjdk.org> wrote:
>> Mikhail Yankelevich has updated the pull request incrementally with one >> additional commit since the last revision: >> >> changes to improve readability and removal of unneded System.out.print > > test/jdk/sun/security/util/Pem/PemEncoding.java line 29: > >> 27: * @summary BASE64 encoded cert not correctly parsed with UTF-16 >> 28: * @library /test/lib >> 29: * @run main/othervm PemEncoding > > Why is `othervm` required? The test launches a sub-process and this VM is not > polluted. Agree, this is not needed. Will be changed in the next commit ------------- PR Review Comment: https://git.openjdk.org/jdk/pull/23492#discussion_r1946516119