On Thu, 29 Jul 2021 15:09:14 GMT, Weijun Wang <wei...@openjdk.org> wrote:
> Looks good. One nit: `Validator::getInstance` methods are internal. Does it > really matter to check for null? Good point. Also, NPE will be thrown later when an `EndEntityChecker` is invoked and it needs to determine the variant to use. However, I had to change the `ConstructorTest` test not to test for this case anymore. ------------- PR: https://git.openjdk.java.net/jdk/pull/4934
