[jdk19] Integrated: 8289558: Need spec clarification of j.l.foreign.*Layout

[Maurizio Cimadamore]

On Fri, 1 Jul 2022 11:03:23 GMT, Maurizio Cimadamore <mcimadamore at openjdk.org> wrote:

> This patch fixes few javadoc issues in the memory layout API.
> The main issues are that `SequenceLayout::flatten` and `SequenceLayout::reshape` still mention failures caused by a lack of size. But that condition is no longer possible in the new API.
> 
> The javadoc of `ValueLayout::arrayElementVarHandle` is suboptimal and can be clarified - UOE is only thrown if the value layout alignment is bigger than its size.
> 
> Finally, the `MemoryLayout::equals` method does not mention value layout carriers.
> 
> The JBS issue associated with this PR mentions also other issues, mostly related to the overly broad visibility of some of the methods in the javadoc (e.g. isPadding). Unfortunately, given the presence of an intermediate, non-public, abstract class, this is what we get from javadoc. Fixing these issues would require a deeper restructuring of the implementation, which would be too riskt at this stage.

This pull request has now been integrated.

Changeset: 889150b4
Author:    Maurizio Cimadamore <mcimadamore at openjdk.org>
URL:       https://git.openjdk.org/jdk19/commit/889150b47a7a33d302c1883320d2cfbb915c52e7
Stats:     48 lines in 5 files changed: 14 ins; 11 del; 23 mod

8289558: Need spec clarification of j.l.foreign.*Layout

Reviewed-by: psandoz, jvernee

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

PR: https://git.openjdk.org/jdk19/pull/98