Remove redundant calls of toString()

[Phil Race]

I am quite sure that you are right that these were deliberate so
I am adding Jim who was the author of all of the 2d ones to explain the 
intent.

BTW Otávio, changes in 2D code like this should not slip by on the 
core-libs list.
It ought to go to 2d-dev.

-phil.

On 4/28/2014 9:24 AM, Louis Wasserman wrote:
> -    public final static String methodSignature =
> "DrawGlyphList(...)".toString();
> +    public final static String methodSignature = "DrawGlyphList(...)";
>
> I conjecture that this, and other similar examples, are to avoid having the
> string be a compile-time constant that can be inlined into other files.
>   Has anyone checked that there aren't any issues arising from changing this?
>
>
>
>
>

n 4/27/2014 6:15 AM, 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
>
> 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();
>
>