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

Jorn Vernee jvernee at openjdk.org
Mon Jul 4 11:45:06 UTC 2022


On Fri, 1 Jul 2022 11:18:35 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.
>
> Maurizio Cimadamore has updated the pull request incrementally with one additional commit since the last revision:
> 
>   Use pattern match in equals implementation
>   Consolidate hashcode

Marked as reviewed by jvernee (Reviewer).

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

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


More information about the core-libs-dev mailing list