RFR: 8325858: Replace Unsafe usage in XEmbeddingContainer with FFM API [v3]
Per Minborg
pminborg at openjdk.org
Thu Feb 22 16:30:57 UTC 2024
On Wed, 21 Feb 2024 22:45:17 GMT, Brent Christian <bchristi at openjdk.org> 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
More information about the client-libs-dev
mailing list