RFR: 8189171: Move GC argument processing into GC specific classes
Per Liden
per.liden at oracle.com
Thu Oct 26 09:43:53 UTC 2017
Hi,
On 2017-10-25 18:11, Erik Osterlund wrote:
[...]
>> So let me just put your changes up for review (again), if you don't mind:
>>
>> Full webrev:
>> http://cr.openjdk.java.net/~eosterlund/8189171/webrev.03/
>> <http://cr.openjdk.java.net/%7Eeosterlund/8189171/webrev.03/>
I like this. Just two naming suggestions:
src/hotspot/share/gc/shared/gcArguments.hpp:
39 static jint create_instance();
40 static bool is_initialized();
41 static GCArguments* instance();
Could we call these:
39 static jint initialize();
40 static bool is_initialized();
41 static GCArguments* arguments();
Reasoning: initialize() maps better to its companion is_initialized().
GCArguments::arguments() maps better to the existing patterns we have
with CollectedHeap::heap().
cheers,
Per
More information about the hotspot-gc-dev
mailing list