<div dir="auto">Note that “NMT::MemType” may lead to many uses of just “MemType”  since a lot of the usage will happen inside a future NMT namespace and people will just drop the namespace. Will make it a bit more difficult to grep for, since you need to look for both variants.</div><div dir="auto"><br></div><div dir="auto">Adding an NMT namespace also opens other questions. E.g. writing - all lower case is the standard.</div><div dir="auto"><br></div><div dir="auto"><br></div><div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Thu 29. Aug 2024 at 17:15, Gerard Ziemski <<a href="mailto:gziemski@openjdk.org">gziemski@openjdk.org</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left-width:1px;border-left-style:solid;padding-left:1ex;border-left-color:rgb(204,204,204)">On Wed, 7 Aug 2024 17:13:06 GMT, Gerard Ziemski <<a href="mailto:gziemski@openjdk.org" target="_blank">gziemski@openjdk.org</a>> wrote:<br>
<br>
> Please review this cleanup, where we rename `MEMFLAGS` to `MemType`.<br>
> <br>
> `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.<br>
> <br>
> 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](<a href="https://bugs.openjdk.org/browse/JDK-8337836" rel="noreferrer" target="_blank">https://bugs.openjdk.org/browse/JDK-8337836</a>)<br>
<br>
Taking Dan's feedback into account we have:<br>
<br>
-------------<br>
<br>
PR Comment: <a href="https://git.openjdk.org/jdk/pull/20497#issuecomment-2318039929" rel="noreferrer" target="_blank">https://git.openjdk.org/jdk/pull/20497#issuecomment-2318039929</a><br>
</blockquote></div></div>