RFR: JDK-8149542: Missing failure reporting in HeapRegion::verify

Joseph Provino joseph.provino at oracle.com
Wed Feb 10 14:38:30 UTC 2016


Looks good!

joe

On 2/10/2016 7:50 AM, Bengt Rutisson wrote:
>
> Hi everyone,
>
> Could I have a couple of reviews for this small change?
>
> http://cr.openjdk.java.net/~brutisso/8149542/webrev.00/
> https://bugs.openjdk.java.net/browse/JDK-8149542
>
> Line 830 in heapRegion.cpp should be followed by lines like this:
>
>   *failures = true;
>   return;
>
> http://hg.openjdk.java.net/jdk9/hs-rt/hotspot/file/2a8e87190908/src/share/vm/gc/g1/heapRegion.cpp#l830 
>
>
> Thanks,
> Bengt




More information about the hotspot-gc-dev mailing list