RFR: 8357053: ZGC: Improved utility for ZPageAge [v2]
Joel Sikström
jsikstro at openjdk.org
Fri May 23 12:24:56 UTC 2025
On Fri, 23 May 2025 11:09:20 GMT, Stefan Karlsson <stefank at openjdk.org> wrote:
>> src/hotspot/share/utilities/enumIterator.hpp line 269:
>>
>>> 267: }
>>> 268:
>>> 269: template <EnumType Start, EnumType End>
>>
>> I think this should be using T to match the surrounding code.
>> Suggestion:
>>
>> template <T Start, T End>
>
> BTW, if you are going to change this could you also fix the pre-existing issues with the includes so that it matches the style guide?
I agree, `T` is better. I'll fix includes.
-------------
PR Review Comment: https://git.openjdk.org/jdk/pull/25251#discussion_r2104477364
More information about the hotspot-gc-dev
mailing list