On Wed, 29 Jun 2022 13:44:11 GMT, Weijun Wang <[email protected]> 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