RFR: JDK-8291509 Minor cleanup could be done in sun.security [v8]

[Weijun Wang]

On Wed, 14 Sep 2022 13:48:19 GMT, Mark Powers <mpowers at openjdk.org> wrote:

>> https://bugs.openjdk.org/browse/JDK-8291509
>
> Mark Powers has updated the pull request incrementally with one additional commit since the last revision:
> 
>   final comments from Max

Almost done. I only have 2 comments on `ssl/ECDHClientKeyExchange.java` and `certpath/ForwardBuilder.java`, separately.

src/java.base/share/classes/sun/security/provider/certpath/ForwardBuilder.java line 572:

> 570:                 if (distanceTto1 < 0 || distanceTto2 < 0) {
> 571:                     if (distanceTto1 == distanceTto2) {
> 572:                         return -1;

This block can be modified like the previous one. This time it's about "at least one is negative".

-------------

PR: https://git.openjdk.org/jdk/pull/9972