On Tue, 29 Jul 2025 14:55:04 GMT, Roger Riggs <rri...@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

Reply via email to