RFR: 8292446: Make TableRateStatistics optional in CHT [v2]

Johan Sjölén duke at openjdk.org
Thu Aug 18 08:17:18 UTC 2022


On Wed, 17 Aug 2022 15:16:55 GMT, Coleen Phillimore <coleenp at openjdk.org> wrote:

>> Johan Sjölén has updated the pull request incrementally with two additional commits since the last revision:
>> 
>>  - Uninitialized pointers do not default to nullptr
>>  - Fixes
>
> src/hotspot/share/utilities/concurrentHashTable.hpp line 46:
> 
>> 44:  private:
>> 45:   // _stats_rate is null if statistics are not enabled.
>> 46:   TableRateStatistics* _stats_rate  = nullptr;
> 
> Don't you have to initialize this in the constructor?

My Google-fu fails me in what the difference between this assignment and doing it in the constructor is, so I'm moving it to the constructor to be sure.

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

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


More information about the hotspot-dev mailing list