RFR(S): 8137756: Remove hrs_err_msg and hrs_ext_msg from heapRegionSet

Per Liden per.liden at oracle.com
Mon Nov 2 11:43:19 UTC 2015


Hi David,

On 2015-10-30 14:54, David Lindholm wrote:
> Hi,
>
> Please review this follow-up patch to the varargs assert()-patch. This
> patch deletes the hrs_err_msg and hrs_ext_msg from heapRegionSet.
>
> Bug: https://bugs.openjdk.java.net/browse/JDK-8137756
> Webrev: http://cr.openjdk.java.net/~david/JDK-8137756/webrev.00/

heapRegionSet.cpp
-----------------
   32   assert_heap_region_set(hr->containing_set() == NULL, "should not 
already have a containing set %u");

The %u in there looks wrong.

Other than that the change looks good. I don't need to see another 
webrev just for this.

cheers,
/Per



More information about the hotspot-gc-dev mailing list