On Wed, 14 Sep 2022 13:48:19 GMT, Mark Powers <mpow...@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

Reply via email to