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 2011/05/01 16:35:35 UTC

svn commit: r1098336 - /maven/plugins/trunk/maven-changes-plugin/src/main/java/org/apache/maven/plugin/changes/ChangesReportGenerator.java

Author: dennisl
Date: Sun May  1 14:35:34 2011
New Revision: 1098336

URL: http://svn.apache.org/viewvc?rev=1098336&view=rev
Log:
[MCHANGES-210] Update to Java 5

- Start using generics in ChangesReportGenerator

Modified:
    maven/plugins/trunk/maven-changes-plugin/src/main/java/org/apache/maven/plugin/changes/ChangesReportGenerator.java

Modified: maven/plugins/trunk/maven-changes-plugin/src/main/java/org/apache/maven/plugin/changes/ChangesReportGenerator.java
URL: http://svn.apache.org/viewvc/maven/plugins/trunk/maven-changes-plugin/src/main/java/org/apache/maven/plugin/changes/ChangesReportGenerator.java?rev=1098336&r1=1098335&r2=1098336&view=diff
==============================================================================
--- maven/plugins/trunk/maven-changes-plugin/src/main/java/org/apache/maven/plugin/changes/ChangesReportGenerator.java (original)
+++ maven/plugins/trunk/maven-changes-plugin/src/main/java/org/apache/maven/plugin/changes/ChangesReportGenerator.java Sun May  1 14:35:34 2011
@@ -333,7 +333,7 @@ public class ChangesReportGenerator exte
     {
 
         // Create a Map with key : dueTo name, value : dueTo email
-        Map namesEmailMap = new LinkedHashMap();
+        Map<String,String> namesEmailMap = new LinkedHashMap<String,String>();
 
         // Only add the dueTo specified as attributes, if it has either a dueTo or a dueToEmail
         if ( StringUtils.isNotEmpty( action.getDueTo() ) || StringUtils.isNotEmpty( action.getDueToEmail() ) )
@@ -354,10 +354,9 @@ public class ChangesReportGenerator exte
 
         sink.text( " " + bundle.getString( "report.changes.text.thanx" ) + " " );
         int i = 0;
-        for ( Iterator iterator = namesEmailMap.keySet().iterator(); iterator.hasNext(); )
+        for ( String currentDueTo : namesEmailMap.keySet() )
         {
-            String currentDueTo = (String) iterator.next();
-            String currentDueToEmail = (String) namesEmailMap.get( currentDueTo );
+            String currentDueToEmail = namesEmailMap.get( currentDueTo );
             i++;
 
             if ( StringUtils.isNotEmpty( currentDueToEmail ) )