On Fri, 9 Jun 2023 13:40:47 GMT, Maurizio Cimadamore <mcimadam...@openjdk.org> wrote:
>> FWIW, it looks like this needs `@Test(expectedExceptions = >> IllegalArgumentException.class)` since range elements are not allowed for >> `byteOffsetHandle` > > Note sure - javadoc says: > > * @throws IllegalArgumentException if the layout path is not <a > href="#well-formedness">well-formed</a> for this layout. > * @throws IllegalArgumentException if the layout path contains one or > more <a href=#deref-path-elements>dereference path elements</a>. > > > Which seems correct. The handle version can accept indices, so that's ok? Note that the javadoc has been rectified here (as part of the PR) as I think it has always been wrong in the past (probably cut and paste issue from the non-MH version?) ------------- PR Review Comment: https://git.openjdk.org/jdk/pull/14098#discussion_r1224329494