Re: RFR: 8232066: Remove outdated code/methods from PKIX implementation [v2]

2021-07-29 Thread Sean Mullan
> Please review this fix to remove the Validator.VAR_PLUGIN_CODE_SIGNING > variant and related code that is no longer needed in the PKIX implementation. Sean Mullan has updated the pull request incrementally with one additional commit since the last revision: Remove NPE check in Validator.get

Re: RFR: 8232066: Remove outdated code/methods from PKIX implementation [v2]

2021-07-29 Thread Sean Mullan
On Thu, 29 Jul 2021 15:09:14 GMT, Weijun Wang 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. H

Re: RFR: 8232066: Remove outdated code/methods from PKIX implementation

2021-07-29 Thread Weijun Wang
On Thu, 29 Jul 2021 14:24:52 GMT, Sean Mullan wrote: > Please review this fix to remove the Validator.VAR_PLUGIN_CODE_SIGNING > variant and related code that is no longer needed in the PKIX implementation. Looks good. One nit: `Validator::getInstance` methods are internal. Does it really matte

RFR: 8232066: Remove outdated code/methods from PKIX implementation

2021-07-29 Thread Sean Mullan
Please review this fix to remove the Validator.VAR_PLUGIN_CODE_SIGNING variant and related code that is no longer needed in the PKIX implementation. - Commit messages: - 8232066: Remove outdated code/methods from PKIX implementation Changes: https://git.openjdk.java.net/jdk/pull/49