RFR: 8344734: Rename memoryFileTracker to nativeMemoryFileTracker [v5]
Johan Sjölen
jsjolen at openjdk.org
Fri Apr 25 20:15:51 UTC 2025
On Fri, 25 Apr 2025 14:53:36 GMT, Gerard Ziemski <gziemski at openjdk.org> wrote:
> > Hi,
> > I think that the name `MemoryFileTracker` is fine. We haven't renamed `VirtualMemoryTracker` to have `Native` as a prefix, so why is it necessary for this case?
>
> It is not necessary, it's a nice to have. Technical debt kind of thing. Eventually we want consistent name everywhere. We are doing it one file a time to keep changes as small as possible.
I'm not convinced that it is a nice to have. I'd rather have all of them changed at once so that we can have a discussion on what kind of naming standards we should have for NMT components.
-------------
PR Comment: https://git.openjdk.org/jdk/pull/24579#issuecomment-2831346506
More information about the hotspot-dev
mailing list