RFR: 8324649: Shenandoah: replace implementation of free set [v57]
Kelvin Nilsen
kdnilsen at openjdk.org
Mon May 13 03:04:21 UTC 2024
On Mon, 13 May 2024 03:01:55 GMT, Kelvin Nilsen <kdnilsen at openjdk.org> wrote:
>> src/hotspot/share/gc/shenandoah/shenandoahFreeSet.cpp line 100:
>>
>>> 98: }
>>> 99:
>>> 100: ShenandoahRegionPartitions::~ShenandoahRegionPartitions() {
>>
>> I think there's no need to put this trivial destructor definition in the .cpp file. Just place it with the declaration in the .hpp file.
>
> Thanks. Fixed.
Thanks. Fixed.
-------------
PR Review Comment: https://git.openjdk.org/jdk/pull/17561#discussion_r1597816290
More information about the hotspot-gc-dev
mailing list