svn commit: r399728 - /maven/components/trunk/maven-project/src/main/java/org/apache/maven/project/ModelUtils.java

[kenney]

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();