On Fri, 3 Nov 2023 16:17:32 GMT, Jorn Vernee <jver...@openjdk.org> wrote:

>> src/java.base/share/classes/java/lang/foreign/Linker.java line 247:
>> 
>>> 245:  * </ul>
>>> 246:  *
>>> 247:  * Linker implementations may optionally support additional layouts, 
>>> such as 'packed' struct layouts.
>> 
>> Optional: should we say what a packed struct layout is? E.g. make `packed` 
>> italic. Then follow up with a sentence like: 
>> 
>> a packed struct is a struct in which there is at least one member layouts L 
>> whose offsets does not respect L's natural alignment: <example>
>
> Yes, good idea

See 
https://github.com/openjdk/jdk/pull/16485/commits/fea8a26325b3e6142c652821e0ef95d78992b36c

I've avoided talking about offset, since packing can also be used to avoid 
trailing padding.

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

PR Review Comment: https://git.openjdk.org/jdk/pull/16485#discussion_r1381955593

Reply via email to