RFR: 8267089: Use typedef KVHashtable for ID2KlassTable

Xin Liu xliu at openjdk.java.net
Mon May 17 05:47:17 UTC 2021


On Thu, 13 May 2021 07:08:11 GMT, Yi Yang <yyang at openjdk.org> wrote:

> While backporting some stuff to jdk11, I noticed main functionalities of ID2KlassTable have been removed afterJDK-8213587, it seems that we can use a simple typedef KVHashtable instead.

I think you patch looks good. in particular, you tuck away ID2KlassTable in . That increases encapsulation. 

I mean you can change  the return type of `table()` to 'ID2KlassTable*[whitespace]'. 
In hotspot cod style, pointer '*' is close to its type.

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

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


More information about the hotspot-runtime-dev mailing list