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); }