--- old/src/share/vm/gc_implementation/concurrentMarkSweep/concurrentMarkSweepGeneration.hpp 2013-05-10 15:55:57.664885077 +0200 +++ new/src/share/vm/gc_implementation/concurrentMarkSweep/concurrentMarkSweepGeneration.hpp 2013-05-10 15:55:57.596885079 +0200 @@ -485,10 +485,6 @@ assert(!span.is_empty(), "Empty span could spell trouble"); } - void do_object(oop obj) { - assert(false, "not to be invoked"); - } - bool do_object_b(oop obj); }; @@ -1536,9 +1532,6 @@ _bit_map(bit_map), _par_scan_closure(cl) { } - void do_object(oop obj) { - guarantee(false, "Call do_object_b(oop, MemRegion) instead"); - } bool do_object_b(oop obj) { guarantee(false, "Call do_object_b(oop, MemRegion) form instead"); return false; --- old/src/share/vm/gc_implementation/g1/concurrentMark.hpp 2013-05-10 15:55:58.160885064 +0200 +++ new/src/share/vm/gc_implementation/g1/concurrentMark.hpp 2013-05-10 15:55:58.076885065 +0200 @@ -44,9 +44,6 @@ public: G1CMIsAliveClosure(G1CollectedHeap* g1) : _g1(g1) { } - void do_object(oop obj) { - ShouldNotCallThis(); - } bool do_object_b(oop obj); }; --- old/src/share/vm/gc_implementation/g1/g1CollectedHeap.cpp 2013-05-10 15:55:58.584885050 +0200 +++ new/src/share/vm/gc_implementation/g1/g1CollectedHeap.cpp 2013-05-10 15:55:58.492885054 +0200 @@ -5086,7 +5086,6 @@ G1CollectedHeap* _g1; public: G1AlwaysAliveClosure(G1CollectedHeap* g1) : _g1(g1) {} - void do_object(oop p) { assert(false, "Do not call."); } bool do_object_b(oop p) { if (p != NULL) { return true; --- old/src/share/vm/gc_implementation/g1/g1CollectedHeap.hpp 2013-05-10 15:55:59.080885037 +0200 +++ new/src/share/vm/gc_implementation/g1/g1CollectedHeap.hpp 2013-05-10 15:55:59.012885039 +0200 @@ -165,7 +165,6 @@ G1CollectedHeap* _g1; public: G1STWIsAliveClosure(G1CollectedHeap* g1) : _g1(g1) {} - void do_object(oop p) { assert(false, "Do not call."); } bool do_object_b(oop p); }; --- old/src/share/vm/gc_implementation/parallelScavenge/psMarkSweep.cpp 2013-05-10 15:55:59.500885023 +0200 +++ new/src/share/vm/gc_implementation/parallelScavenge/psMarkSweep.cpp 2013-05-10 15:55:59.448885026 +0200 @@ -562,7 +562,6 @@ // This should be moved to the shared markSweep code! class PSAlwaysTrueClosure: public BoolObjectClosure { public: - void do_object(oop p) { ShouldNotReachHere(); } bool do_object_b(oop p) { return true; } }; static PSAlwaysTrueClosure always_true; --- old/src/share/vm/gc_implementation/parallelScavenge/psParallelCompact.cpp 2013-05-10 15:55:59.912885011 +0200 +++ new/src/share/vm/gc_implementation/parallelScavenge/psParallelCompact.cpp 2013-05-10 15:55:59.844885015 +0200 @@ -781,7 +781,6 @@ PSParallelCompact::IsAliveClosure PSParallelCompact::_is_alive_closure; -void PSParallelCompact::IsAliveClosure::do_object(oop p) { ShouldNotReachHere(); } bool PSParallelCompact::IsAliveClosure::do_object_b(oop p) { return mark_bitmap()->is_marked(p); } void PSParallelCompact::KeepAliveClosure::do_oop(oop* p) { PSParallelCompact::KeepAliveClosure::do_oop_work(p); } @@ -2406,7 +2405,6 @@ // This should be moved to the shared markSweep code! class PSAlwaysTrueClosure: public BoolObjectClosure { public: - void do_object(oop p) { ShouldNotReachHere(); } bool do_object_b(oop p) { return true; } }; static PSAlwaysTrueClosure always_true; --- old/src/share/vm/gc_implementation/parallelScavenge/psParallelCompact.hpp 2013-05-10 15:56:00.304885001 +0200 +++ new/src/share/vm/gc_implementation/parallelScavenge/psParallelCompact.hpp 2013-05-10 15:56:00.244885003 +0200 @@ -784,7 +784,6 @@ // class IsAliveClosure: public BoolObjectClosure { public: - virtual void do_object(oop p); virtual bool do_object_b(oop p); }; --- old/src/share/vm/gc_implementation/parallelScavenge/psScavenge.cpp 2013-05-10 15:56:00.700884988 +0200 +++ new/src/share/vm/gc_implementation/parallelScavenge/psScavenge.cpp 2013-05-10 15:56:00.632884990 +0200 @@ -70,9 +70,6 @@ // Define before use class PSIsAliveClosure: public BoolObjectClosure { public: - void do_object(oop p) { - assert(false, "Do not call."); - } bool do_object_b(oop p) { return (!PSScavenge::is_obj_in_young((HeapWord*) p)) || p->is_forwarded(); } --- old/src/share/vm/gc_implementation/shared/markSweep.cpp 2013-05-10 15:56:01.092884978 +0200 +++ new/src/share/vm/gc_implementation/shared/markSweep.cpp 2013-05-10 15:56:01.024884980 +0200 @@ -166,7 +166,6 @@ MarkSweep::IsAliveClosure MarkSweep::is_alive; -void MarkSweep::IsAliveClosure::do_object(oop p) { ShouldNotReachHere(); } bool MarkSweep::IsAliveClosure::do_object_b(oop p) { return p->is_gc_marked(); } MarkSweep::KeepAliveClosure MarkSweep::keep_alive; --- old/src/share/vm/gc_implementation/shared/markSweep.hpp 2013-05-10 15:56:01.456884966 +0200 +++ new/src/share/vm/gc_implementation/shared/markSweep.hpp 2013-05-10 15:56:01.396884968 +0200 @@ -88,7 +88,6 @@ // Used for java/lang/ref handling class IsAliveClosure: public BoolObjectClosure { public: - virtual void do_object(oop p); virtual bool do_object_b(oop p); }; --- old/src/share/vm/memory/defNewGeneration.cpp 2013-05-10 15:56:01.828884955 +0200 +++ new/src/share/vm/memory/defNewGeneration.cpp 2013-05-10 15:56:01.760884957 +0200 @@ -50,9 +50,6 @@ DefNewGeneration::IsAliveClosure::IsAliveClosure(Generation* g) : _g(g) { assert(g->level() == 0, "Optimized for youngest gen."); } -void DefNewGeneration::IsAliveClosure::do_object(oop p) { - assert(false, "Do not call."); -} bool DefNewGeneration::IsAliveClosure::do_object_b(oop p) { return (HeapWord*)p >= _g->reserved().end() || p->is_forwarded(); } --- old/src/share/vm/memory/defNewGeneration.hpp 2013-05-10 15:56:02.212884944 +0200 +++ new/src/share/vm/memory/defNewGeneration.hpp 2013-05-10 15:56:02.144884946 +0200 @@ -150,7 +150,6 @@ Generation* _g; public: IsAliveClosure(Generation* g); - void do_object(oop p); bool do_object_b(oop p); }; --- old/src/share/vm/memory/iterator.hpp 2013-05-10 15:56:02.580884933 +0200 +++ new/src/share/vm/memory/iterator.hpp 2013-05-10 15:56:02.516884935 +0200 @@ -158,7 +158,7 @@ }; -class BoolObjectClosure : public ObjectClosure { +class BoolObjectClosure { public: virtual bool do_object_b(oop obj) = 0; }; --- old/src/share/vm/memory/referenceProcessor.cpp 2013-05-10 15:56:02.968884922 +0200 +++ new/src/share/vm/memory/referenceProcessor.cpp 2013-05-10 15:56:02.904884925 +0200 @@ -252,7 +252,6 @@ class AlwaysAliveClosure: public BoolObjectClosure { public: virtual bool do_object_b(oop obj) { return true; } - virtual void do_object(oop obj) { assert(false, "Don't call"); } }; class CountHandleClosure: public OopClosure { --- old/src/share/vm/memory/sharedHeap.cpp 2013-05-10 15:56:03.364884910 +0200 +++ new/src/share/vm/memory/sharedHeap.cpp 2013-05-10 15:56:03.296884912 +0200 @@ -212,7 +212,6 @@ class AlwaysTrueClosure: public BoolObjectClosure { public: - void do_object(oop p) { ShouldNotReachHere(); } bool do_object_b(oop p) { return true; } }; static AlwaysTrueClosure always_true; --- old/src/share/vm/runtime/jniHandles.cpp 2013-05-10 15:56:03.724884901 +0200 +++ new/src/share/vm/runtime/jniHandles.cpp 2013-05-10 15:56:03.664884903 +0200 @@ -188,7 +188,6 @@ class AlwaysAliveClosure: public BoolObjectClosure { public: bool do_object_b(oop obj) { return true; } - void do_object(oop obj) { assert(false, "Don't call"); } }; class CountHandleClosure: public OopClosure {