svn commit: r478756 - /webservices/commons/trunk/modules/axiom/modules/axiom-api/src/main/java/org/apache/axiom/om/util/StAXUtils.java

[dims]

Author: dims
Date: Thu Nov 23 19:11:56 2006
New Revision: 478756

URL: http://svn.apache.org/viewvc?view=rev&rev=478756
Log:
prevent string concat by checking if log is enabled

Modified:
    
webservices/commons/trunk/modules/axiom/modules/axiom-api/src/main/java/org/apache/axiom/om/util/StAXUtils.java

Modified: 
webservices/commons/trunk/modules/axiom/modules/axiom-api/src/main/java/org/apache/axiom/om/util/StAXUtils.java
URL: 
http://svn.apache.org/viewvc/webservices/commons/trunk/modules/axiom/modules/axiom-api/src/main/java/org/apache/axiom/om/util/StAXUtils.java?view=diff&rev=478756&r1=478755&r2=478756
==============================================================================
--- 
webservices/commons/trunk/modules/axiom/modules/axiom-api/src/main/java/org/apache/axiom/om/util/StAXUtils.java
 (original)
+++ 
webservices/commons/trunk/modules/axiom/modules/axiom-api/src/main/java/org/apache/axiom/om/util/StAXUtils.java
 Thu Nov 23 19:11:56 2006
@@ -17,7 +17,8 @@
 
 
 public class StAXUtils {
-       private static interface ObjectCreator {
+
+    private static interface ObjectCreator {
                Object newObject();
        }
 
@@ -86,6 +87,7 @@
 
        
        private static Log log = LogFactory.getLog(StAXUtils.class);
+    private static boolean isDebugEnabled = log.isDebugEnabled();
           
 
        /**
@@ -111,7 +113,9 @@
         XMLInputFactory inputFactory = getXMLInputFactory();
         try {
                XMLStreamReader reader = inputFactory.createXMLStreamReader(in, 
encoding);
-               log.debug("XMLStreamReader is " + reader.getClass().getName());
+            if(isDebugEnabled) {
+                log.debug("XMLStreamReader is " + reader.getClass().getName());
+            }
             return reader;
         } finally {
             releaseXMLInputFactory(inputFactory);
@@ -123,7 +127,9 @@
         XMLInputFactory inputFactory = getXMLInputFactory();
         try {
             XMLStreamReader reader = inputFactory.createXMLStreamReader(in);
-            log.debug("XMLStreamReader is " + reader.getClass().getName());
+            if(isDebugEnabled) {
+                log.debug("XMLStreamReader is " + reader.getClass().getName());
+            }
             return reader;
         } finally {
             releaseXMLInputFactory(inputFactory);
@@ -135,7 +141,9 @@
         XMLInputFactory inputFactory = getXMLInputFactory();
         try {
             XMLStreamReader reader = inputFactory.createXMLStreamReader(in);
-            log.debug("XMLStreamReader is " + reader.getClass().getName());
+            if(isDebugEnabled) {
+                log.debug("XMLStreamReader is " + reader.getClass().getName());
+            }
             return reader;
         } finally {
             releaseXMLInputFactory(inputFactory);
@@ -165,7 +173,9 @@
         XMLOutputFactory outputFactory = getXMLOutputFactory();
         try {
             XMLStreamWriter writer = outputFactory.createXMLStreamWriter(out);
-            log.debug("XMLStreamWriter is " + writer.getClass().getName());
+            if(isDebugEnabled) {
+                log.debug("XMLStreamWriter is " + writer.getClass().getName());
+            }
             return writer;
         } finally {
             releaseXMLOutputFactory(outputFactory);
@@ -177,7 +187,9 @@
         XMLOutputFactory outputFactory = getXMLOutputFactory();
         try {
             XMLStreamWriter writer = outputFactory.createXMLStreamWriter(out);
-            log.debug("XMLStreamWriter is " + writer.getClass().getName());
+            if(isDebugEnabled) {
+                log.debug("XMLStreamWriter is " + writer.getClass().getName());
+            }
             return writer;
         } finally {
             releaseXMLOutputFactory(outputFactory);



---------------------------------------------------------------------
To unsubscribe, e-mail: [EMAIL PROTECTED]
For additional commands, e-mail: [EMAIL PROTECTED]