On Thu, 1 Feb 2024 14:25:24 GMT, Per Minborg <pminb...@openjdk.org> wrote:

>> src/java.base/share/classes/jdk/internal/foreign/LayoutPath.java line 454:
>> 
>>> 452:     }
>>> 453: 
>>> 454:     public static final class SequenceElement
>> 
>> Why are these not empty records?
>
> I had that in the beginning but converted them to regular classes as they do 
> not have any record components. But maybe it is better to make them all 
> records for consistency?

I think so, even less code :-)

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

PR Review Comment: https://git.openjdk.org/jdk/pull/17651#discussion_r1474622217

Reply via email to