maven-commits mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From ken...@apache.org
Subject svn commit: r399728 - /maven/components/trunk/maven-project/src/main/java/org/apache/maven/project/ModelUtils.java
Date Thu, 04 May 2006 15:22:13 GMT
Author: kenney
Date: Thu May  4 08:22:10 2006
New Revision: 399728

URL: http://svn.apache.org/viewcvs?rev=399728&view=rev
Log:
added a comment on strange code

Modified:
    maven/components/trunk/maven-project/src/main/java/org/apache/maven/project/ModelUtils.java

Modified: maven/components/trunk/maven-project/src/main/java/org/apache/maven/project/ModelUtils.java
URL: http://svn.apache.org/viewcvs/maven/components/trunk/maven-project/src/main/java/org/apache/maven/project/ModelUtils.java?rev=399728&r1=399727&r2=399728&view=diff
==============================================================================
--- maven/components/trunk/maven-project/src/main/java/org/apache/maven/project/ModelUtils.java
(original)
+++ maven/components/trunk/maven-project/src/main/java/org/apache/maven/project/ModelUtils.java
Thu May  4 08:22:10 2006
@@ -106,6 +106,9 @@
                 }
             }
 
+	    // FIXME: not sure what's intended here, but this entire
+	    // loop can be replaced by 'mergedPlugins.addAll( childPlugins.values() );
+	    // since assembledPlugins is never updated and remains empty.
             for ( Iterator it = childPlugins.values().iterator(); it.hasNext(); )
             {
                 Plugin childPlugin = (Plugin) it.next();



Mime
View raw message