You are viewing a plain text version of this content. The canonical link for it is here.
Posted to scm@geronimo.apache.org by jd...@apache.org on 2006/09/29 20:54:34 UTC

svn commit: r451383 - /geronimo/server/trunk/maven-plugins/geronimo-maven-plugin/src/main/java/org/apache/geronimo/mavenplugins/geronimo/reporting/ReportingMojoSupport.java

Author: jdillon
Date: Fri Sep 29 11:54:33 2006
New Revision: 451383

URL: http://svn.apache.org/viewvc?view=rev&rev=451383
Log:
Fixed debug log NPE when no reporters are configured

Modified:
    geronimo/server/trunk/maven-plugins/geronimo-maven-plugin/src/main/java/org/apache/geronimo/mavenplugins/geronimo/reporting/ReportingMojoSupport.java

Modified: geronimo/server/trunk/maven-plugins/geronimo-maven-plugin/src/main/java/org/apache/geronimo/mavenplugins/geronimo/reporting/ReportingMojoSupport.java
URL: http://svn.apache.org/viewvc/geronimo/server/trunk/maven-plugins/geronimo-maven-plugin/src/main/java/org/apache/geronimo/mavenplugins/geronimo/reporting/ReportingMojoSupport.java?view=diff&rev=451383&r1=451382&r2=451383
==============================================================================
--- geronimo/server/trunk/maven-plugins/geronimo-maven-plugin/src/main/java/org/apache/geronimo/mavenplugins/geronimo/reporting/ReportingMojoSupport.java (original)
+++ geronimo/server/trunk/maven-plugins/geronimo-maven-plugin/src/main/java/org/apache/geronimo/mavenplugins/geronimo/reporting/ReportingMojoSupport.java Fri Sep 29 11:54:33 2006
@@ -77,7 +77,12 @@
         init();
 
         if (log.isDebugEnabled()) {
-            log.debug("Reporters: " + Arrays.asList(reporters));
+            if (reporters != null) {
+                log.debug("Reporters: " + Arrays.asList(reporters));
+            }
+            else {
+                log.debug("No reporters configured");
+            }
         }
         
         reportBegin();