On Tue, 22 Nov 2022 19:53:44 GMT, Sean Mullan <[email protected]> wrote:
>> Weijun Wang has updated the pull request incrementally with one additional
>> commit since the last revision:
>>
>> IssuerAlternativeNameExtension names
>
> src/java.base/share/classes/sun/security/x509/SubjectAlternativeNameExtension.java
> line 91:
>
>> 89: */
>> 90: public SubjectAlternativeNameExtension(Boolean critical,
>> GeneralNames names)
>> 91: throws IOException {
>
> `GeneralNames` should probably also be modified so that it cannot contain an
> empty or null List.
It still has `add` which is used in multiple places. I'll see if I can remove
this method and make it immutable.
-------------
PR: https://git.openjdk.org/jdk/pull/11137