[jbossws-commits] JBossWS SVN: r10654 - in common/trunk/src/main/java/org/jboss: wsf/common and 1 other directories.
Author: alessio.soldano(a)jboss.com
Date: 2009-09-04 13:00:03 -0400 (Fri, 04 Sep 2009)
New Revision: 10654
Modified:
common/trunk/src/main/java/org/jboss/ws/core/utils/JBossWSEntityResolver.java
common/trunk/src/main/java/org/jboss/wsf/common/DOMUtils.java
common/trunk/src/main/java/org/jboss/wsf/common/injection/InjectionHelper.java
Log:
[JBWS-2750] skip debug logs when debug level is not enabled
Modified: common/trunk/src/main/java/org/jboss/ws/core/utils/JBossWSEntityResolver.java
===================================================================
---
common/trunk/src/main/java/org/jboss/ws/core/utils/JBossWSEntityResolver.java 2009-09-03
09:09:14 UTC (rev 10653)
+++
common/trunk/src/main/java/org/jboss/ws/core/utils/JBossWSEntityResolver.java 2009-09-04
17:00:03 UTC (rev 10654)
@@ -111,7 +111,10 @@
inputSource = resolveSystemIDAsURL(systemId, log.isTraceEnabled());
if (inputSource == null)
- log.debug("Cannot resolve entity: [pub=" + publicId +
",sysid=" + systemId + "]");
+ {
+ if (log.isDebugEnabled())
+ log.debug("Cannot resolve entity: [pub=" + publicId +
",sysid=" + systemId + "]");
+ }
return inputSource;
}
Modified: common/trunk/src/main/java/org/jboss/wsf/common/DOMUtils.java
===================================================================
--- common/trunk/src/main/java/org/jboss/wsf/common/DOMUtils.java 2009-09-03 09:09:14 UTC
(rev 10653)
+++ common/trunk/src/main/java/org/jboss/wsf/common/DOMUtils.java 2009-09-04 17:00:03 UTC
(rev 10654)
@@ -113,6 +113,7 @@
EntityResolver entityResolver = null;
ClassLoader loader = SecurityActions.getContextClassLoader();
+ boolean debugEnabled = log.isDebugEnabled();
for (String resolver : resolvers)
{
try
@@ -122,7 +123,8 @@
}
catch (Exception ex)
{
- log.debug("Cannot load: " + resolver);
+ if (debugEnabled)
+ log.debug("Cannot load: " + resolver);
}
}
Modified: common/trunk/src/main/java/org/jboss/wsf/common/injection/InjectionHelper.java
===================================================================
---
common/trunk/src/main/java/org/jboss/wsf/common/injection/InjectionHelper.java 2009-09-03
09:09:14 UTC (rev 10653)
+++
common/trunk/src/main/java/org/jboss/wsf/common/injection/InjectionHelper.java 2009-09-04
17:00:03 UTC (rev 10654)
@@ -169,7 +169,8 @@
if (methods.size() > 0)
{
final Method method = methods.iterator().next();
- LOG.debug("Calling @PostConstruct annotated method: " + method);
+ if (LOG.isDebugEnabled())
+ LOG.debug("Calling @PostConstruct annotated method: " + method);
try
{
invokeMethod(instance, method, null);
@@ -199,7 +200,8 @@
if (methods.size() > 0)
{
final Method method = methods.iterator().next();
- LOG.debug("Calling @PreDestroy annotated method: " + method);
+ if (LOG.isDebugEnabled())
+ LOG.debug("Calling @PreDestroy annotated method: " + method);
try
{
invokeMethod(instance, method, null);
@@ -393,7 +395,8 @@
private static void inject(final Object instance, final Method method, final String
jndiName, final Context ctx)
{
final Object value = lookup(jndiName, ctx);
- LOG.debug("Injecting method: " + method);
+ if (LOG.isDebugEnabled())
+ LOG.debug("Injecting method: " + method);
invokeMethod(instance, method, new Object[] {value});
}
@@ -408,7 +411,8 @@
private static void inject(final Object instance, final Field field, final String
jndiName, final Context ctx)
{
final Object value = lookup(jndiName, ctx);
- LOG.debug("Injecting field: " + field);
+ if (LOG.isDebugEnabled())
+ LOG.debug("Injecting field: " + field);
setField(instance, field, value);
}