RFR(S): 7095243: Disambiguate ReferenceProcessor::_discoveredSoftRefs

Y. S. Ramakrishna y.s.ramakrishna at oracle.com
Tue Sep 27 11:45:26 PDT 2011


Looks good to me.

-- ramki

On 09/27/11 10:08, John Cuthbertson wrote:
> Hi Everyone,
> 
> Can I have a couple of volunteers review the changes for this CR? The 
> webrev can be found at: http://cr.openjdk.java.net/~johnc/7095243/webrev.0/
> 
> Summary:
> As part of the review comments for 6484982 (adding reference processing 
> to G1 evacuation pauses) a couple of the reviewers thought that using 
> _discoveredSoftRefs when referring to the entire array of discovered 
> lists was unclear and it was requested that role of _discoveredSoftRefs 
> be disambiguated. These changes address that review comment. I have 
> renamed the array of discovered reference lists (_discoveredSoftRefs et 
> al are just pointers into the array) and use the more generic name in 
> the contexts where we are referring to the entire array.
> 
> Testing: Dacapo2006 and GC locker test from the GC test suite with _all_ 
> collectors and jprt.
> 
> Thanks,
> 
> JohnC


More information about the hotspot-gc-dev mailing list