<div dir="ltr"><div dir="ltr"><div dir="ltr"><div dir="ltr"><div dir="ltr">I would agree - it doesn't feel right to add such as specific method for debug/textual representation of a MemorySegment when it could instead be a more flexible utility in a separate support library class.</div><div dir="ltr">Currently developers can implement a reasonably useful hex format using toArray() and HexFormat, perhaps need something like:<br></div><div dir="ltr"><br></div><div>     new DebugFormat(segment).wantHex().wantChars().wantOctal().stream().forEach( ... ).</div><div><div>     new DebugFormat(byte[] somebytearray, int from int to).wantHex().wantChars().stream().forEach( ... ).</div><div><br></div></div><div>Kind regards</div><div><br></div><div>Duncan</div></div></div></div></div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Thu, 4 Aug 2022 at 11:01, Maurizio Cimadamore <<a href="mailto:mcimadamore@openjdk.org">mcimadamore@openjdk.org</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">On Thu, 4 Aug 2022 09:46:29 GMT, Maurizio Cimadamore <<a href="mailto:mcimadamore@openjdk.org" target="_blank">mcimadamore@openjdk.org</a>> wrote:<br>
<br>
>> Per-Åke Minborg has updated the pull request incrementally with one additional commit since the last revision:<br>
>> <br>
>>   Fix failing test and rename variables<br>
><br>
> src/java.base/share/classes/java/lang/foreign/MemorySegment.java line 1892:<br>
> <br>
>> 1890:      *         </ul><br>
>> 1891:      *     </li><br>
>> 1892:      *     <li>Grouped layouts are enclosed in curly brackets.</li><br>
> <br>
> Shouldn't it be "values that are part of a group layout" ? Also maybe add a linkplain to `GroupLayout`. Same for sequence layout. I also note that you use the "are rendered" in the above text. Maybe you can lean on it - e.g.<br>
> <br>
> "values in a group layout are rendered with enclosing curly braces".<br>
<br>
Possible questions: why are address values represented as toString? Or numbers in decimal form? What if I want hex form? This is the part that makes me uncomfortable about this method: in a way, you almost want it to generate some kind of XML or json schema - and then leave the toString mapping to the user - but this method is doing everything, so it has to make opinionated choices at every step of the way, and I can see how some of those choices would be good for some and bad for others.<br>
<br>
-------------<br>
<br>
PR: <a href="https://git.openjdk.org/panama-foreign/pull/695" rel="noreferrer" target="_blank">https://git.openjdk.org/panama-foreign/pull/695</a><br>
</blockquote></div>