RFR: 8262227: Change SystemDictionary::find() to return an InstanceKlass*.

Harold Seigel hseigel at openjdk.java.net
Thu Feb 25 18:55:41 UTC 2021


On Thu, 25 Feb 2021 13:11:11 GMT, Coleen Phillimore <coleenp at openjdk.org> wrote:

>> I think the get_java_* set of functions could be streamlined so that you pass in the symbol for the field you need. Also perhaps VersionProps could be a well-known class so we don't have to look it up. But this seems a future RFE. Even the current changes seem a little out-of-scope for this change.
>
> yes, this might be a useful change in a follow up RFE. What you have is good.  You were going to change the lines anyway.

Hi Ioi, I'd like to do this cleanup in a separate RFE.  Is that okay?

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

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


More information about the hotspot-dev mailing list