On Thu, 1 Sep 2022 03:31:32 GMT, Weijun Wang <wei...@openjdk.org> wrote:
>> Mark Powers has updated the pull request incrementally with one additional >> commit since the last revision: >> >> comments from Sean and Max > > src/java.base/share/classes/sun/security/tools/keytool/Main.java line 183: > >> 181: private List<String> weakWarnings = new ArrayList<>(); >> 182: >> 183: private Set<X509Certificate> trustedCerts = new HashSet<>(); > > Put all these `final`s in one group. If I do that, it will mess up existing groups, e.g. `// Warnings on weak algorithms etc`. I was able to move a few files around, but not many. ------------- PR: https://git.openjdk.org/jdk/pull/9972
