You are viewing a plain text version of this content. The canonical link for it is here.
Posted to server-dev@james.apache.org by rd...@apache.org on 2011/05/10 15:06:00 UTC

svn commit: r1101450 - /james/mailet/maven-mailetdocs-plugin/trunk/src/main/java/org/apache/james/mailet/DefaultDescriptorsExtractor.java

Author: rdonkin
Date: Tue May 10 13:05:59 2011
New Revision: 1101450

URL: http://svn.apache.org/viewvc?rev=1101450&view=rev
Log:
MAILETDOCS-3 Factor out interface logging, and make it conditional.

Modified:
    james/mailet/maven-mailetdocs-plugin/trunk/src/main/java/org/apache/james/mailet/DefaultDescriptorsExtractor.java

Modified: james/mailet/maven-mailetdocs-plugin/trunk/src/main/java/org/apache/james/mailet/DefaultDescriptorsExtractor.java
URL: http://svn.apache.org/viewvc/james/mailet/maven-mailetdocs-plugin/trunk/src/main/java/org/apache/james/mailet/DefaultDescriptorsExtractor.java?rev=1101450&r1=1101449&r2=1101450&view=diff
==============================================================================
--- james/mailet/maven-mailetdocs-plugin/trunk/src/main/java/org/apache/james/mailet/DefaultDescriptorsExtractor.java (original)
+++ james/mailet/maven-mailetdocs-plugin/trunk/src/main/java/org/apache/james/mailet/DefaultDescriptorsExtractor.java Tue May 10 13:05:59 2011
@@ -97,12 +97,8 @@ public class DefaultDescriptorsExtractor
                 final MailetMatcherDescriptor descriptor = describeMatcher(log,
                         nextClass, nameOfNextClass, klass);
                 res.add(descriptor);
-            } else if (allInterfaces.size() > 0) {
-                for (int k = 0; k < allInterfaces.size(); k++) {
-                    log.debug("I: " + ((Class<?>) allInterfaces.get(k)).getName());
-                }
             } else {
-                log.debug("No interfaces for " + klass.getName());
+                logInterfaces(log, klass, allInterfaces);
             }
 
         } catch (ClassNotFoundException e) {
@@ -117,6 +113,21 @@ public class DefaultDescriptorsExtractor
     }
 
 
+    private void logInterfaces(Log log, final Class<?> klass,
+            final List<Class<?>> allInterfaces) {
+        if (log.isDebugEnabled()) {
+            if (allInterfaces.size() > 0) {
+                log.debug("Interfaces for " + klass.getName());
+                for (int k = 0; k < allInterfaces.size(); k++) {
+                    log.debug("Interface: " + ((Class<?>) allInterfaces.get(k)).getName());
+                }
+            } else {
+                log.debug("No interfaces for " + klass.getName());
+            }
+        }
+    }
+
+
     private MailetMatcherDescriptor describeMatcher(Log log,
             final JavaClass nextClass, final String nameOfNextClass,
             final Class<?> klass) {



---------------------------------------------------------------------
To unsubscribe, e-mail: server-dev-unsubscribe@james.apache.org
For additional commands, e-mail: server-dev-help@james.apache.org