RFR: 8337563: NMT: rename MEMFLAGS to MemFlag
Gerard Ziemski
gziemski at openjdk.org
Fri Aug 9 21:33:43 UTC 2024
On Wed, 7 Aug 2024 17:13:06 GMT, Gerard Ziemski <gziemski at openjdk.org> wrote:
> Please review this cleanup, where we rename `MEMFLAGS` to `MemType`.
>
> `MEMFLAGS` implies that we can use more than one at the same time, but those are exclusive values, so `MemType` is much more suitable name.
>
> There is a bunch of other related cleanup that we can do, but I will leave for follow up issues such as [NMT: rename NMTUtil::flag to NMTUtil::type](https://bugs.openjdk.org/browse/JDK-8337836)
The current proposed fix renames from MEMFLAGS to MemType, other proposed names are:
- MemTypeFlag
- NMTCategory
- NMTCat
I also like:
- NMTMemType
Any other suggestions?
I don't think I will be following up with argument renaming after this, though, whatever the final name we agree on...
-------------
PR Comment: https://git.openjdk.org/jdk/pull/20497#issuecomment-2278781358
More information about the serviceability-dev
mailing list