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