You are viewing a plain text version of this content. The canonical link for it is here.
Posted to scm-commits@maven.apache.org by be...@apache.org on 2009/03/23 20:37:19 UTC

svn commit: r757515 - in /maven/scm/trunk/maven-scm-providers/maven-scm-provider-perforce/src: main/java/org/apache/maven/scm/provider/perforce/command/changelog/ test/java/org/apache/maven/scm/provider/perforce/command/changelog/

Author: bentmann
Date: Mon Mar 23 19:37:19 2009
New Revision: 757515

URL: http://svn.apache.org/viewvc?rev=757515&view=rev
Log:
o Stabilized ordering of change sets

Modified:
    maven/scm/trunk/maven-scm-providers/maven-scm-provider-perforce/src/main/java/org/apache/maven/scm/provider/perforce/command/changelog/PerforceChangeLogConsumer.java
    maven/scm/trunk/maven-scm-providers/maven-scm-provider-perforce/src/test/java/org/apache/maven/scm/provider/perforce/command/changelog/PerforceChangeLogConsumerTest.java

Modified: maven/scm/trunk/maven-scm-providers/maven-scm-provider-perforce/src/main/java/org/apache/maven/scm/provider/perforce/command/changelog/PerforceChangeLogConsumer.java
URL: http://svn.apache.org/viewvc/maven/scm/trunk/maven-scm-providers/maven-scm-provider-perforce/src/main/java/org/apache/maven/scm/provider/perforce/command/changelog/PerforceChangeLogConsumer.java?rev=757515&r1=757514&r2=757515&view=diff
==============================================================================
--- maven/scm/trunk/maven-scm-providers/maven-scm-provider-perforce/src/main/java/org/apache/maven/scm/provider/perforce/command/changelog/PerforceChangeLogConsumer.java (original)
+++ maven/scm/trunk/maven-scm-providers/maven-scm-provider-perforce/src/main/java/org/apache/maven/scm/provider/perforce/command/changelog/PerforceChangeLogConsumer.java Mon Mar 23 19:37:19 2009
@@ -29,8 +29,9 @@
 
 import java.util.ArrayList;
 import java.util.Date;
-import java.util.HashMap;
+import java.util.LinkedHashMap;
 import java.util.List;
+import java.util.Map;
 
 /**
  * @author <a href="mailto:evenisse@apache.org">Emmanuel Venisse</a>
@@ -142,7 +143,7 @@
     	// entries to have only one entry per changelist
     	
     	// Date > ChangeSet
-        HashMap groupedEntries = new HashMap();
+        Map groupedEntries = new LinkedHashMap();
         for ( int i = 0; i < entries.size(); i++ )
         {
             ChangeSet cs = (ChangeSet) entries.get( i );

Modified: maven/scm/trunk/maven-scm-providers/maven-scm-provider-perforce/src/test/java/org/apache/maven/scm/provider/perforce/command/changelog/PerforceChangeLogConsumerTest.java
URL: http://svn.apache.org/viewvc/maven/scm/trunk/maven-scm-providers/maven-scm-provider-perforce/src/test/java/org/apache/maven/scm/provider/perforce/command/changelog/PerforceChangeLogConsumerTest.java?rev=757515&r1=757514&r2=757515&view=diff
==============================================================================
--- maven/scm/trunk/maven-scm-providers/maven-scm-provider-perforce/src/test/java/org/apache/maven/scm/provider/perforce/command/changelog/PerforceChangeLogConsumerTest.java (original)
+++ maven/scm/trunk/maven-scm-providers/maven-scm-provider-perforce/src/test/java/org/apache/maven/scm/provider/perforce/command/changelog/PerforceChangeLogConsumerTest.java Mon Mar 23 19:37:19 2009
@@ -56,7 +56,7 @@
 
         ArrayList entries = new ArrayList( consumer.getModifications() );
         assertEquals( "Wrong number of entries returned", 7, entries.size() );
-        ChangeSet entry = (ChangeSet) entries.get(2);
+        ChangeSet entry = (ChangeSet) entries.get(0);
         assertEquals( "jim", entry.getAuthor() );
         assertEquals( "demo/demo.c", ( (ChangeFile) entry.getFiles().get( 0 ) ).getName() );
         assertEquals( "2003-10-01", entry.getDateFormatted() );