RFR: 8364187: Make getClassAccessFlagsRaw non-native [v2]

Chen Liang liach at openjdk.org
Tue Jul 29 15:26:56 UTC 2025


On Tue, 29 Jul 2025 14:55:04 GMT, Roger Riggs <rriggs at openjdk.org> wrote:

>> Coleen Phillimore has updated the pull request incrementally with one additional commit since the last revision:
>> 
>>   Rename getRawClassAccessFlags to getClassFileAccessFlags and fix the test to reflect what the JVM does.
>
> src/java.base/share/classes/java/lang/Class.java line 248:
> 
>> 246:         protectionDomain = pd;
>> 247:         primitive = isPrim;
>> 248:         classFileAccessFlags = flags;
> 
> Its not that hard to add a field, why not have done this for identity?

Fields are initialized by injection in javaClasses instead of through this constructor, so it's easy for us to accidentally forget to inject a field.

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

PR Review Comment: https://git.openjdk.org/jdk/pull/26517#discussion_r2240207422


More information about the core-libs-dev mailing list