8268520: VirtualSpace::print_on() should be const

Reviewed-by: kbarrett, stuefe, tschatzl

Author: iklam

src/hotspot/share/gc/epsilon/epsilonHeap.cpp

@@ -293,8 +293,7 @@ void EpsilonHeap::object_iterate(ObjectClosure *cl) {
 void EpsilonHeap::print_on(outputStream *st) const {
   st->print_cr("Epsilon Heap");
 
-  // Cast away constness:
-  ((VirtualSpace)_virtual_space).print_on(st);
+  _virtual_space.print_on(st);
 
   if (_space != NULL) {
     st->print_cr("Allocation space:");

src/hotspot/share/memory/virtualspace.cpp

@@ -1036,7 +1036,7 @@ void VirtualSpace::check_for_contiguity() {
   assert(high() <= upper_high(), "upper high");
 }
 
-void VirtualSpace::print_on(outputStream* out) {
+void VirtualSpace::print_on(outputStream* out) const {
   out->print   ("Virtual space:");
   if (special()) out->print(" (pinned in memory)");
   out->cr();
@@ -1046,7 +1046,7 @@ void VirtualSpace::print_on(outputStream* out) {
   out->print_cr(" - [low_b, high_b]: [" INTPTR_FORMAT ", " INTPTR_FORMAT "]",  p2i(low_boundary()), p2i(high_boundary()));
 }
 
-void VirtualSpace::print() {
+void VirtualSpace::print() const {
   print_on(tty);
 }
 

src/hotspot/share/memory/virtualspace.hpp

@@ -240,8 +240,8 @@ class VirtualSpace {
   void check_for_contiguity() PRODUCT_RETURN;
 
   // Debugging
-  void print_on(outputStream* out) PRODUCT_RETURN;
-  void print();
+  void print_on(outputStream* out) const PRODUCT_RETURN;
+  void print() const;
 };
 
 #endif // SHARE_MEMORY_VIRTUALSPACE_HPP