You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@maven.apache.org by ke...@apache.org on 2006/05/04 17:22:13 UTC
svn commit: r399728 -
/maven/components/trunk/maven-project/src/main/java/org/apache/maven/project/ModelUtils.java
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();