Author: alessio.soldano(a)jboss.com Date: 2009-09-04 13:00:53 -0400 (Fri, 04 Sep 2009) New Revision: 10655 Modified: framework/trunk/src/main/java/org/jboss/wsf/framework/DefaultSPIProvider.java framework/trunk/src/main/java/org/jboss/wsf/framework/deployment/ArchiveDeploymentImpl.java framework/trunk/src/main/java/org/jboss/wsf/framework/deployment/DefaultDeploymentAspectManagerFactory.java framework/trunk/src/main/java/org/jboss/wsf/framework/deployment/DefaultLifecycleHandler.java framework/trunk/src/main/java/org/jboss/wsf/framework/deployment/DeploymentAspectManagerImpl.java Log: [JBWS-2750] skip debug logs when debug level is not enabled Modified: framework/trunk/src/main/java/org/jboss/wsf/framework/DefaultSPIProvider.java =================================================================== --- framework/trunk/src/main/java/org/jboss/wsf/framework/DefaultSPIProvider.java 2009-09-04 17:00:03 UTC (rev 10654) +++ framework/trunk/src/main/java/org/jboss/wsf/framework/DefaultSPIProvider.java 2009-09-04 17:00:53 UTC (rev 10655) @@ -67,7 +67,9 @@ */ public <T> T getSPI(Class<T> spiType) { - log.debug("provide SPI '" + spiType + "'"); + boolean debugEnabled = log.isDebugEnabled(); + if (debugEnabled) + log.debug("provide SPI '" + spiType + "'"); T returnType = null; @@ -149,7 +151,8 @@ throw new WSFException("Failed to provide SPI '" + spiType + "'"); // help debugging - log.debug(spiType + " Implementation: " + returnType); + if (debugEnabled) + log.debug(spiType + " Implementation: " + returnType); return returnType; } Modified: framework/trunk/src/main/java/org/jboss/wsf/framework/deployment/ArchiveDeploymentImpl.java =================================================================== --- framework/trunk/src/main/java/org/jboss/wsf/framework/deployment/ArchiveDeploymentImpl.java 2009-09-04 17:00:03 UTC (rev 10654) +++ framework/trunk/src/main/java/org/jboss/wsf/framework/deployment/ArchiveDeploymentImpl.java 2009-09-04 17:00:53 UTC (rev 10655) @@ -114,7 +114,8 @@ } else { - log.debug("Cannot get " + resourcePath + " from root file, trying with additional metadata files", e); + if (log.isDebugEnabled()) + log.debug("Cannot get " + resourcePath + " from root file, trying with additional metadata files", e); } } @@ -142,7 +143,8 @@ } catch (IOException e) { - log.debug("Cannot get " + resourcePath + " from " + uvf, e); + if (log.isDebugEnabled()) + log.debug("Cannot get " + resourcePath + " from " + uvf, e); } } } Modified: framework/trunk/src/main/java/org/jboss/wsf/framework/deployment/DefaultDeploymentAspectManagerFactory.java =================================================================== --- framework/trunk/src/main/java/org/jboss/wsf/framework/deployment/DefaultDeploymentAspectManagerFactory.java 2009-09-04 17:00:03 UTC (rev 10654) +++ framework/trunk/src/main/java/org/jboss/wsf/framework/deployment/DefaultDeploymentAspectManagerFactory.java 2009-09-04 17:00:53 UTC (rev 10655) @@ -57,7 +57,8 @@ public DeploymentAspectManager getDeploymentAspectManager(String beanName) { - log.debug("DeploymentAspectManager for: " + beanName); + if (log.isDebugEnabled()) + log.debug("DeploymentAspectManager for: " + beanName); return new KernelAwareSPIFactory().getKernelProvidedSPI(beanName, DeploymentAspectManager.class); } } Modified: framework/trunk/src/main/java/org/jboss/wsf/framework/deployment/DefaultLifecycleHandler.java =================================================================== --- framework/trunk/src/main/java/org/jboss/wsf/framework/deployment/DefaultLifecycleHandler.java 2009-09-04 17:00:03 UTC (rev 10654) +++ framework/trunk/src/main/java/org/jboss/wsf/framework/deployment/DefaultLifecycleHandler.java 2009-09-04 17:00:53 UTC (rev 10655) @@ -40,7 +40,8 @@ public void start(Endpoint ep) { - log.debug("Start: " + ep.getName()); + if (log.isDebugEnabled()) + log.debug("Start: " + ep.getName()); EndpointState state = ep.getState(); if (state != EndpointState.UNDEFINED) @@ -63,7 +64,8 @@ public void stop(Endpoint ep) { - log.debug("Stop: " + ep.getName()); + if (log.isDebugEnabled()) + log.debug("Stop: " + ep.getName()); EndpointState state = ep.getState(); if (state != EndpointState.STARTED) Modified: framework/trunk/src/main/java/org/jboss/wsf/framework/deployment/DeploymentAspectManagerImpl.java =================================================================== --- framework/trunk/src/main/java/org/jboss/wsf/framework/deployment/DeploymentAspectManagerImpl.java 2009-09-04 17:00:03 UTC (rev 10654) +++ framework/trunk/src/main/java/org/jboss/wsf/framework/deployment/DeploymentAspectManagerImpl.java 2009-09-04 17:00:53 UTC (rev 10655) @@ -78,12 +78,15 @@ depAspects.clear(); depAspects.addAll(aspects); - // Debug the aspect list - StringBuilder builder = new StringBuilder("setDeploymentAspects on " + name); - for (DeploymentAspect aspect : aspects) - builder.append("\n " + aspect.getClass().getName() + " provides: " + aspect.getProvidesAsSet() + " requires: " + aspect.getRequiresAsSet()); - - log.debug(builder); + if (log.isDebugEnabled()) + { + // Debug the aspect list + StringBuilder builder = new StringBuilder("setDeploymentAspects on " + name); + for (DeploymentAspect aspect : aspects) + builder.append("\n " + aspect.getClass().getName() + " provides: " + aspect.getProvidesAsSet() + " requires: " + aspect.getRequiresAsSet()); + + log.debug(builder); + } } /** @@ -165,7 +168,8 @@ { String name = aspect.getClass().getName(); name = name.substring(name.lastIndexOf(".") + 1); - log.debug("[" + this.name + "]" + name + ":" + method); + if (log.isDebugEnabled()) + log.debug("[" + this.name + "]" + name + ":" + method); } }