changeset 20261:27188ae8bbbb

8035648: Don't use Handle in java_lang_String::print Reviewed-by: coleenp, pliden
author stefank
date Thu, 27 Feb 2014 10:34:55 +0100
parents 556a06aec3fa
children 76b588255908
files src/share/vm/classfile/javaClasses.cpp src/share/vm/classfile/javaClasses.hpp src/share/vm/oops/instanceKlass.cpp
diffstat 3 files changed, 7 insertions(+), 9 deletions(-) [+]
line wrap: on
line diff
--- a/src/share/vm/classfile/javaClasses.cpp	Fri Feb 21 10:19:09 2014 +0100
+++ b/src/share/vm/classfile/javaClasses.cpp	Thu Feb 27 10:34:55 2014 +0100
@@ -463,12 +463,11 @@
   return true;
 }
 
-void java_lang_String::print(Handle java_string, outputStream* st) {
-  oop          obj    = java_string();
-  assert(obj->klass() == SystemDictionary::String_klass(), "must be java_string");
-  typeArrayOop value  = java_lang_String::value(obj);
-  int          offset = java_lang_String::offset(obj);
-  int          length = java_lang_String::length(obj);
+void java_lang_String::print(oop java_string, outputStream* st) {
+  assert(java_string->klass() == SystemDictionary::String_klass(), "must be java_string");
+  typeArrayOop value  = java_lang_String::value(java_string);
+  int          offset = java_lang_String::offset(java_string);
+  int          length = java_lang_String::length(java_string);
 
   int end = MIN2(length, 100);
   if (value == NULL) {
--- a/src/share/vm/classfile/javaClasses.hpp	Fri Feb 21 10:19:09 2014 +0100
+++ b/src/share/vm/classfile/javaClasses.hpp	Thu Feb 27 10:34:55 2014 +0100
@@ -208,7 +208,7 @@
   }
 
   // Debugging
-  static void print(Handle java_string, outputStream* st);
+  static void print(oop java_string, outputStream* st);
   friend class JavaClasses;
 };
 
--- a/src/share/vm/oops/instanceKlass.cpp	Fri Feb 21 10:19:09 2014 +0100
+++ b/src/share/vm/oops/instanceKlass.cpp	Thu Feb 27 10:34:55 2014 +0100
@@ -3042,8 +3042,7 @@
         offset          <= (juint) value->length() &&
         offset + length <= (juint) value->length()) {
       st->print(BULLET"string: ");
-      Handle h_obj(obj);
-      java_lang_String::print(h_obj, st);
+      java_lang_String::print(obj, st);
       st->cr();
       if (!WizardMode)  return;  // that is enough
     }