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