Author: jlmonteiro
Date: Thu May 15 14:48:58 2014
New Revision: 1594936

URL: http://svn.apache.org/r1594936
Log:
Make the test a bit clever

Modified:
    
tomee/tomee/trunk/container/openejb-core/src/main/java/org/apache/openejb/config/AutoConfig.java

Modified: 
tomee/tomee/trunk/container/openejb-core/src/main/java/org/apache/openejb/config/AutoConfig.java
URL: 
http://svn.apache.org/viewvc/tomee/tomee/trunk/container/openejb-core/src/main/java/org/apache/openejb/config/AutoConfig.java?rev=1594936&r1=1594935&r2=1594936&view=diff
==============================================================================
--- 
tomee/tomee/trunk/container/openejb-core/src/main/java/org/apache/openejb/config/AutoConfig.java
 (original)
+++ 
tomee/tomee/trunk/container/openejb-core/src/main/java/org/apache/openejb/config/AutoConfig.java
 Thu May 15 14:48:58 2014
@@ -2169,30 +2169,23 @@ public class AutoConfig implements Dynam
         private final Map<String, List<String>> containerIdsByType = new 
TreeMap<String, List<String>>();
 
         public void dump() {
+            if (!logger.isDebugEnabled()) return;
             for (final String s : resourceAdapterIds) {
-                if (logger.isDebugEnabled()) {
-                    logger.debug(appId + " module contains resource adapter 
id: " + s);
-                }
+                logger.debug(appId + " module contains resource adapter id: " 
+ s);
             }
             for (final String s : resourceIdsByType.keySet()) {
                 for (final String value : resourceIdsByType.get(s)) {
-                    if (logger.isDebugEnabled()) {
-                        logger.debug(appId + " module contains resource type: 
" + s + " --> " + value);
-                    }
+                    logger.debug(appId + " module contains resource type: " + 
s + " --> " + value);
                 }
             }
             for (final String s : resourceEnvIdsByType.keySet()) {
                 for (final String value : resourceEnvIdsByType.get(s)) {
-                    if (logger.isDebugEnabled()) {
-                        logger.debug(appId + " module contains resource env 
type: " + s + " --> " + value);
-                    }
+                    logger.debug(appId + " module contains resource env type: 
" + s + " --> " + value);
                 }
             }
             for (final String s : containerIdsByType.keySet()) {
                 for (final String value : containerIdsByType.get(s)) {
-                    if (logger.isDebugEnabled()) {
-                        logger.debug(appId + " module contains container type: 
" + s + " --> " + value);
-                    }
+                    logger.debug(appId + " module contains container type: " + 
s + " --> " + value);
                 }
             }
         }


Reply via email to