RFR(S): 7095243: Disambiguate ReferenceProcessor::_discoveredSoftRefs
Y. S. Ramakrishna
y.s.ramakrishna at oracle.com
Tue Sep 27 18:45:26 UTC 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