RFR: 8349532: Refactor ./util/Pem/encoding.sh to run in java [v2]

Mikhail Yankelevich duke at openjdk.org
Fri Feb 7 13:23:10 UTC 2025


On Fri, 7 Feb 2025 12:52:45 GMT, Weijun Wang <weijun at 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


More information about the security-dev mailing list