--- old/src/share/vm/gc_implementation/g1/heapRegionSets.cpp	2014-03-13 22:21:57.000000000 +0100
+++ new/src/share/vm/gc_implementation/g1/heapRegionSets.cpp	2014-03-13 22:21:57.000000000 +0100
@@ -64,7 +64,7 @@
       name(), total_capacity_bytes(), capacity));
 }
 
-bool MasterFreeRegionListMtSafeChecker::check() {
+void MasterFreeRegionListMtSafeChecker::check() {
   // Master Free List MT safety protocol:
   // (a) If we're at a safepoint, operations on the master free list
   // should be invoked by either the VM thread (which will serialize
@@ -79,21 +79,17 @@
   } else {
     guarantee(Heap_lock->owned_by_self(), "master free list MT safety protocol outside a safepoint");
   }
-
-  return true;
 }
 
-bool SecondaryFreeRegionListMtSafeChecker::check() {
+void SecondaryFreeRegionListMtSafeChecker::check() {
   // Secondary Free List MT safety protocol:
   // Operations on the secondary free list should always be invoked
   // while holding the SecondaryFreeList_lock.
 
   guarantee(SecondaryFreeList_lock->owned_by_self(), "secondary free list MT safety protocol");
-
-  return true;
 }
 
-bool OldRegionSetMtSafeChecker::check() {
+void OldRegionSetMtSafeChecker::check() {
   // Master Old Set MT safety protocol:
   // (a) If we're at a safepoint, operations on the master old set
   // should be invoked:
@@ -114,11 +110,9 @@
   } else {
     guarantee(Heap_lock->owned_by_self(), "master old set MT safety protocol outside a safepoint");
   }
-
-  return true;
 }
 
-bool HumongousRegionSetMtSafeChecker::check() {
+void HumongousRegionSetMtSafeChecker::check() {
   // Humongous Set MT safety protocol:
   // (a) If we're at a safepoint, operations on the master humongous
   // set should be invoked by either the VM thread (which will
@@ -135,6 +129,4 @@
     guarantee(Heap_lock->owned_by_self(),
               "master humongous set MT safety protocol outside a safepoint");
   }
-
-  return true;
 }