RFR: 8266822: Rename MetaspaceShared::is_old_class to has_old_class_version [v5]

Coleen Phillimore coleenp at openjdk.java.net
Tue May 11 22:21:57 UTC 2021


On Tue, 11 May 2021 22:07:40 GMT, Calvin Cheung <ccheung at openjdk.org> wrote:

>> Please review this simple patch for renaming the function from `MetaspaceShared::is_old_class` to `MetaspaceShared::has_old_class_version`. Also added some comment to the function.
>> 
>> Tests:
>> - [x] tier1, 2
>
> Calvin Cheung has updated the pull request incrementally with one additional commit since the last revision:
> 
>   convert has_old_class_version to an instance method

Thank you for making this change!

src/hotspot/share/oops/instanceKlass.cpp line 2576:

> 2574: // retrieved during dump time.
> 2575: // Verification of archived old classes will be performed during run time.
> 2576: bool InstanceKlass::has_old_class_version() {

very small nit: Can you make this function 'const' ?

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

Marked as reviewed by coleenp (Reviewer).

PR: https://git.openjdk.java.net/jdk/pull/3983


More information about the hotspot-runtime-dev mailing list