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