svn commit: r557720 - /maven/plugins/trunk/maven-changelog-plugin/src/main/java/org/apache/maven/plugin/changelog/ChangeLogReport.java

[de...@apache.org]

Author: dennisl
Date: Thu Jul 19 11:22:23 2007
New Revision: 557720

URL: http://svn.apache.org/viewvc?view=rev&rev=557720
Log:
o Fix errors reported by Checkstyle.

Modified:
    maven/plugins/trunk/maven-changelog-plugin/src/main/java/org/apache/maven/plugin/changelog/ChangeLogReport.java

Modified: maven/plugins/trunk/maven-changelog-plugin/src/main/java/org/apache/maven/plugin/changelog/ChangeLogReport.java
URL: http://svn.apache.org/viewvc/maven/plugins/trunk/maven-changelog-plugin/src/main/java/org/apache/maven/plugin/changelog/ChangeLogReport.java?view=diff&rev=557720&r1=557719&r2=557720
==============================================================================
--- maven/plugins/trunk/maven-changelog-plugin/src/main/java/org/apache/maven/plugin/changelog/ChangeLogReport.java (original)
+++ maven/plugins/trunk/maven-changelog-plugin/src/main/java/org/apache/maven/plugin/changelog/ChangeLogReport.java Thu Jul 19 11:22:23 2007
@@ -84,6 +84,11 @@
     private static final String FILE_TOKEN = "%FILE%";
 
     /**
+     * The number of days to use as a range, when this is not specified.
+     */
+    private static final int DEFAULT_RANGE = 30;
+
+    /**
      * Used to specify the format to use for the dates in the headings of the
      * report.
      *
@@ -465,7 +470,8 @@
             }
             else if ( "tag".equals( type ) )
             {
-                if( repository.getProvider().equals( "svn" ) ) {
+                if ( repository.getProvider().equals( "svn" ) )
+                {
                     throw new MavenReportException( "The type '" + type + "' isn't supported for svn." );
                 }
 
@@ -751,7 +757,7 @@
         {
             if ( range == -1 )
             {
-                range = 30;
+                range = DEFAULT_RANGE;
             }
         }
         else if ( "date".equals( type ) )
@@ -759,7 +765,8 @@
             if ( dates == null )
             {
                 throw new MavenReportException(
-                    "The dates parameter is required when type=\"date\". The value should be the absolute date for the start of the log." );
+                    "The dates parameter is required when type=\"date\"."
+                    + " The value should be the absolute date for the start of the log." );
             }
         }
         else if ( "tag".equals( type ) )
@@ -927,7 +934,8 @@
         else
         {
             sink.text( bundle.getString( "report.SetRangeBetween" ) );
-            sink.text( " " + headingDateFormater.format( set.getStartDate() ) + " " + bundle.getString( "report.And" ) + " "
+            sink.text( " " + headingDateFormater.format( set.getStartDate() )
+                + " " + bundle.getString( "report.And" ) + " "
                 + headingDateFormater.format( set.getEndDate() ) );
         }
         sink.sectionTitle2_();
@@ -1302,10 +1310,13 @@
 
         String head;
         String tail;
-        if (pos < 0) {
+        if ( pos < 0 )
+        {
             head = "";
             tail = name;
-        } else {
+        }
+        else
+        {
             head = name.substring( 0, pos ) + "/";
             tail = name.substring( pos + 1 );
         }