RFR (XS): JDK-8058534: Remove HeapRegion::_orig_end

Joseph Provino joseph.provino at oracle.com
Thu Sep 18 22:43:12 UTC 2014


Kim, thanks.  I made that change you suggested.

Webrev:  http://cr.openjdk.java.net/~jprovino/8058534/webrev.01/

joe



On 9/18/2014 6:24 PM, Kim Barrett wrote:
> On Sep 18, 2014, at 4:32 PM, Joseph Provino <joseph.provino at oracle.com> wrote:
>> Can I have reviews for the following small change that removes the
>> _orig_end member from the HeapRegion class. The information this member
>> stores can be easily recalculated on the fly, so it is just a waste of a
>> few bytes.
>>
>> CR:https://bugs.openjdk.java.net/browse/JDK-8058534
>>
>> Webrev:http://cr.openjdk.java.net/~jprovino/8058534/webrev.00/
>>
>> Testing: jprt
> In the original code, the _orig_end member that is being removed had a descriptive
> comment.  That description no longer exists in the changed code.  Maybe that
> comment should be moved to the orig_end() accessor?
>
> Otherwise looks good.
>




More information about the hotspot-gc-dev mailing list