8015470: Remove redundant calls of toString() on String objects

Reviewed-by: dholmes, alanb
Contributed-by: NOtavio Goncalves <otariojava@java.net>

src/share/classes/com/sun/jndi/toolkit/dir/SearchFilter.java

@@ -396,7 +396,7 @@ public class SearchFilter implements AttrFilter {
 
             // do we need to begin with the first token?
             if(proto.charAt(0) != WILDCARD_TOKEN &&
-                    !value.toString().toLowerCase(Locale.ENGLISH).startsWith(
+                    !value.toLowerCase(Locale.ENGLISH).startsWith(
                         subStrs.nextToken().toLowerCase(Locale.ENGLISH))) {
                 if(debug) {
                     System.out.println("faild initial test");

src/share/classes/java/lang/annotation/IncompleteAnnotationException.java

@@ -55,8 +55,7 @@ public class IncompleteAnnotationException extends RuntimeException {
     public IncompleteAnnotationException(
             Class<? extends Annotation> annotationType,
             String elementName) {
-        super(annotationType.getName().toString() +
-              " missing element " +
+        super(annotationType.getName() + " missing element " +
               elementName.toString());
 
         this.annotationType = annotationType;

src/share/classes/sun/rmi/rmic/Main.java

@@ -879,9 +879,9 @@ public class Main implements sun.rmi.rmic.Constants {
         }
 
         String[] args = new String[3];
-        args[0] = (arg0 != null ? arg0.toString() : "null");
-        args[1] = (arg1 != null ? arg1.toString() : "null");
-        args[2] = (arg2 != null ? arg2.toString() : "null");
+        args[0] = (arg0 != null ? arg0 : "null");
+        args[1] = (arg1 != null ? arg1 : "null");
+        args[2] = (arg2 != null ? arg2 : "null");
 
         return java.text.MessageFormat.format(format, (Object[]) args);
     }

src/share/classes/sun/tools/java/MemberDefinition.java

@@ -256,8 +256,8 @@ class MemberDefinition implements Constants {
         }
         String name = this.name.toString();
         return name.startsWith(prefixVal)
-            || name.toString().startsWith(prefixLoc)
-            || name.toString().startsWith(prefixThis);
+            || name.startsWith(prefixLoc)
+            || name.startsWith(prefixThis);
     }
 
     public boolean isAccessMethod() {

src/share/classes/sun/tools/jconsole/inspector/Utils.java

@@ -352,7 +352,7 @@ public class Utils {
             result = new Character(value.charAt(0));
         } else if (Number.class.isAssignableFrom(Utils.getClass(type))) {
             result = createNumberFromStringValue(value);
-        } else if (value == null || value.toString().equals("null")) {
+        } else if (value == null || value.equals("null")) {
             // hack for null value
             result = null;
         } else {