RFR: 8300682: InstanceKlassMiscStatus is a bad name [v2]

Coleen Phillimore coleenp at openjdk.org
Thu Jan 19 22:22:46 UTC 2023


On Thu, 19 Jan 2023 22:07:07 GMT, Coleen Phillimore <coleenp at openjdk.org> wrote:

>> src/hotspot/share/oops/instanceKlassFlags.hpp line 30:
>> 
>>> 28: class ClassLoaderData;
>>> 29: 
>>> 30: class InstanceKlassFlags {
>> 
>> The macro to prevent multiple inclusions should have the same name as the file after the renaming.
>> Should we add a comment stating that "Flags" are immutable, and mutable status should be put elsewhere?
>
> Oh whoops. Thank you for noticing that.

I was going to put this comment when adding the mutable status class, in a following change.

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

PR: https://git.openjdk.org/jdk/pull/12106


More information about the hotspot-dev mailing list