On Wed, 7 Jan 2026 01:01:52 GMT, Anthony Scarpino <[email protected]> wrote:

>> Weijun Wang has updated the pull request incrementally with one additional 
>> commit since the last revision:
>> 
>>   must have username; more comments
>
> src/jdk.security.auth/share/classes/com/sun/security/auth/module/UnixSystem.java
>  line 88:
> 
>> 86:             = (ValueLayout) LINKER.canonicalLayouts().get("size_t");
>> 87: 
>> 88:     private static final StructLayout capturedStateLayout = 
>> Linker.Option.captureStateLayout();
> 
> Should the variables below be capitalized because they are `static final`?

Per asked the same above. I'm not exactly sure if all of them should be 
capitalized. I'll willing to choose either.

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

PR Review Comment: https://git.openjdk.org/jdk/pull/28931#discussion_r2670175161

Reply via email to