Author: dims
Date: Mon Jan 1 10:45:56 2007
New Revision: 491612
URL: http://svn.apache.org/viewvc?view=rev&rev=491612
Log:
ensure isDebugEnabled only once
Modified:
webservices/commons/trunk/modules/axiom/modules/axiom-api/src/main/java/org/apache/axiom/om/impl/builder/StAXOMBuilder.java
webservices/commons/trunk/modules/axiom/modules/axiom-impl/src/main/java/org/apache/axiom/om/impl/llom/OMSourcedElementImpl.java
Modified:
webservices/commons/trunk/modules/axiom/modules/axiom-api/src/main/java/org/apache/axiom/om/impl/builder/StAXOMBuilder.java
URL:
http://svn.apache.org/viewvc/webservices/commons/trunk/modules/axiom/modules/axiom-api/src/main/java/org/apache/axiom/om/impl/builder/StAXOMBuilder.java?view=diff&rev=491612&r1=491611&r2=491612
==============================================================================
---
webservices/commons/trunk/modules/axiom/modules/axiom-api/src/main/java/org/apache/axiom/om/impl/builder/StAXOMBuilder.java
(original)
+++
webservices/commons/trunk/modules/axiom/modules/axiom-api/src/main/java/org/apache/axiom/om/impl/builder/StAXOMBuilder.java
Mon Jan 1 10:45:56 2007
@@ -41,7 +41,7 @@
*/
private static final Log log = LogFactory.getLog(StAXOMBuilder.class);
- private boolean doDebug = false;
+ private boolean doDebug = log.isDebugEnabled();
private static int nsCount = 0;
/**
@@ -53,7 +53,6 @@
public StAXOMBuilder(OMFactory ombuilderFactory, XMLStreamReader parser) {
super(ombuilderFactory, parser);
document = ombuilderFactory.createOMDocument(this);
- doDebug = log.isDebugEnabled();
}
/**
@@ -67,7 +66,6 @@
this(factory, parser);
lastNode = element;
document.setOMDocumentElement(element);
- doDebug = log.isDebugEnabled();
populateOMElement(element);
}
@@ -78,7 +76,6 @@
*/
public StAXOMBuilder(String filePath) throws XMLStreamException,
FileNotFoundException {
this(StAXUtils.createXMLStreamReader(new FileInputStream(filePath)));
- doDebug = log.isDebugEnabled();
}
/**
@@ -87,7 +84,6 @@
*/
public StAXOMBuilder(InputStream inStream) throws XMLStreamException {
this(StAXUtils.createXMLStreamReader(inStream));
- doDebug = log.isDebugEnabled();
}
/**
@@ -99,7 +95,6 @@
super(parser);
omfactory = OMAbstractFactory.getOMFactory();
document = omfactory.createOMDocument(this);
- doDebug = log.isDebugEnabled();
}
/**
Modified:
webservices/commons/trunk/modules/axiom/modules/axiom-impl/src/main/java/org/apache/axiom/om/impl/llom/OMSourcedElementImpl.java
URL:
http://svn.apache.org/viewvc/webservices/commons/trunk/modules/axiom/modules/axiom-impl/src/main/java/org/apache/axiom/om/impl/llom/OMSourcedElementImpl.java?view=diff&rev=491612&r1=491611&r2=491612
==============================================================================
---
webservices/commons/trunk/modules/axiom/modules/axiom-impl/src/main/java/org/apache/axiom/om/impl/llom/OMSourcedElementImpl.java
(original)
+++
webservices/commons/trunk/modules/axiom/modules/axiom-impl/src/main/java/org/apache/axiom/om/impl/llom/OMSourcedElementImpl.java
Mon Jan 1 10:45:56 2007
@@ -61,7 +61,8 @@
private boolean isParserSet;
private static Log log = LogFactory.getLog(OMSourcedElementImpl.class);
-
+ private static final boolean isDebugEnabled = log.isDebugEnabled();
+
/**
* Constructor.
*
@@ -132,7 +133,7 @@
private void forceExpand() {
if (!isParserSet) {
- if (log.isDebugEnabled()) {
+ if (isDebugEnabled) {
log.debug("forceExpand: expanding element " +
getPrintableName());
}
@@ -340,7 +341,7 @@
* @see org.apache.axiom.om.OMElement#getXMLStreamReader()
*/
public XMLStreamReader getXMLStreamReader() {
- if (log.isDebugEnabled()) {
+ if (isDebugEnabled) {
log.debug("getting XMLStreamReader for " + getPrintableName());
}
if (isParserSet) {
@@ -354,7 +355,7 @@
* @see org.apache.axiom.om.OMElement#getXMLStreamReaderWithoutCaching()
*/
public XMLStreamReader getXMLStreamReaderWithoutCaching() {
- if (log.isDebugEnabled()) {
+ if (isDebugEnabled) {
log.debug("getting XMLStreamReader without caching for " +
getPrintableName());
}
@@ -543,7 +544,7 @@
* @see
org.apache.axiom.om.OMNode#internalSerializeAndConsume(javax.xml.stream.XMLStreamWriter)
*/
public void internalSerializeAndConsume(XMLStreamWriter writer) throws
XMLStreamException {
- if (log.isDebugEnabled()) {
+ if (isDebugEnabled) {
log.debug("serialize " + getPrintableName() + " to
XMLStreamWriter");
}
if (isDataSourceConsumed()) {
@@ -599,7 +600,7 @@
* @see
org.apache.axiom.om.OMNode#serializeAndConsume(java.io.OutputStream)
*/
public void serializeAndConsume(OutputStream output) throws
XMLStreamException {
- if (log.isDebugEnabled()) {
+ if (isDebugEnabled) {
log.debug("serialize " + getPrintableName() + " to output stream");
}
if (isDataSourceConsumed()) {
@@ -613,7 +614,7 @@
* @see org.apache.axiom.om.OMNode#serializeAndConsume(java.io.Writer)
*/
public void serializeAndConsume(Writer writer) throws XMLStreamException {
- if (log.isDebugEnabled()) {
+ if (isDebugEnabled) {
log.debug("serialize " + getPrintableName() + " to writer");
}
if (isDataSourceConsumed()) {
@@ -627,7 +628,7 @@
* @see
org.apache.axiom.om.OMNode#serializeAndConsume(java.io.OutputStream,
org.apache.axiom.om.OMOutputFormat)
*/
public void serializeAndConsume(OutputStream output, OMOutputFormat
format) throws XMLStreamException {
- if (log.isDebugEnabled()) {
+ if (isDebugEnabled) {
log.debug("serialize formatted " + getPrintableName() +
" to output stream");
}
@@ -642,7 +643,7 @@
* @see org.apache.axiom.om.OMNode#serializeAndConsume(java.io.Writer,
org.apache.axiom.om.OMOutputFormat)
*/
public void serializeAndConsume(Writer writer, OMOutputFormat format)
throws XMLStreamException {
- if (log.isDebugEnabled()) {
+ if (isDebugEnabled) {
log.debug("serialize formatted " + getPrintableName() +
" to writer");
}
---------------------------------------------------------------------
To unsubscribe, e-mail: [EMAIL PROTECTED]
For additional commands, e-mail: [EMAIL PROTECTED]
