RFR: 8334137: Marlin: replace sun.misc.Unsafe memory access methods with FFM [v5]
Kevin Rushforth
kcr at openjdk.org
Mon Jun 9 17:55:23 UTC 2025
On Mon, 9 Jun 2025 15:44:08 GMT, Per Minborg <pminborg at openjdk.org> wrote:
>> Kevin Rushforth has updated the pull request incrementally with one additional commit since the last revision:
>>
>> Review comments
>
> modules/javafx.graphics/src/main/java/com/sun/marlin/OffHeapArray.java line 47:
>
>> 45: /* members */
>> 46: private MemorySegment segment;
>> 47: private long length;
>
> Maybe we can remove the `length` field and instead use `segment.byteSize()`?
Thanks. I've done this.
> modules/javafx.graphics/src/main/java/com/sun/marlin/OffHeapArray.java line 106:
>
>> 104: */
>> 105: void resize(final long len) {
>> 106: Arena newArena = Arena.ofConfined();
>
> The comment "updating address is MANDATORY" can be removed.
Good catch. I've remove this and added a better description.
-------------
PR Review Comment: https://git.openjdk.org/jfx/pull/1814#discussion_r2136154724
PR Review Comment: https://git.openjdk.org/jfx/pull/1814#discussion_r2136155932
More information about the openjfx-dev
mailing list