[foreign-preview] RFR: 8280597: Refactor java.lang.invoke.MethodHandles::memoryAccessVarHandle [v2]
Maurizio Cimadamore
mcimadamore at openjdk.java.net
Mon Mar 7 14:33:07 UTC 2022
> This patch renames `MethodHandles::memoryAccessVarHandle` to `MethodHandles:memorySegmentViewVarHandle`, which is consistent with other methods in that class like `byteArrayViewVarHandle` and `byteBufferViewVarHandle`.
>
> When doing this, I also renamed several implementation classes and methods (included some automatically generated ones) to use the new names, and be more consistent with the naming scheme adopted for byte buffer var handle views.
>
> Finally, I tweaked the javadoc to use the new name. When doing this, I realized that the toplevel package javadoc was diving into var handles and combinators way too soon, so I have beefed up the memory segment dereference section (in the `MemorySegment` javadoc) and added a link from the package javadoc to that section.
>
> I'm still not 100% happy about how the memory segment javadoc looks, but for unrelated reasons, and I will deal with that in a separate pass.
Maurizio Cimadamore has updated the pull request incrementally with one additional commit since the last revision:
Address review comments
-------------
Changes:
- all: https://git.openjdk.java.net/panama-foreign/pull/663/files
- new: https://git.openjdk.java.net/panama-foreign/pull/663/files/9c10297b..b69731a9
Webrevs:
- full: https://webrevs.openjdk.java.net/?repo=panama-foreign&pr=663&range=01
- incr: https://webrevs.openjdk.java.net/?repo=panama-foreign&pr=663&range=00-01
Stats: 6 lines in 3 files changed: 0 ins; 0 del; 6 mod
Patch: https://git.openjdk.java.net/panama-foreign/pull/663.diff
Fetch: git fetch https://git.openjdk.java.net/panama-foreign pull/663/head:pull/663
PR: https://git.openjdk.java.net/panama-foreign/pull/663
More information about the panama-dev
mailing list