From 6dca9007eb5d97f57e78e347cf886dfbad061765 Mon Sep 17 00:00:00 2001
From: Brett Leslie Porter <brett@apache.org>
Date: Thu, 5 May 2005 05:44:44 +0000
Subject: [PATCH] a couple of notes

git-svn-id: https://svn.apache.org/repos/asf/maven/components/trunk@168278 13f79535-47bb-0310-9956-ffa450edef68
---
 .../org/apache/maven/lifecycle/DefaultLifecycleExecutor.java   | 3 ++-
 .../java/org/apache/maven/plugin/DefaultPluginManager.java     | 1 +
 2 files changed, 3 insertions(+), 1 deletion(-)

diff --git a/maven-core/src/main/java/org/apache/maven/lifecycle/DefaultLifecycleExecutor.java b/maven-core/src/main/java/org/apache/maven/lifecycle/DefaultLifecycleExecutor.java
index 66d57d39ba..a99e7abb7d 100644
--- a/maven-core/src/main/java/org/apache/maven/lifecycle/DefaultLifecycleExecutor.java
+++ b/maven-core/src/main/java/org/apache/maven/lifecycle/DefaultLifecycleExecutor.java
@@ -46,7 +46,8 @@ import java.util.Map;
  * @author <a href="mailto:jason@maven.org">Jason van Zyl </a>
  * @version $Id: DefaultLifecycleExecutor.java,v 1.16 2005/03/04 09:04:25
  *          jdcasey Exp $
- * @todo there is some duplication between this and the plugin manager
+ * @todo this is structured somewhat confusingly. Attempt to "flatten out" to reduce the number of paths through by
+ * compiling the list of plugins/tasks first.
  */
 public class DefaultLifecycleExecutor
     extends AbstractLogEnabled
diff --git a/maven-core/src/main/java/org/apache/maven/plugin/DefaultPluginManager.java b/maven-core/src/main/java/org/apache/maven/plugin/DefaultPluginManager.java
index 69c43ec7a5..8f959dd07e 100644
--- a/maven-core/src/main/java/org/apache/maven/plugin/DefaultPluginManager.java
+++ b/maven-core/src/main/java/org/apache/maven/plugin/DefaultPluginManager.java
@@ -101,6 +101,7 @@ public class DefaultPluginManager
      *
      * @param name
      * @return
+     * @todo remove
      */
     public MojoDescriptor getMojoDescriptor( String name )
     {