On Thu, 28 Oct 2021 17:34:46 GMT, Weijun Wang <wei...@openjdk.org> wrote:

>> Hai-May Chao has updated the pull request incrementally with one additional 
>> commit since the last revision:
>> 
>>   Update output per review comment
>
> src/java.base/share/classes/sun/security/tools/keytool/Main.java line 2895:
> 
>> 2893:                             ss.add(signer);
>> 2894:                         }
>> 2895:                     }
> 
> I think the `while (entries.hasMoreElements())` block should end here, so 
> that you collect all signers of all entries before printing them out. Also, 
> there is no need to count `CodeSigner`, you can simply get it from 
> `ss.size()`.

Fixed.

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

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

Reply via email to