svn commit: r1225277 - in /xalan/java/trunk/src/org/apache/xpath/jaxp: XPathExpressionImpl.java XPathFactoryImpl.java

Author: mrglavas
Date: Wed Dec 28 18:50:56 2011
New Revision: 1225277

URL: http://svn.apache.org/viewvc?rev=1225277&view=rev
Log:
Reducing object allocation. Use Boolean constants instead.

Modified:
    xalan/java/trunk/src/org/apache/xpath/jaxp/XPathExpressionImpl.java
    xalan/java/trunk/src/org/apache/xpath/jaxp/XPathFactoryImpl.java

Modified: xalan/java/trunk/src/org/apache/xpath/jaxp/XPathExpressionImpl.java
URL: http://svn.apache.org/viewvc/xalan/java/trunk/src/org/apache/xpath/jaxp/XPathExpressionImpl.java?rev=1225277&r1=1225276&r2=1225277&view=diff
==============================================================================
--- xalan/java/trunk/src/org/apache/xpath/jaxp/XPathExpressionImpl.java (original)
+++ xalan/java/trunk/src/org/apache/xpath/jaxp/XPathExpressionImpl.java Wed Dec 28 18:50:56 2011
@@ -342,7 +342,7 @@ public class XPathExpressionImpl  implem
         }
         // XPathConstants.BOOLEAN
         if ( returnType.equals( XPathConstants.BOOLEAN ) ) {
-            return new Boolean( resultObject.bool());
+            return resultObject.bool() ? Boolean.TRUE : Boolean.FALSE;
         }
         // XPathConstants.NODESET ---ORdered, UNOrdered???
         if ( returnType.equals( XPathConstants.NODESET ) ) {

Modified: xalan/java/trunk/src/org/apache/xpath/jaxp/XPathFactoryImpl.java
URL: http://svn.apache.org/viewvc/xalan/java/trunk/src/org/apache/xpath/jaxp/XPathFactoryImpl.java?rev=1225277&r1=1225276&r2=1225277&view=diff
==============================================================================
--- xalan/java/trunk/src/org/apache/xpath/jaxp/XPathFactoryImpl.java (original)
+++ xalan/java/trunk/src/org/apache/xpath/jaxp/XPathFactoryImpl.java Wed Dec 28 18:50:56 2011
@@ -34,7 +34,7 @@ import javax.xml.xpath.XPathVariableReso
  * @version $Revision$
  * @author  Ramesh Mandava
  */
-public  class XPathFactoryImpl extends XPathFactory {
+public class XPathFactoryImpl extends XPathFactory {
 	
 	/**
 	 * <p>Name of class as a constant to use for debugging.</p>
@@ -140,7 +140,7 @@ public  class XPathFactoryImpl extends X
             if (name == null) {
                 String fmsg = XSLMessages.createXPATHMessage(
                         XPATHErrorResources.ER_FEATURE_NAME_NULL,
-                        new Object[] { CLASS_NAME, new Boolean( value) } );
+                        new Object[] { CLASS_NAME, value ? Boolean.TRUE : Boolean.FALSE } );
                 throw new NullPointerException( fmsg );
              }
 		
@@ -156,7 +156,7 @@ public  class XPathFactoryImpl extends X
             // unknown feature
             String fmsg = XSLMessages.createXPATHMessage(
                     XPATHErrorResources.ER_FEATURE_UNKNOWN,
-                    new Object[] { name, CLASS_NAME, new Boolean(value) } );
+                    new Object[] { name, CLASS_NAME, value ? Boolean.TRUE : Boolean.FALSE } );
             throw new XPathFactoryConfigurationException( fmsg );
 	}