You are viewing a plain text version of this content. The canonical link for it is here.

Posted to commits@maven.apache.org by de...@apache.org on 2006/10/14 15:38:03 UTC

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

Author: dennisl
Date: Sat Oct 14 06:38:02 2006
New Revision: 463930

URL: http://svn.apache.org/viewvc?view=rev&rev=463930
Log:
[MCHANGELOG-50] Changes are not sorted by date properly
Submitted by: Fabrice Bellingard

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=463930&r1=463929&r2=463930
==============================================================================
--- 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 Sat Oct 14 06:38:02 2006
@@ -916,7 +916,8 @@
 
         initReportUrls();
 
-        Collections.sort( new ArrayList( entries ), new Comparator()
+        List sortedEntries = new ArrayList(entries);
+        Collections.sort( sortedEntries, new Comparator()
         {
             public int compare( Object arg0, Object arg1 )
             {
@@ -926,7 +927,7 @@
             }
         } );
 
-        for ( Iterator i = entries.iterator(); i.hasNext(); )
+        for ( Iterator i = sortedEntries.iterator(); i.hasNext(); )
         {
             ChangeSet entry = (ChangeSet) i.next();