RFR: 8320370: NMT: Change MallocMemorySnapshot to simplify code. [v2]

[Johan Sjölen]

On Mon, 20 Nov 2023 12:38:32 GMT, Thomas Stuefe <stuefe at openjdk.org> wrote:

>> Johan Sjölen has updated the pull request incrementally with one additional commit since the last revision:
>> 
>>   Thomas comments
>
> src/hotspot/share/nmt/mallocTracker.cpp line 46:
> 
>> 44: #include "utilities/vmError.hpp"
>> 45: 
>> 46: MallocMemorySnapshot MallocMemorySummary::_snapshot{};
> 
> Why the curly braces?

They're not necessary, I just have a tendency to explicitly use brace/"uniform" initialization everywhere. I've removed them.

> src/hotspot/share/nmt/virtualMemoryTracker.cpp line 52:
> 
>> 50: 
>> 51: void VirtualMemorySummary::initialize() {
>> 52: }
> 
> Can be removed.

Oh, thanks! Should've seen that :-).

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

PR Review Comment: https://git.openjdk.org/jdk/pull/16724#discussion_r1399217021
PR Review Comment: https://git.openjdk.org/jdk/pull/16724#discussion_r1399216448