RFR (M): 7163191 G1: introduce a "heap spanning table" abstraction
bengt.rutisson at oracle.com
Wed Sep 25 09:20:00 UTC 2013
> 25 sep 2013 kl. 11:02 skrev Thomas Schatzl <thomas.schatzl at oracle.com>:
>> On Tue, 2013-09-24 at 11:45 +0200, Bengt Rutisson wrote:
>> Hi Thomas,
>> Sorry for taking so long to get back to you about this.
>> Looks good.
>> Thanks for adding the test! One minor nit about the test:
>> Lately there has been some unit tests added that use a wrapper function
>> like test_g1biasedarray() that you introduce. These tests, for example
>> TestReservedSpace_test(), use a forward declaration in jni.cpp rather
>> than including the hpp file and they also use a naming convention that
>> kind of mimics a static method on the class to be tested. This is pretty
>> new, so I don't know if this is the accepted way to do it, but unless
>> you have strong feelings I suggest following that same pattern.
> Fixed. See http://cr.openjdk.java.net/~tschatzl/7163191/webrev.2/
More information about the hotspot-gc-dev