On Wed, 21 Feb 2024 22:45:17 GMT, Brent Christian <[email protected]> wrote:
>> Per Minborg has updated the pull request incrementally with one additional
>> commit since the last revision:
>>
>> Suppress restricted warning
>
> src/java.desktop/unix/classes/sun/awt/X11/XEmbeddingContainer.java line 150:
>
>> 148: }
>> 149: } finally {
>> 150: Reference.reachabilityFence(data);
>
> Can you explain the need for this `reachabilityFence()`? I don't understand
> how `data` could become unreachable before the `Arena` is closed. Also, it's
> an in-scope local variable. Thanks!
You are right. As we have the arena, we do not need to invoke the
reachabilityFence.
-------------
PR Review Comment: https://git.openjdk.org/jdk/pull/17846#discussion_r1499530743
