Patch minimizes the number of temporary objects which are created and
disposed of at the same time
2008-01-09 Stefan Huehner <stefan at huehner.org>
* gnu/classpath/jdwp/event/ExceptionEvent.java,
* gnu/java/awt/peer/gtk/GtkMainThread.java:
use Boolean.TRUE|FALSE instead of new Boolean(true|false)
* gnu/java/rmi/server/ConnectionRunnerPool.java,
* gnu/xml/aelfred2/XmlParser.java,
* gnu/xml/libxmlj/dom/GnomeXPathResult.java,
* gnu/xml/stream/XIncludeFilter.java:
use Integer|Double|Charater.toString(var) instead of
new Integer|Double|Character(var).toString()
Index: gnu/classpath/jdwp/event/ExceptionEvent.java
===================================================================
RCS file: /sources/classpath/classpath/gnu/classpath/jdwp/event/ExceptionEvent.java,v
retrieving revision 1.3
diff -u -r1.3 ExceptionEvent.java
--- gnu/classpath/jdwp/event/ExceptionEvent.java 27 Jul 2006 14:45:38 -0000 1.3
+++ gnu/classpath/jdwp/event/ExceptionEvent.java 9 Jan 2008 20:46:15 -0000
@@ -117,9 +117,9 @@
return _exception.getClass();
else if (type == EVENT_EXCEPTION_CAUGHT)
if (_catchLocation.getMethod() != null)
- return new Boolean(true);
+ return Boolean.TRUE;
else
- return new Boolean(false);
+ return Boolean.FALSE;
return null;
}
Index: gnu/java/awt/peer/gtk/GtkMainThread.java
===================================================================
RCS file: /sources/classpath/classpath/gnu/java/awt/peer/gtk/GtkMainThread.java,v
retrieving revision 1.18
diff -u -r1.18 GtkMainThread.java
--- gnu/java/awt/peer/gtk/GtkMainThread.java 12 Feb 2007 21:39:20 -0000 1.18
+++ gnu/java/awt/peer/gtk/GtkMainThread.java 9 Jan 2008 20:46:19 -0000
@@ -134,7 +134,7 @@
}
}
GtkGenericPeer.q()
- .postEvent(new NativeEventLoopRunningEvent(new Boolean(true)));
+ .postEvent(new NativeEventLoopRunningEvent(Boolean.TRUE));
}
}
}
@@ -161,7 +161,7 @@
}
}
GtkGenericPeer.q()
- .postEvent(new NativeEventLoopRunningEvent(new Boolean(false)));
+ .postEvent(new NativeEventLoopRunningEvent(Boolean.FALSE));
}
}
}
Index: gnu/java/rmi/server/ConnectionRunnerPool.java
===================================================================
RCS file: /sources/classpath/classpath/gnu/java/rmi/server/ConnectionRunnerPool.java,v
retrieving revision 1.5
diff -u -r1.5 ConnectionRunnerPool.java
--- gnu/java/rmi/server/ConnectionRunnerPool.java 2 Jul 2005 20:32:14 -0000 1.5
+++ gnu/java/rmi/server/ConnectionRunnerPool.java 9 Jan 2008 20:46:19 -0000
@@ -103,7 +103,7 @@
static {
ConnectionRunner[] pools = new ConnectionRunner[size];
for(int i = 0; i < pools.length; i++){
- pools[i] = new ConnectionRunner(group, new Integer(i).toString());
+ pools[i] = new ConnectionRunner(group, Integer.toString(i));
pools[i].setContextClassLoader(Thread.currentThread().getContextClassLoader());
pools[i].start();
}
@@ -123,7 +123,7 @@
if(freelist.size() == 0){
if(size < max_size){
++size;
- ConnectionRunner a = new ConnectionRunner(group, new Integer(size).toString());
+ ConnectionRunner a = new ConnectionRunner(group, Integer.toString(size));
a.start();
freelist.add(a);
}else
Index: gnu/xml/aelfred2/XmlParser.java
===================================================================
RCS file: /sources/classpath/classpath/gnu/xml/aelfred2/XmlParser.java,v
retrieving revision 1.10
diff -u -r1.10 XmlParser.java
--- gnu/xml/aelfred2/XmlParser.java 21 Sep 2007 18:05:21 -0000 1.10
+++ gnu/xml/aelfred2/XmlParser.java 9 Jan 2008 20:46:26 -0000
@@ -542,7 +542,7 @@
private void error(String message, char textFound, String textExpected)
throws SAXException
{
- error(message, new Character(textFound).toString(), textExpected);
+ error(message, Character.toString(textFound), textExpected);
}
/**
@@ -2281,7 +2281,7 @@
{
// too big for surrogate
error("character reference " + value + " is too large for UTF-16",
- new Integer(value).toString(), null);
+ Integer.toString(value), null);
}
}
@@ -2375,7 +2375,7 @@
{
// too big for surrogate
error("character reference " + value + " is too large for UTF-16",
- new Integer(value).toString(), null);
+ Integer.toString(value), null);
}
if (doFlush)
{
@@ -3294,7 +3294,7 @@
catch (EOFException e)
{
error("end of input while looking for delimiter (started on line "
- + startLine + ')', null, new Character(delim).toString());
+ + startLine + ')', null, Character.toString(delim));
}
inLiteral = false;
expandPE = saved;
@@ -3598,7 +3598,7 @@
if (c != delim)
{
- error("required character", c, new Character(delim).toString());
+ error("required character", c, Character.toString(delim));
}
}
@@ -4298,7 +4298,7 @@
}
else
{
- pushString(null, new Character(c).toString());
+ pushString(null, Character.toString(c));
}
}
Index: gnu/xml/libxmlj/dom/GnomeXPathResult.java
===================================================================
RCS file: /sources/classpath/classpath/gnu/xml/libxmlj/dom/GnomeXPathResult.java,v
retrieving revision 1.2
diff -u -r1.2 GnomeXPathResult.java
--- gnu/xml/libxmlj/dom/GnomeXPathResult.java 2 Jul 2005 20:32:16 -0000 1.2
+++ gnu/xml/libxmlj/dom/GnomeXPathResult.java 9 Jan 2008 20:46:27 -0000
@@ -101,7 +101,7 @@
case STRING_TYPE:
return getStringValue ();
case NUMBER_TYPE:
- return new Double (getNumberValue ()).toString ();
+ return Double.toString (getNumberValue ());
case BOOLEAN_TYPE:
return Boolean.valueOf (getBooleanValue ()).toString ();
case UNORDERED_NODE_SNAPSHOT_TYPE:
Index: gnu/xml/stream/XIncludeFilter.java
===================================================================
RCS file: /sources/classpath/classpath/gnu/xml/stream/XIncludeFilter.java,v
retrieving revision 1.9
diff -u -r1.9 XIncludeFilter.java
--- gnu/xml/stream/XIncludeFilter.java 18 Sep 2007 21:52:32 -0000 1.9
+++ gnu/xml/stream/XIncludeFilter.java 9 Jan 2008 20:46:28 -0000
@@ -499,7 +499,7 @@
return event;
case XPathResult.NUMBER_TYPE:
double nval = result.getNumberValue();
- String ntext = new Double(nval).toString();
+ String ntext = Double.toString(nval);
buf = ntext.toCharArray();
len = buf.length;
result = null;