On Fri, 18 Sep 2026 12:07:28 GMT, Johan Sjölen <[email protected]> wrote:
>> src/hotspot/share/oops/layoutKind.hpp line 138:
>>
>>> 136: // The different layouts available for a particular Klass
>>> 137: struct LayoutDescriptions {
>>> 138: constexpr static int NoValue = -1; // Unsupported layouts are
>>> assigned this value
>>
>> Suggestion:
>>
>> constexpr static int NoValue = -1; // Unsupported layouts are assigned
>> this value
>
> These look identical?
It's not. It separates that static constant from the member variables. Making
the code clearer to read.
-------------
PR Review Comment: https://git.openjdk.org/jdk/pull/32276#discussion_r4046673558
