In general, I think Objects.requireNonNull() should often be considered idiomatic Java.

If the constant-folding is to be avoided, I would prefer to see

    "foo".toString();

have a comment like

    "foo".toString(); // Avoid mandatory constant propagation

-Joe

On 04/27/2014 08:05 PM, Otávio Gonçalves de Santana wrote:
In my opinion not, because Objects.requireNonNull is more readable than
just string.toString. This way is more understandable which field is
required and doesn't impact on performance.


On Sun, Apr 27, 2014 at 11:33 PM, David Holmes <[email protected]>wrote:

On 28/04/2014 3:41 AM, Otávio Gonçalves de Santana wrote:

sorry.
I tried answer and the message was twice.
   But Yes when has null pointer possibility I replaced to
Objects.requireNonNull.

In my opinion that is making the code worse not better.

David
-----


  I am review the code again.
The code below:

diff -r e323c74edabd
src/share/classes/com/sun/tools/example/debug/tty/Commands.java
--- a/src/share/classes/com/sun/tools/example/debug/tty/Commands.java Wed
Apr 23 11:35:40 2014 -0700
+++ b/src/share/classes/com/sun/tools/example/debug/tty/Commands.java Sun
Apr 27 14:33:45 2014 -0300
@@ -1653,20 +1653,20 @@
               String expr = t.nextToken("");
               Value val = evaluate(expr);
               if (val == null) {
-                MessageOutput.println("expr is null", expr.toString());
+                MessageOutput.println("expr is
null",Objects.requireNonNull(expr));
               } else if (dumpObject && (val instanceof ObjectReference) &&
                          !(val instanceof StringReference)) {
                   ObjectReference obj = (ObjectReference)val;
                   ReferenceType refType = obj.referenceType();
                   MessageOutput.println("expr is value",
-                                      new Object [] {expr.toString(),
+                                      new Object []
{Objects.requireNonNull(expr),

MessageOutput.format("grouping begin character")});
                   dump(obj, refType, refType);
                   MessageOutput.println("grouping end character");
               } else {
                     String strVal = getStringValue();
                     if (strVal != null) {
-                     MessageOutput.println("expr is value", new Object []
{expr.toString(),
+                     MessageOutput.println("expr is value", new Object []
{Objects.requireNonNull(expr),

   strVal});
                      }
               }
diff -r e323c74edabd
src/share/classes/java/lang/annotation/IncompleteAnnotationException.java
---
a/src/share/classes/java/lang/annotation/IncompleteAnnotationException.java
Wed
Apr 23 11:35:40 2014 -0700
+++
b/src/share/classes/java/lang/annotation/IncompleteAnnotationException.java
Sun
Apr 27 14:33:45 2014 -0300
@@ -25,6 +25,8 @@

   package java.lang.annotation;

+import java.util.Objects;
+
   /**
    * Thrown to indicate that a program has attempted to access an element
of
    * an annotation type that was added to the annotation type definition
after
@@ -56,7 +58,7 @@
               Class<? extends Annotation> annotationType,
               String elementName) {
           super(annotationType.getName() + " missing element " +
-              elementName.toString());
+                Objects.requireNonNull(elementName));

           this.annotationType = annotationType;
           this.elementName = elementName;
diff -r e323c74edabd src/share/classes/java/text/DateFormatSymbols.java
--- a/src/share/classes/java/text/DateFormatSymbols.java Wed Apr 23
11:35:40 2014 -0700
+++ b/src/share/classes/java/text/DateFormatSymbols.java Sun Apr 27
14:33:45 2014 -0300
@@ -594,7 +594,7 @@
        */
       public void setLocalPatternChars(String newLocalPatternChars) {
           // Call toString() to throw an NPE in case the argument is null
-        localPatternChars = newLocalPatternChars.toString();
+        localPatternChars = Objects.requireNonNull(
newLocalPatternChars);
           cachedHashCode = 0;
       }

diff -r e323c74edabd
src/share/classes/javax/management/modelmbean/DescriptorSupport.java
--- a/src/share/classes/javax/management/modelmbean/DescriptorSupport.java
Wed
Apr 23 11:35:40 2014 -0700
+++ b/src/share/classes/javax/management/modelmbean/DescriptorSupport.java
Sun
Apr 27 14:33:45 2014 -0300
@@ -43,13 +43,7 @@
   import java.lang.reflect.Constructor;

   import java.security.AccessController;
-import java.util.HashMap;
-import java.util.Iterator;
-import java.util.Map;
-import java.util.Set;
-import java.util.SortedMap;
-import java.util.StringTokenizer;
-import java.util.TreeMap;
+import java.util.*;
   import java.util.logging.Level;

   import javax.management.Descriptor;
@@ -665,7 +659,7 @@
                               "getFieldNames()", "Field is null");
                   }
               } else {
-                responseFields[i] = currElement.getKey().toString();
+                responseFields[i] =
Objects.requireNonNull(currElement.getKey());
               }
           }

@@ -929,7 +923,7 @@
               if (currElement != null) {
                   if (currElement.getValue() != null) {
                       // validate the field valued...
-                    if (validateField((currElement.getKey()).toString(),
+                    if
(validateField((Objects.requireNonNull(currElement.getKey())),

   (currElement.getValue()).toString())) {
                           continue;
                       } else {
diff -r e323c74edabd src/share/classes/javax/swing/
text/html/FormView.java
--- a/src/share/classes/javax/swing/text/html/FormView.java Wed Apr 23
11:35:40 2014 -0700
+++ b/src/share/classes/javax/swing/text/html/FormView.java Sun Apr 27
14:33:45 2014 -0300
@@ -478,7 +478,7 @@
                                           base.getPort(), base.getFile())
                   : new URL(base, action);
               if (!isPostMethod) {
-                String query = data.toString();
+                String query = Objects.requireNonNull(data);
                   actionURL = new URL(actionURL + "?" + query);
               }
           } catch (MalformedURLException e) {
diff -r e323c74edabd src/share/classes/sun/java2d/loops/Blit.java
--- a/src/share/classes/sun/java2d/loops/Blit.java Wed Apr 23 11:35:40
2014
-0700
+++ b/src/share/classes/sun/java2d/loops/Blit.java Sun Apr 27 14:33:45
2014
-0300
@@ -50,7 +50,7 @@

   public class Blit extends GraphicsPrimitive
   {
-    public static final String methodSignature = "Blit(...)".toString();
+    public static final String methodSignature = "Blit(...)";

       public static final int primTypeID = makePrimTypeID();

diff -r e323c74edabd src/share/classes/sun/java2d/loops/BlitBg.java
--- a/src/share/classes/sun/java2d/loops/BlitBg.java Wed Apr 23 11:35:40
2014 -0700
+++ b/src/share/classes/sun/java2d/loops/BlitBg.java Sun Apr 27 14:33:45
2014 -0300
@@ -54,7 +54,7 @@
    */
   public class BlitBg extends GraphicsPrimitive
   {
-    public static final String methodSignature =
"BlitBg(...)".toString();
+    public static final String methodSignature = "BlitBg(...)";

       public static final int primTypeID = makePrimTypeID();

diff -r e323c74edabd src/share/classes/sun/java2d/
loops/DrawGlyphList.java
--- a/src/share/classes/sun/java2d/loops/DrawGlyphList.java Wed Apr 23
11:35:40 2014 -0700
+++ b/src/share/classes/sun/java2d/loops/DrawGlyphList.java Sun Apr 27
14:33:45 2014 -0300
@@ -39,7 +39,7 @@
    */
   public class DrawGlyphList extends GraphicsPrimitive {

-    public final static String methodSignature =
"DrawGlyphList(...)".toString();
+    public final static String methodSignature = "DrawGlyphList(...)";

       public final static int primTypeID = makePrimTypeID();

diff -r e323c74edabd src/share/classes/sun/java2d/
loops/DrawGlyphListAA.java
--- a/src/share/classes/sun/java2d/loops/DrawGlyphListAA.java Wed Apr 23
11:35:40 2014 -0700
+++ b/src/share/classes/sun/java2d/loops/DrawGlyphListAA.java Sun Apr 27
14:33:45 2014 -0300
@@ -39,7 +39,7 @@
    */
   public class DrawGlyphListAA extends GraphicsPrimitive {

-    public final static String methodSignature =
"DrawGlyphListAA(...)".toString();
+    public final static String methodSignature = "DrawGlyphListAA(...)";

       public final static int primTypeID = makePrimTypeID();

diff -r e323c74edabd
src/share/classes/sun/java2d/loops/DrawGlyphListLCD.java
--- a/src/share/classes/sun/java2d/loops/DrawGlyphListLCD.java Wed Apr 23
11:35:40 2014 -0700
+++ b/src/share/classes/sun/java2d/loops/DrawGlyphListLCD.java Sun Apr 27
14:33:45 2014 -0300
@@ -40,7 +40,7 @@
   public class DrawGlyphListLCD extends GraphicsPrimitive {

       public final static String
-        methodSignature = "DrawGlyphListLCD(...)".toString();
+        methodSignature = "DrawGlyphListLCD(...)";

       public final static int primTypeID = makePrimTypeID();

diff -r e323c74edabd src/share/classes/sun/java2d/loops/DrawLine.java
--- a/src/share/classes/sun/java2d/loops/DrawLine.java Wed Apr 23
11:35:40
2014 -0700
+++ b/src/share/classes/sun/java2d/loops/DrawLine.java Sun Apr 27
14:33:45
2014 -0300
@@ -44,7 +44,7 @@
    */
   public class DrawLine extends GraphicsPrimitive
   {
-    public final static String methodSignature =
"DrawLine(...)".toString();
+    public final static String methodSignature = "DrawLine(...)";

       public final static int primTypeID = makePrimTypeID();

diff -r e323c74edabd
src/share/classes/sun/java2d/loops/DrawParallelogram.java
--- a/src/share/classes/sun/java2d/loops/DrawParallelogram.java Wed Apr
23
11:35:40 2014 -0700
+++ b/src/share/classes/sun/java2d/loops/DrawParallelogram.java Sun Apr
27
14:33:45 2014 -0300
@@ -43,7 +43,7 @@
   public class DrawParallelogram extends GraphicsPrimitive
   {
       public final static String methodSignature =
-        "DrawParallelogram(...)".toString();
+        "DrawParallelogram(...)";

       public final static int primTypeID = makePrimTypeID();

diff -r e323c74edabd src/share/classes/sun/java2d/loops/DrawPath.java
--- a/src/share/classes/sun/java2d/loops/DrawPath.java Wed Apr 23
11:35:40
2014 -0700
+++ b/src/share/classes/sun/java2d/loops/DrawPath.java Sun Apr 27
14:33:45
2014 -0300
@@ -39,7 +39,7 @@
   public class DrawPath extends GraphicsPrimitive {

       public final static String methodSignature =
-        "DrawPath(...)".toString();
+        "DrawPath(...)";

       public final static int primTypeID = makePrimTypeID();

diff -r e323c74edabd src/share/classes/sun/java2d/loops/DrawPolygons.java
--- a/src/share/classes/sun/java2d/loops/DrawPolygons.java Wed Apr 23
11:35:40 2014 -0700
+++ b/src/share/classes/sun/java2d/loops/DrawPolygons.java Sun Apr 27
14:33:45 2014 -0300
@@ -41,7 +41,7 @@
    */
   public class DrawPolygons extends GraphicsPrimitive
   {
-    public final static String methodSignature =
"DrawPolygons(...)".toString();
+    public final static String methodSignature = "DrawPolygons(...)";

       public final static int primTypeID = makePrimTypeID();

diff -r e323c74edabd src/share/classes/sun/java2d/loops/DrawRect.java
--- a/src/share/classes/sun/java2d/loops/DrawRect.java Wed Apr 23
11:35:40
2014 -0700
+++ b/src/share/classes/sun/java2d/loops/DrawRect.java Sun Apr 27
14:33:45
2014 -0300
@@ -44,7 +44,7 @@
    */
   public class DrawRect extends GraphicsPrimitive
   {
-    public final static String methodSignature =
"DrawRect(...)".toString();
+    public final static String methodSignature = "DrawRect(...)";

       public final static int primTypeID = makePrimTypeID();

diff -r e323c74edabd
src/share/classes/sun/java2d/loops/FillParallelogram.java
--- a/src/share/classes/sun/java2d/loops/FillParallelogram.java Wed Apr
23
11:35:40 2014 -0700
+++ b/src/share/classes/sun/java2d/loops/FillParallelogram.java Sun Apr
27
14:33:45 2014 -0300
@@ -41,7 +41,7 @@
   public class FillParallelogram extends GraphicsPrimitive
   {
       public final static String methodSignature =
-        "FillParallelogram(...)".toString();
+        "FillParallelogram(...)";

       public final static int primTypeID = makePrimTypeID();

diff -r e323c74edabd src/share/classes/sun/java2d/loops/FillPath.java
--- a/src/share/classes/sun/java2d/loops/FillPath.java Wed Apr 23
11:35:40
2014 -0700
+++ b/src/share/classes/sun/java2d/loops/FillPath.java Sun Apr 27
14:33:45
2014 -0300
@@ -39,7 +39,7 @@
   public class FillPath extends GraphicsPrimitive {

       public final static String methodSignature =
-        "FillPath(...)".toString();
+        "FillPath(...)";

       public final static int primTypeID = makePrimTypeID();

diff -r e323c74edabd src/share/classes/sun/java2d/loops/FillRect.java
--- a/src/share/classes/sun/java2d/loops/FillRect.java Wed Apr 23
11:35:40
2014 -0700
+++ b/src/share/classes/sun/java2d/loops/FillRect.java Sun Apr 27
14:33:45
2014 -0300
@@ -44,7 +44,7 @@
    */
   public class FillRect extends GraphicsPrimitive
   {
-    public final static String methodSignature =
"FillRect(...)".toString();
+    public final static String methodSignature = "FillRect(...)";

       public final static int primTypeID = makePrimTypeID();

diff -r e323c74edabd src/share/classes/sun/java2d/loops/FillSpans.java
--- a/src/share/classes/sun/java2d/loops/FillSpans.java Wed Apr 23
11:35:40
2014 -0700
+++ b/src/share/classes/sun/java2d/loops/FillSpans.java Sun Apr 27
14:33:45
2014 -0300
@@ -45,7 +45,7 @@
    */
   public class FillSpans extends GraphicsPrimitive
   {
-    public final static String methodSignature =
"FillSpans(...)".toString();
+    public final static String methodSignature = "FillSpans(...)";

       public final static int primTypeID = makePrimTypeID();

diff -r e323c74edabd src/share/classes/sun/java2d/loops/MaskBlit.java
--- a/src/share/classes/sun/java2d/loops/MaskBlit.java Wed Apr 23
11:35:40
2014 -0700
+++ b/src/share/classes/sun/java2d/loops/MaskBlit.java Sun Apr 27
14:33:45
2014 -0300
@@ -48,7 +48,7 @@

   public class MaskBlit extends GraphicsPrimitive
   {
-    public static final String methodSignature =
"MaskBlit(...)".toString();
+    public static final String methodSignature = "MaskBlit(...)";

       public static final int primTypeID = makePrimTypeID();

diff -r e323c74edabd src/share/classes/sun/java2d/loops/MaskFill.java
--- a/src/share/classes/sun/java2d/loops/MaskFill.java Wed Apr 23
11:35:40
2014 -0700
+++ b/src/share/classes/sun/java2d/loops/MaskFill.java Sun Apr 27
14:33:45
2014 -0300
@@ -50,11 +50,11 @@
    */
   public class MaskFill extends GraphicsPrimitive
   {
-    public static final String methodSignature =
"MaskFill(...)".toString();
+    public static final String methodSignature = "MaskFill(...)";
       public static final String fillPgramSignature =
-        "FillAAPgram(...)".toString();
+        "FillAAPgram(...)";
       public static final String drawPgramSignature =
-        "DrawAAPgram(...)".toString();
+        "DrawAAPgram(...)";

       public static final int primTypeID = makePrimTypeID();

diff -r e323c74edabd src/share/classes/sun/java2d/loops/ScaledBlit.java
--- a/src/share/classes/sun/java2d/loops/ScaledBlit.java Wed Apr 23
11:35:40 2014 -0700
+++ b/src/share/classes/sun/java2d/loops/ScaledBlit.java Sun Apr 27
14:33:45 2014 -0300
@@ -45,7 +45,7 @@

   public class ScaledBlit extends GraphicsPrimitive
   {
-    public static final String methodSignature =
"ScaledBlit(...)".toString();
+    public static final String methodSignature = "ScaledBlit(...)";

       public static final int primTypeID = makePrimTypeID();

diff -r e323c74edabd src/share/classes/sun/java2d/
loops/TransformBlit.java
--- a/src/share/classes/sun/java2d/loops/TransformBlit.java Wed Apr 23
11:35:40 2014 -0700
+++ b/src/share/classes/sun/java2d/loops/TransformBlit.java Sun Apr 27
14:33:45 2014 -0300
@@ -47,7 +47,7 @@
   public class TransformBlit extends GraphicsPrimitive
   {
       public static final String methodSignature =
-        "TransformBlit(...)".toString();
+        "TransformBlit(...)";

       public static final int primTypeID = makePrimTypeID();

diff -r e323c74edabd src/share/classes/sun/java2d/
loops/TransformHelper.java
--- a/src/share/classes/sun/java2d/loops/TransformHelper.java Wed Apr 23
11:35:40 2014 -0700
+++ b/src/share/classes/sun/java2d/loops/TransformHelper.java Sun Apr 27
14:33:45 2014 -0300
@@ -46,7 +46,7 @@
   public class TransformHelper extends GraphicsPrimitive
   {
       public static final String methodSignature =
-        "TransformHelper(...)".toString();
+        "TransformHelper(...)";

       public static final int primTypeID = makePrimTypeID();

diff -r e323c74edabd src/share/classes/sun/misc/ExtensionInfo.java
--- a/src/share/classes/sun/misc/ExtensionInfo.java Wed Apr 23 11:35:40
2014 -0700
+++ b/src/share/classes/sun/misc/ExtensionInfo.java Sun Apr 27 14:33:45
2014 -0300
@@ -25,6 +25,7 @@

   package sun.misc;

+import java.util.Objects;
   import java.util.StringTokenizer;
   import java.util.jar.Attributes;
   import java.util.jar.Attributes.Name;
@@ -260,11 +261,11 @@

           // Convert token into meaning number for comparision
           if (stk.hasMoreTokens())
-            n = convertToken(stk.nextToken().toString());
+            n = convertToken(Objects.requireNonNull(stk.nextToken()));

           // Convert token into meaning number for comparision
           if (ttk.hasMoreTokens())
-            m = convertToken(ttk.nextToken().toString());
+            m = convertToken(Objects.requireNonNull(ttk.nextToken()));

           if (n > m)
               return 1;
diff -r e323c74edabd
src/share/classes/sun/tools/jconsole/inspector/Utils.java
--- a/src/share/classes/sun/tools/jconsole/inspector/Utils.java Wed Apr
23
11:35:40 2014 -0700
+++ b/src/share/classes/sun/tools/jconsole/inspector/Utils.java Sun Apr
27
14:33:45 2014 -0300
@@ -378,7 +378,7 @@
               if (userInput instanceof XObject) {
                   result[i] = ((XObject) userInput).getObject();
               } else {
-                result[i] = createObjectFromString(params[i].toString(),
+                result[i] =
createObjectFromString(Objects.requireNonNull(params[i]),
                           (String) userInput);
               }
           }




On Sun, Apr 27, 2014 at 1:09 PM, Remi Forax <[email protected]> wrote:

  On 04/27/2014 03:15 PM, Otávio Gonçalves de Santana wrote:
  There is an issue that was opened to remove redundant calls of
toString()
on String objects. [1]
I went deep on all JVM sources and I found all, 32 changes.


[1]https://bugs.openjdk.java.net/browse/JDK-8015470


Otavio,
calling toString() on a String has the side effect to implicitly check
that the reference is not null.
Do you have checked that for each redundant call, the String can never be
null ?

regards,
Rémi




diff -r e323c74edabd
src/share/classes/com/sun/tools/example/debug/tty/Commands.java
--- a/src/share/classes/com/sun/tools/example/debug/tty/Commands.java
Wed
Apr 23 11:35:40 2014 -0700
+++ b/src/share/classes/com/sun/tools/example/debug/tty/Commands.java
Sat
Apr 26 01:40:27 2014 -0300
@@ -1653,20 +1653,20 @@
                String expr = t.nextToken("");
                Value val = evaluate(expr);
                if (val == null) {
-                MessageOutput.println("expr is null", expr.toString());
+                MessageOutput.println("expr is null", expr);
                } else if (dumpObject && (val instanceof
ObjectReference) &&
                           !(val instanceof StringReference)) {
                    ObjectReference obj = (ObjectReference)val;
                    ReferenceType refType = obj.referenceType();
                    MessageOutput.println("expr is value",
-                                      new Object [] {expr.toString(),
+                                      new Object [] {expr,

MessageOutput.format("grouping begin character")});
                    dump(obj, refType, refType);
                    MessageOutput.println("grouping end character");
                } else {
                      String strVal = getStringValue();
                      if (strVal != null) {
-                     MessageOutput.println("expr is value", new Object
[]
{expr.toString(),
+                     MessageOutput.println("expr is value", new Object
[]
{expr,

    strVal});
                       }
                }
diff -r e323c74edabd
src/share/classes/java/lang/annotation/IncompleteAnnotationException.
java
---
a/src/share/classes/java/lang/annotation/IncompleteAnnotationException.
java
Wed
Apr 23 11:35:40 2014 -0700
+++
b/src/share/classes/java/lang/annotation/IncompleteAnnotationException.
java
Sat
Apr 26 01:40:27 2014 -0300
@@ -56,7 +56,7 @@
                Class<? extends Annotation> annotationType,
                String elementName) {
            super(annotationType.getName() + " missing element " +
-              elementName.toString());
+              elementName);

            this.annotationType = annotationType;
            this.elementName = elementName;
diff -r e323c74edabd src/share/classes/java/text/DateFormatSymbols.java
--- a/src/share/classes/java/text/DateFormatSymbols.java Wed Apr 23
11:35:40 2014 -0700
+++ b/src/share/classes/java/text/DateFormatSymbols.java Sat Apr 26
01:40:27
2014 -0300
@@ -593,8 +593,8 @@
         * pattern characters.
         */
        public void setLocalPatternChars(String newLocalPatternChars) {
-        // Call toString() to throw an NPE in case the argument is null
-        localPatternChars = newLocalPatternChars.toString();
+
+        localPatternChars = Objects.requireNonNull(
newLocalPatternChars);
            cachedHashCode = 0;
        }

diff -r e323c74edabd
src/share/classes/javax/management/modelmbean/DescriptorSupport.java
--- a/src/share/classes/javax/management/modelmbean/
DescriptorSupport.java
Wed
Apr 23 11:35:40 2014 -0700
+++ b/src/share/classes/javax/management/modelmbean/
DescriptorSupport.java
Sat
Apr 26 01:40:27 2014 -0300
@@ -665,7 +665,7 @@
                                "getFieldNames()", "Field is null");
                    }
                } else {
-                responseFields[i] = currElement.getKey().toString();
+                responseFields[i] = currElement.getKey();
                }
            }

@@ -929,8 +929,8 @@
                if (currElement != null) {
                    if (currElement.getValue() != null) {
                        // validate the field valued...
-                    if (validateField((currElement.
getKey()).toString(),
-
    (currElement.getValue()).toString())) {
+                    if (validateField((currElement.getKey()),
+                                      (currElement.getValue()))) {
                            continue;
                        } else {
                            if (MODELMBEAN_LOGGER.isLoggable(
Level.FINEST))
{
diff -r e323c74edabd src/share/classes/javax/swing/
text/html/FormView.java
--- a/src/share/classes/javax/swing/text/html/FormView.java Wed Apr 23
11:35:40 2014 -0700
+++ b/src/share/classes/javax/swing/text/html/FormView.java Sat Apr 26
01:40:27 2014 -0300
@@ -478,7 +478,7 @@
                                            base.getPort(),
base.getFile())
                    : new URL(base, action);
                if (!isPostMethod) {
-                String query = data.toString();
+                String query = data;
                    actionURL = new URL(actionURL + "?" + query);
                }
            } catch (MalformedURLException e) {
diff -r e323c74edabd src/share/classes/sun/java2d/loops/Blit.java
--- a/src/share/classes/sun/java2d/loops/Blit.java Wed Apr 23 11:35:40
2014
-0700
+++ b/src/share/classes/sun/java2d/loops/Blit.java Sat Apr 26 01:40:27
2014
-0300
@@ -50,7 +50,7 @@

    public class Blit extends GraphicsPrimitive
    {
-    public static final String methodSignature =
"Blit(...)".toString();
+    public static final String methodSignature = "Blit(...)";

        public static final int primTypeID = makePrimTypeID();

diff -r e323c74edabd src/share/classes/sun/java2d/loops/BlitBg.java
--- a/src/share/classes/sun/java2d/loops/BlitBg.java Wed Apr 23
11:35:40
2014 -0700
+++ b/src/share/classes/sun/java2d/loops/BlitBg.java Sat Apr 26
01:40:27
2014 -0300
@@ -54,7 +54,7 @@
     */
    public class BlitBg extends GraphicsPrimitive
    {
-    public static final String methodSignature =
"BlitBg(...)".toString();
+    public static final String methodSignature = "BlitBg(...)";

        public static final int primTypeID = makePrimTypeID();

diff -r e323c74edabd src/share/classes/sun/java2d/
loops/DrawGlyphList.java
--- a/src/share/classes/sun/java2d/loops/DrawGlyphList.java Wed Apr 23
11:35:40 2014 -0700
+++ b/src/share/classes/sun/java2d/loops/DrawGlyphList.java Sat Apr 26
01:40:27 2014 -0300
@@ -39,7 +39,7 @@
     */
    public class DrawGlyphList extends GraphicsPrimitive {

-    public final static String methodSignature =
"DrawGlyphList(...)".toString();
+    public final static String methodSignature = "DrawGlyphList(...)";

        public final static int primTypeID = makePrimTypeID();

diff -r e323c74edabd src/share/classes/sun/java2d/
loops/DrawGlyphListAA.java
--- a/src/share/classes/sun/java2d/loops/DrawGlyphListAA.java Wed Apr
23
11:35:40 2014 -0700
+++ b/src/share/classes/sun/java2d/loops/DrawGlyphListAA.java Sat Apr
26
01:40:27 2014 -0300
@@ -39,7 +39,7 @@
     */
    public class DrawGlyphListAA extends GraphicsPrimitive {

-    public final static String methodSignature =
"DrawGlyphListAA(...)".toString();
+    public final static String methodSignature =
"DrawGlyphListAA(...)";

        public final static int primTypeID = makePrimTypeID();

diff -r e323c74edabd
src/share/classes/sun/java2d/loops/DrawGlyphListLCD.java
--- a/src/share/classes/sun/java2d/loops/DrawGlyphListLCD.java Wed Apr
23
11:35:40 2014 -0700
+++ b/src/share/classes/sun/java2d/loops/DrawGlyphListLCD.java Sat Apr
26
01:40:27 2014 -0300
@@ -40,7 +40,7 @@
    public class DrawGlyphListLCD extends GraphicsPrimitive {

        public final static String
-        methodSignature = "DrawGlyphListLCD(...)".toString();
+        methodSignature = "DrawGlyphListLCD(...)";

        public final static int primTypeID = makePrimTypeID();

diff -r e323c74edabd src/share/classes/sun/java2d/loops/DrawLine.java
--- a/src/share/classes/sun/java2d/loops/DrawLine.java Wed Apr 23
11:35:40
2014 -0700
+++ b/src/share/classes/sun/java2d/loops/DrawLine.java Sat Apr 26
01:40:27
2014 -0300
@@ -44,7 +44,7 @@
     */
    public class DrawLine extends GraphicsPrimitive
    {
-    public final static String methodSignature =
"DrawLine(...)".toString();
+    public final static String methodSignature = "DrawLine(...)";

        public final static int primTypeID = makePrimTypeID();

diff -r e323c74edabd
src/share/classes/sun/java2d/loops/DrawParallelogram.java
--- a/src/share/classes/sun/java2d/loops/DrawParallelogram.java Wed Apr
23
11:35:40 2014 -0700
+++ b/src/share/classes/sun/java2d/loops/DrawParallelogram.java Sat Apr
26
01:40:27 2014 -0300
@@ -43,7 +43,7 @@
    public class DrawParallelogram extends GraphicsPrimitive
    {
        public final static String methodSignature =
-        "DrawParallelogram(...)".toString();
+        "DrawParallelogram(...)";

        public final static int primTypeID = makePrimTypeID();

diff -r e323c74edabd src/share/classes/sun/java2d/loops/DrawPath.java
--- a/src/share/classes/sun/java2d/loops/DrawPath.java Wed Apr 23
11:35:40
2014 -0700
+++ b/src/share/classes/sun/java2d/loops/DrawPath.java Sat Apr 26
01:40:27
2014 -0300
@@ -39,7 +39,7 @@
    public class DrawPath extends GraphicsPrimitive {

        public final static String methodSignature =
-        "DrawPath(...)".toString();
+        "DrawPath(...)";

        public final static int primTypeID = makePrimTypeID();

diff -r e323c74edabd src/share/classes/sun/java2d/
loops/DrawPolygons.java
--- a/src/share/classes/sun/java2d/loops/DrawPolygons.java Wed Apr 23
11:35:40 2014 -0700
+++ b/src/share/classes/sun/java2d/loops/DrawPolygons.java Sat Apr 26
01:40:27
2014 -0300
@@ -41,7 +41,7 @@
     */
    public class DrawPolygons extends GraphicsPrimitive
    {
-    public final static String methodSignature =
"DrawPolygons(...)".toString();
+    public final static String methodSignature = "DrawPolygons(...)";

        public final static int primTypeID = makePrimTypeID();

diff -r e323c74edabd src/share/classes/sun/java2d/loops/DrawRect.java
--- a/src/share/classes/sun/java2d/loops/DrawRect.java Wed Apr 23
11:35:40
2014 -0700
+++ b/src/share/classes/sun/java2d/loops/DrawRect.java Sat Apr 26
01:40:27
2014 -0300
@@ -44,7 +44,7 @@
     */
    public class DrawRect extends GraphicsPrimitive
    {
-    public final static String methodSignature =
"DrawRect(...)".toString();
+    public final static String methodSignature = "DrawRect(...)";

        public final static int primTypeID = makePrimTypeID();

diff -r e323c74edabd
src/share/classes/sun/java2d/loops/FillParallelogram.java
--- a/src/share/classes/sun/java2d/loops/FillParallelogram.java Wed Apr
23
11:35:40 2014 -0700
+++ b/src/share/classes/sun/java2d/loops/FillParallelogram.java Sat Apr
26
01:40:27 2014 -0300
@@ -41,7 +41,7 @@
    public class FillParallelogram extends GraphicsPrimitive
    {
        public final static String methodSignature =
-        "FillParallelogram(...)".toString();
+        "FillParallelogram(...)";

        public final static int primTypeID = makePrimTypeID();

diff -r e323c74edabd src/share/classes/sun/java2d/loops/FillPath.java
--- a/src/share/classes/sun/java2d/loops/FillPath.java Wed Apr 23
11:35:40
2014 -0700
+++ b/src/share/classes/sun/java2d/loops/FillPath.java Sat Apr 26
01:40:27
2014 -0300
@@ -39,7 +39,7 @@
    public class FillPath extends GraphicsPrimitive {

        public final static String methodSignature =
-        "FillPath(...)".toString();
+        "FillPath(...)";

        public final static int primTypeID = makePrimTypeID();

diff -r e323c74edabd src/share/classes/sun/java2d/loops/FillRect.java
--- a/src/share/classes/sun/java2d/loops/FillRect.java Wed Apr 23
11:35:40
2014 -0700
+++ b/src/share/classes/sun/java2d/loops/FillRect.java Sat Apr 26
01:40:27
2014 -0300
@@ -44,7 +44,7 @@
     */
    public class FillRect extends GraphicsPrimitive
    {
-    public final static String methodSignature =
"FillRect(...)".toString();
+    public final static String methodSignature = "FillRect(...)";

        public final static int primTypeID = makePrimTypeID();

diff -r e323c74edabd src/share/classes/sun/java2d/loops/FillSpans.java
--- a/src/share/classes/sun/java2d/loops/FillSpans.java Wed Apr 23
11:35:40
2014 -0700
+++ b/src/share/classes/sun/java2d/loops/FillSpans.java Sat Apr 26
01:40:27
2014 -0300
@@ -45,7 +45,7 @@
     */
    public class FillSpans extends GraphicsPrimitive
    {
-    public final static String methodSignature =
"FillSpans(...)".toString();
+    public final static String methodSignature = "FillSpans(...)";

        public final static int primTypeID = makePrimTypeID();

diff -r e323c74edabd src/share/classes/sun/java2d/loops/MaskBlit.java
--- a/src/share/classes/sun/java2d/loops/MaskBlit.java Wed Apr 23
11:35:40
2014 -0700
+++ b/src/share/classes/sun/java2d/loops/MaskBlit.java Sat Apr 26
01:40:27
2014 -0300
@@ -48,7 +48,7 @@

    public class MaskBlit extends GraphicsPrimitive
    {
-    public static final String methodSignature =
"MaskBlit(...)".toString();
+    public static final String methodSignature = "MaskBlit(...)";

        public static final int primTypeID = makePrimTypeID();

diff -r e323c74edabd src/share/classes/sun/java2d/loops/MaskFill.java
--- a/src/share/classes/sun/java2d/loops/MaskFill.java Wed Apr 23
11:35:40
2014 -0700
+++ b/src/share/classes/sun/java2d/loops/MaskFill.java Sat Apr 26
01:40:27
2014 -0300
@@ -50,11 +50,11 @@
     */
    public class MaskFill extends GraphicsPrimitive
    {
-    public static final String methodSignature =
"MaskFill(...)".toString();
+    public static final String methodSignature = "MaskFill(...)";
        public static final String fillPgramSignature =
-        "FillAAPgram(...)".toString();
+        "FillAAPgram(...)";
        public static final String drawPgramSignature =
-        "DrawAAPgram(...)".toString();
+        "DrawAAPgram(...)";

        public static final int primTypeID = makePrimTypeID();

diff -r e323c74edabd src/share/classes/sun/java2d/loops/ScaledBlit.java
--- a/src/share/classes/sun/java2d/loops/ScaledBlit.java Wed Apr 23
11:35:40 2014 -0700
+++ b/src/share/classes/sun/java2d/loops/ScaledBlit.java Sat Apr 26
01:40:27
2014 -0300
@@ -45,7 +45,7 @@

    public class ScaledBlit extends GraphicsPrimitive
    {
-    public static final String methodSignature =
"ScaledBlit(...)".toString();
+    public static final String methodSignature = "ScaledBlit(...)";

        public static final int primTypeID = makePrimTypeID();

diff -r e323c74edabd src/share/classes/sun/java2d/
loops/TransformBlit.java
--- a/src/share/classes/sun/java2d/loops/TransformBlit.java Wed Apr 23
11:35:40 2014 -0700
+++ b/src/share/classes/sun/java2d/loops/TransformBlit.java Sat Apr 26
01:40:27 2014 -0300
@@ -47,7 +47,7 @@
    public class TransformBlit extends GraphicsPrimitive
    {
        public static final String methodSignature =
-        "TransformBlit(...)".toString();
+        "TransformBlit(...)";

        public static final int primTypeID = makePrimTypeID();

diff -r e323c74edabd src/share/classes/sun/java2d/
loops/TransformHelper.java
--- a/src/share/classes/sun/java2d/loops/TransformHelper.java Wed Apr
23
11:35:40 2014 -0700
+++ b/src/share/classes/sun/java2d/loops/TransformHelper.java Sat Apr
26
01:40:27 2014 -0300
@@ -46,7 +46,7 @@
    public class TransformHelper extends GraphicsPrimitive
    {
        public static final String methodSignature =
-        "TransformHelper(...)".toString();
+        "TransformHelper(...)";

        public static final int primTypeID = makePrimTypeID();

diff -r e323c74edabd src/share/classes/sun/misc/ExtensionInfo.java
--- a/src/share/classes/sun/misc/ExtensionInfo.java Wed Apr 23 11:35:40
2014 -0700
+++ b/src/share/classes/sun/misc/ExtensionInfo.java Sat Apr 26 01:40:27
2014 -0300
@@ -260,11 +260,11 @@

            // Convert token into meaning number for comparision
            if (stk.hasMoreTokens())
-            n = convertToken(stk.nextToken().toString());
+            n = convertToken(stk.nextToken());

            // Convert token into meaning number for comparision
            if (ttk.hasMoreTokens())
-            m = convertToken(ttk.nextToken().toString());
+            m = convertToken(ttk.nextToken());

            if (n > m)
                return 1;
diff -r e323c74edabd
src/share/classes/sun/tools/jconsole/inspector/Utils.java
--- a/src/share/classes/sun/tools/jconsole/inspector/Utils.java Wed Apr
23
11:35:40 2014 -0700
+++ b/src/share/classes/sun/tools/jconsole/inspector/Utils.java Sat Apr
26
01:40:27 2014 -0300
@@ -378,7 +378,7 @@
                if (userInput instanceof XObject) {
                    result[i] = ((XObject) userInput).getObject();
                } else {
-                result[i] = createObjectFromString(params[
i].toString(),
+                result[i] = createObjectFromString(params[i],
                            (String) userInput);
                }
            }







Reply via email to