RFR: 8232613: Move Object.registerNatives into HotSpot

[Andrew Dinn]

Hi Claes/David,

On 22/10/2019 00:08, David Holmes wrote:
> My only nit with this is that I don't think register_native and friends
> belongs in the SystemDictionary class as it has nothing to do with the
> SD. This code seems to be all about Methods so that seems like the place
> to put this.
Claes, the patch looks good.

David, I'm not clear why you think this is about methods. The calls to
register_native occur under

  SystemDictionary::resolve_well_known_classes

which indicates to me that this code is very much about Classes/klasses
-- specifically initializing class Object during early bootstrap. The
fact that it is /methods/ of Object that are being fixed up doesn't
really change that for me.

So, I would have thought it would be clearer for the code that does this
specific bit of 'well known class init' to be located with the other
'well known class init' code.

regards,


Andrew Dinn
-----------