On Tue, 15 Feb 2022 15:21:08 GMT, Michael Osipov <d...@openjdk.java.net> wrote:

>> Weijun Wang has updated the pull request incrementally with one additional 
>> commit since the last revision:
>> 
>>   string at 4th place
>
> test/jdk/sun/security/x509/OtherName/Parse.java line 89:
> 
>> 87:         int found = 0;
>> 88:         for (var san : x.getSubjectAlternativeNames()) {
>> 89:             if (san.size() == 4) {
> 
> Make it `>=` if this is going to change in the future..

OK, but this is always implementation-specific. Precisely, it's a "may" so I 
should not check the count.

-------------

PR: https://git.openjdk.java.net/jdk/pull/7167

Reply via email to