RFR: 8355369: Remove setAccessible usage for setting final fields in java.util.concurrent [v2]

[Alan Bateman]

On Wed, 23 Apr 2025 16:23:00 GMT, Chen Liang <liach at openjdk.org> wrote:

>> Viktor Klang has updated the pull request incrementally with one additional commit since the last revision:
>> 
>>   Adding a releaseFence() to ConcurrentskipListSet.clone()
>
> Thanks. We can investigate how clone cleanups interact with truly final or strict finals in another issue.

> @liach @AlanBateman My proposal here would be to move forward with the proposed changes for now, and then once 25 is branched off there's an opportunity to take a broader look at `clone()`.

Okay with me. We can also add a non-public constructor to some of these classes and use it from readResolve so that we don't need readObject methods that mutate finals.

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

PR Comment: https://git.openjdk.org/jdk/pull/24821#issuecomment-2827444652