On Wed, 28 Oct 2026 16:53:49 GMT, Artur Barashev <[email protected]> wrote:
>> Roland Mesde has updated the pull request incrementally with one additional
>> commit since the last revision:
>>
>> Add Invalidation Details to Comment
>
> src/java.base/share/classes/sun/security/ssl/SSLSessionImpl.java line 826:
>
>> 824: }
>> 825:
>> 826: boolean isInvalid() {
>
> I suggest to rename `isInvalid()` to `isInvalidated()`. I find having both
> `isInvalid()` and `isValid()` methods in `SSLSessionImpl` confusing.
Done. I will updated the test logs once the test runs are complete, though
there is a low chance of change due to the simple refactor.
-------------
PR Review Comment: https://git.openjdk.org/jdk/pull/33447#discussion_r4404618081
