On Wed, 16 Nov 2022 07:41:45 GMT, pandaapo <d...@openjdk.org> wrote: > Because `JarVerifier#setEagerValidation` will be removed, the field > `eagerValidation` will always be false. > > There are some codes using this field: > https://github.com/openjdk/jdk/blob/master/src/java.base/share/classes/java/util/jar/JarVerifier.java#L361-L381 > Can I remove the `eagerValidation` condition from `if(eagerValidation && > sigFileSigners.get(name) != null)`?
You can remove the whole if block since `eagerValidation` will never be true, and this block was only performed if it was true. > > If I can, can I remove this field from `JarVerifier` then? Yes. ------------- PR: https://git.openjdk.org/jdk/pull/11072