RFR: JDK-8297958: NMT: Display peak values [v2]

Thomas Stuefe stuefe at openjdk.org
Tue Dec 6 15:07:56 UTC 2022


On Tue, 6 Dec 2022 12:11:18 GMT, Johan Sjölen <jsjolen at openjdk.org> wrote:

>> Thomas Stuefe has updated the pull request incrementally with one additional commit since the last revision:
>> 
>>   small fix
>
> src/hotspot/share/services/mallocTracker.hpp line 140:
> 
>> 138: 
>> 139:   const MemoryCounter* malloc_counter() const { return &_malloc; }
>> 140:   const MemoryCounter* arena_counter()  const { return &_arena;  }
> 
> It seems to me like we can keep these as returning references, if the printers take them in as references instead. Am I missing something?
> 
> I'm fine with keeping them as pointers, but I do want to make sure that I'm not missing some reasoning here.

(My first answer got lost somehow)

You understand correctly. I have a slight preference for pointers since one can tell them easier apart in coding, but ultimately just a coin toss here.

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

PR: https://git.openjdk.org/jdk/pull/11497


More information about the hotspot-runtime-dev mailing list