On Thu, 28 Oct 2021 16:17:44 GMT, Hai-May Chao <hc...@openjdk.org> wrote:
>> This change does a few improvements to the output of `keytool -printcert >> -jarfile` command to help readability and diagnosis. > > 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()`. ------------- PR: https://git.openjdk.java.net/jdk/pull/6126