some more notes

git-svn-id: https://svn.apache.org/repos/asf/maven/repository-manager/trunk@412324 13f79535-47bb-0310-9956-ffa450edef68

maven-repository-indexer/src/main/java/org/apache/maven/repository/indexing/DefaultRepositoryIndexSearcher.java

@@ -47,7 +47,9 @@ import java.util.Map;
 import java.util.StringTokenizer;
 
 /**
- * Implementation Class for searching through the index
+ * Implementation Class for searching through the index.
+ *
+ * @todo this is not a component, but extends ALE, meaning logging will throw an exception! -- should be a component
  */
 public class DefaultRepositoryIndexSearcher
     extends AbstractLogEnabled

maven-repository-indexer/src/main/java/org/apache/maven/repository/indexing/DefaultRepositoryIndexingFactory.java

@@ -23,6 +23,7 @@ import org.apache.maven.repository.digest.Digester;
 /**
  * @author Edwin Punzalan
  * @plexus.component role="org.apache.maven.repository.indexing.RepositoryIndexingFactory"
+ * @todo these methods should be replaced by plexus lookups of some kind!
  */
 public class DefaultRepositoryIndexingFactory
     implements RepositoryIndexingFactory

maven-repository-proxy/src/main/java/org/apache/maven/repository/proxy/DefaultProxyManager.java

@@ -53,6 +53,7 @@ import java.util.Map;
  * @plexus.component role="org.apache.maven.repository.proxy.ProxyManager"
  * @todo too much of wagon manager is reproduced here because checksums need to be downloaded separately - is that necessary?
  * @todo this isn't reusing the parts of wagon manager than handle snapshots [!]
+ * @todo currently, cache must be in the same layout as the request, which prohibits any mapping
  */
 public class DefaultProxyManager
     extends AbstractLogEnabled

maven-repository-proxy/src/main/java/org/apache/maven/repository/proxy/configuration/MavenProxyPropertyLoader.java

@@ -122,7 +122,10 @@ public class MavenProxyPropertyLoader
         return config;
     }
 
-    private void validateRemoteRepo( ProxyConfiguration configuration )
+    /**
+     * @todo should be shared with any other configuration loader - move method to configuration?
+     */
+    private static void validateRemoteRepo( ProxyConfiguration configuration )
         throws ValidationException
     {
         //Verify remote repository set
@@ -171,7 +174,10 @@ public class MavenProxyPropertyLoader
         return value;
     }
 
-    private void validateDirectories( ProxyConfiguration configuration )
+    /**
+     * @todo should be shared with any other configuration loader - move method to configuration?
+     */
+    private static void validateDirectories( ProxyConfiguration configuration )
         throws ValidationException
     {
         File f = new File( configuration.getRepositoryCachePath() );