RFR: 8334137: Marlin: replace sun.misc.Unsafe memory access methods with FFM [v8]

Kevin Rushforth kcr at openjdk.org
Wed Jun 11 17:47:42 UTC 2025


On Wed, 11 Jun 2025 17:30:48 GMT, Andy Goryachev <angorya at openjdk.org> wrote:

>> Kevin Rushforth has updated the pull request incrementally with two additional commits since the last revision:
>> 
>>  - Use the global arena for the ALPHA_MASK arrays.
>>  - Revert "Initialize alpha maps in constructor to ensure they are created on the right thread (QuantumRenderer)"
>>    
>>    This reverts commit 2dd7a35d2eec16b04de1f4cb7aeba31be5d98a55.
>
> modules/javafx.graphics/src/main/java/com/sun/marlin/OffHeapArray.java line 59:
> 
>> 57:      */
>> 58:     OffHeapArray(final Object parent, final long len) {
>> 59:         this(parent, len, false);
> 
> I would suggest to remove this constructor.

Since global is the special case, it seemed easier to keep the existing constructor and only pass "global=true" in the one place that's needed -- the static initializer of `MaskMarlinAlphaConsumer`.

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

PR Review Comment: https://git.openjdk.org/jfx/pull/1814#discussion_r2140748003


More information about the openjfx-dev mailing list