RFR: 8331865: Consolidate size and alignment checks in LayoutPath [v2]

Maurizio Cimadamore mcimadamore at openjdk.org
Fri May 17 16:02:06 UTC 2024


On Fri, 17 May 2024 15:54:04 GMT, Paul Sandoz <psandoz at openjdk.org> wrote:

>> Maurizio Cimadamore has updated the pull request incrementally with one additional commit since the last revision:
>> 
>>   Fix copyrights
>
> src/java.base/share/classes/java/lang/foreign/MemoryLayout.java line 630:
> 
>> 628:      *     <li>The access operation must fall inside the spatial bounds of the accessed
>> 629:      *     memory segment, or an {@link IndexOutOfBoundsException} is thrown. This is the case
>> 630:      *     when {@code B + A <= S}, where {@code O} is the base offset (defined above),
> 
> Do you mean `{@code O + A <= S}`?
> (Still working my way through the changes...)

Yes, apologies

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

PR Review Comment: https://git.openjdk.org/jdk/pull/19251#discussion_r1605237785


More information about the core-libs-dev mailing list