On Wed, 14 Jun 2023 22:28:38 GMT, Jorn Vernee <[email protected]> wrote:
>> Add missing alignment checks for the alignment constraint of the root layout
>> of a VarHandle created through `MemoryLayout::varHandle` and a MethodHandle
>> `MemoryLayout::sliceHandle`.
>>
>> Testing: `jdk_foreign` test suite
>
> Jorn Vernee has updated the pull request incrementally with one additional
> commit since the last revision:
>
> formatting
src/java.base/share/classes/java/lang/foreign/MemoryLayout.java line 383:
> 381: * {@code 0 <= x_i < b_i}, where {@code 1 <= i <= n}, or {@link
> IndexOutOfBoundsException} is thrown.
> 382: * <p>
> 383: * The accessed address must be <a
> href="MemorySegment.html#segment-alignment">aligned</a> according to the
> {@linkplain
Very nice and clear!
-------------
PR Review Comment: https://git.openjdk.org/jdk/pull/14475#discussion_r1230261542
