On Wed, 20 Jan 2021 20:57:49 GMT, Sean Mullan <mul...@openjdk.org> wrote:
>> Hai-May Chao has updated the pull request incrementally with one additional >> commit since the last revision: >> >> Add nonce to the list of extensions > > src/java.base/share/classes/sun/security/provider/certpath/RevocationChecker.java > line 762: > >> 760: } >> 761: >> 762: tmpExtensions.add(nonceExt); > > If you only need the nonce, you could use List.of and save a little bit of > memory, ex: > > if (ocspExtensions.size() > 0) { > tmpExtensions = new > ArrayList<Extension>(ocspExtensions); > tmpExtensions.add(nonceExt); > } else { > tmpExtensions = List.of(nonceExt); > } Thanks for the review. Updated as suggested. ------------- PR: https://git.openjdk.java.net/jdk/pull/2039