From 9b2013182516b77a39123894ad1b71619bd9785b Mon Sep 17 00:00:00 2001 From: Harvey Harrison Date: Fri, 18 Oct 2013 00:16:51 -0700 Subject: jogl: avoid creating a second String object, one is enough One String is already being built, passing it to new String() is just wasteful as the temp String can be returned just as easily. Signed-off-by: Harvey Harrison --- src/nativewindow/classes/javax/media/nativewindow/util/Dimension.java | 2 +- src/nativewindow/classes/javax/media/nativewindow/util/Insets.java | 2 +- src/nativewindow/classes/javax/media/nativewindow/util/Point.java | 2 +- src/nativewindow/classes/javax/media/nativewindow/util/Rectangle.java | 2 +- src/nativewindow/classes/javax/media/nativewindow/util/SurfaceSize.java | 2 +- 5 files changed, 5 insertions(+), 5 deletions(-) (limited to 'src/nativewindow') diff --git a/src/nativewindow/classes/javax/media/nativewindow/util/Dimension.java b/src/nativewindow/classes/javax/media/nativewindow/util/Dimension.java index b8dc53c83..24ccc836a 100644 --- a/src/nativewindow/classes/javax/media/nativewindow/util/Dimension.java +++ b/src/nativewindow/classes/javax/media/nativewindow/util/Dimension.java @@ -87,7 +87,7 @@ public class Dimension implements Cloneable, DimensionImmutable { @Override public String toString() { - return new String(width+" x "+height); + return width + " x " + height; } @Override diff --git a/src/nativewindow/classes/javax/media/nativewindow/util/Insets.java b/src/nativewindow/classes/javax/media/nativewindow/util/Insets.java index c84359dcd..3644916fe 100644 --- a/src/nativewindow/classes/javax/media/nativewindow/util/Insets.java +++ b/src/nativewindow/classes/javax/media/nativewindow/util/Insets.java @@ -103,7 +103,7 @@ public class Insets implements Cloneable, InsetsImmutable { @Override public String toString() { - return new String("[ l "+l+", r "+r+" - t "+t+", b "+b+" - "+getTotalWidth()+"x"+getTotalHeight()+"]"); + return "[ l "+l+", r "+r+" - t "+t+", b "+b+" - "+getTotalWidth()+"x"+getTotalHeight()+"]"; } } diff --git a/src/nativewindow/classes/javax/media/nativewindow/util/Point.java b/src/nativewindow/classes/javax/media/nativewindow/util/Point.java index 57c6fb716..331c1388e 100644 --- a/src/nativewindow/classes/javax/media/nativewindow/util/Point.java +++ b/src/nativewindow/classes/javax/media/nativewindow/util/Point.java @@ -99,7 +99,7 @@ public class Point implements Cloneable, PointImmutable { @Override public String toString() { - return new String( x + " / " + y ); + return x + " / " + y; } public final void set(int x, int y) { this.x = x; this.y = y; } diff --git a/src/nativewindow/classes/javax/media/nativewindow/util/Rectangle.java b/src/nativewindow/classes/javax/media/nativewindow/util/Rectangle.java index 8dc8f4562..d0d8bfb13 100644 --- a/src/nativewindow/classes/javax/media/nativewindow/util/Rectangle.java +++ b/src/nativewindow/classes/javax/media/nativewindow/util/Rectangle.java @@ -195,7 +195,7 @@ public class Rectangle implements Cloneable, RectangleImmutable { @Override public String toString() { - return new String("[ "+x+" / "+y+" "+width+" x "+height+" ]"); + return "[ "+x+" / "+y+" "+width+" x "+height+" ]"; } } diff --git a/src/nativewindow/classes/javax/media/nativewindow/util/SurfaceSize.java b/src/nativewindow/classes/javax/media/nativewindow/util/SurfaceSize.java index f1749dfa6..77619731f 100644 --- a/src/nativewindow/classes/javax/media/nativewindow/util/SurfaceSize.java +++ b/src/nativewindow/classes/javax/media/nativewindow/util/SurfaceSize.java @@ -58,7 +58,7 @@ public class SurfaceSize implements Comparable { @Override public final String toString() { - return new String("[ "+resolution+" x "+bitsPerPixel+" bpp ]"); + return "[ "+resolution+" x "+bitsPerPixel+" bpp ]"; } /** -- cgit v1.2.3