On Wed, 29 Jun 2022 13:44:11 GMT, Weijun Wang <wei...@openjdk.org> wrote:
>> Jayashree Huttanagoudar has updated the pull request incrementally with one >> additional commit since the last revision: >> >> Address review comments > > src/java.base/share/classes/sun/security/x509/X509CertImpl.java line 742: > >> 740: >> 741: if (id.equalsIgnoreCase(INFO)) { >> 742: if ((attr.getSuffix() == null)) { > > You can remove the parentheses that were added last time. @wangweij I have added the test case under: test/jdk/sun/security/x509/X509CertImpl/JDK8283082.java As this is the first I am adding regression test so I am not sure whether the naming convention for the test case is right or not. ------------- PR: https://git.openjdk.org/jdk/pull/9306