RFR: 8324649: Shenandoah: refactor implementation of free set [v39]

[Kelvin Nilsen]

On Sat, 13 Apr 2024 15:17:21 GMT, Kelvin Nilsen <kdnilsen at openjdk.org> wrote:

>> src/hotspot/share/gc/shenandoah/shenandoahFreeSet.hpp line 165:
>> 
>>> 163: 
>>> 164: // Each ShenandoahHeapRegion is associated with a ShenandoahFreeSetPartitionId.
>>> 165: enum ShenandoahFreeSetPartitionId : uint8_t {
>> 
>> Make this an 'enum class' instead.
>> I see that you are using the enum as index, in this case you can declare the enum class like 'enum class ShenandoahFreeSetPartitionId : uint8_t'
>
> Thanks for this suggestion.  Done.

Thanks.  This is done.

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

PR Review Comment: https://git.openjdk.org/jdk/pull/17561#discussion_r1566566426