[foreign-memaccess+abi] RFR: 8293400: Modify static constructors in MemorySegment

[Maurizio Cimadamore]

On Mon, 12 Sep 2022 15:02:04 GMT, Per Minborg <duke at openjdk.org> wrote:

> This PR changes the `MemorySegment` static constructors to be more like the one of `ByteBuffer`.

I like how the change makes simple use cases simpler, by omitting the memory session parameter. This makes the `MemorySegment::allocateNative` API slightly asymmetric, but (a) it reduces the ways to do the same thing (now `MemorySession::allocate` is the star of the show) and (b) we could always add more static overloads later, if needed.

test/jdk/java/foreign/CallGeneratorHelper.java line 381:

> 379:     static Object makeArg(MemoryLayout layout, List<Consumer<Object>> checks, boolean check) throws ReflectiveOperationException {
> 380:         if (layout instanceof GroupLayout) {
> 381:             MemorySegment segment = MemorySession.openImplicit().allocate(layout);

This (and other similar calls) can be simplified to:

`MemorySegment.allocateNative(layout)`

I suggest looking for the pattern `MemorySession.openImplicit().allocate`

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

PR: https://git.openjdk.org/panama-foreign/pull/718