RFR: 8313743: Make fields final in sun.nio.ch [v3]
Brian Burkhalter
bpb at openjdk.org
Fri Aug 4 23:48:29 UTC 2023
On Fri, 4 Aug 2023 13:08:04 GMT, Andrey Turbanov <aturbanov at openjdk.org> wrote:
>> Found a few fields in `sun.nio.ch` package which could be declared `final`.
>>
>> Also I took oportunity to make a couple of cleanups:
>> 1. Converted `sun.nio.ch.OptionKey` to a record. Getters naming are already match.
>> 2. Removed `sun.nio.ch.WindowsAsynchronousSocketChannelImpl#addressSize`/`sun.nio.ch.WindowsSelectorImpl#addressSize` fields. Address size is already cached inside Unsafe (since [JDK-8221477](https://bugs.openjdk.org/browse/JDK-8221477)).
>
> Andrey Turbanov has updated the pull request incrementally with one additional commit since the last revision:
>
> 8313743: Make fields final in sun.nio.ch
>
> revert OptionKey back to regular class
Looks all right now. CI tier 1-3 tests succeeded on Windows-x64.
-------------
Marked as reviewed by bpb (Reviewer).
PR Review: https://git.openjdk.org/jdk/pull/14980#pullrequestreview-1563667580
More information about the nio-dev
mailing list