RFR: 8344734: Rename memoryFileTracker to nativeMemoryFileTracker [v3]

[Anton Artemov]

On Wed, 23 Apr 2025 02:30:21 GMT, David Holmes <dholmes at openjdk.org> wrote:

>> Anton Artemov has updated the pull request with a new target base due to a merge or a rebase. The pull request now contains one commit:
>> 
>>   8344734: renamed memoryFileTracker to nativeMemoryFileTracker
>
> src/hotspot/share/nmt/nativeMemoryFileTracker.hpp line 43:
> 
>> 41: // All memory is accounted as committed, there is no reserved memory.
>> 42: // Any reserved memory is expected to exist in the VirtualMemoryTracker.
>> 43: class NativeMemoryFileTracker {
> 
> You need to update the comment at line 38 as well. The problem with adding adjectives into names like this is that it becomes unclear how to parse it: is this a native-MemoryFileTracker, or a NativeMemory-FileTracker? The class comment should make this clear. Thanks.

Thanks, noted! Addressed in the latest commit. It is rather a NativeMemoryFile-Tracker.

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

PR Review Comment: https://git.openjdk.org/jdk/pull/24579#discussion_r2057853414