On Mon, 15 Jan 2024 07:56:13 GMT, Per Minborg <pminb...@openjdk.org> wrote:
> This PR proposes to add an improved Improve > `LayoutPath.PathElement::toString` method simplifying debugging. > > Opinions and suggestions for `static PathElement sequenceElement(long start, > long step)` are welcome. test/jdk/java/foreign/TestLayoutPaths.java line 341: > 339: public void testSequenceElementRangeToString() { > 340: PathElement e = PathElement.sequenceElement(2, 4); > 341: assertEquals(e.toString(), "sequence range 2 + N * 4, N >= 0"); Not sure this is the best way to express the toString for the range version. ------------- PR Review Comment: https://git.openjdk.org/jdk/pull/17417#discussion_r1452026599