RFR(s): 8076294: Cleanup of CollectedHeap::kind()
stefan.karlsson at oracle.com
Wed Apr 1 08:29:27 UTC 2015
On 2015-03-31 10:21, Per Liden wrote:
> Another small cleanup.
> CollectedHeap::kind() currently returns CollectedHeap::Abstract, which
> makes little sense. Changing it to be a pure virtual member function
> instead. The Abstract and SharedHeap members of CollectedHeap::Name
> can then be removed as they are never used.
> Webrev: http://cr.openjdk.java.net/~pliden/8076294/webrev.0/
The change looks good to me.
It's odd that the enum type is called 'Name' while the function is named
'kind', but I see that other places uses that duality as well. Maybe we
could clean that up in a separate change.
There's also another list of GC names containing the value 'Abstract'.
> Bug: https://bugs.openjdk.java.net/browse/JDK-8076294
> Testing: jprt
More information about the hotspot-gc-dev