On Wed, 23 Apr 2025 16:23:00 GMT, Chen Liang <li...@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
