On Wed, 10 Jan 2024 17:00:42 GMT, Kevin Driver <[email protected]> wrote:
>> Sean Mullan has updated the pull request incrementally with one additional
>> commit since the last revision:
>>
>> Add more comments. Remove unnecessary import.
>
> src/java.base/share/classes/sun/security/provider/certpath/ForwardBuilder.java
> line 523:
>
>> 521: }
>> 522: if (tAo1 != null) {
>> 523: if (tAo2 != null) {
>
> Maybe it's unfamiliarity with this code, but I think we could use some
> comments here, especially since a similar condition is tested on line 534
> (except tAo1 would be null in that case?).
Added more comments.
-------------
PR Review Comment: https://git.openjdk.org/jdk/pull/17248#discussion_r1448917072
