diff --git a/proposal/sandbox/junit/src/main/org/apache/tools/ant/taskdefs/optional/junit/FormatterElement.java b/proposal/sandbox/junit/src/main/org/apache/tools/ant/taskdefs/optional/junit/FormatterElement.java
index 1d0e142e3..5c8dde4ef 100644
--- a/proposal/sandbox/junit/src/main/org/apache/tools/ant/taskdefs/optional/junit/FormatterElement.java
+++ b/proposal/sandbox/junit/src/main/org/apache/tools/ant/taskdefs/optional/junit/FormatterElement.java
@@ -110,7 +110,7 @@ public class FormatterElement {
 
     /**
      * Setting a comma separated list of filters in the specified order.
-     * @see #addFilter(FilterAttribute)
+     * @see #addFilter(FilterElement)
      * @see FilterAttribute
      */
     public void setFilters(String filters) {
diff --git a/proposal/sandbox/junit/src/main/org/apache/tools/ant/taskdefs/optional/junit/JUnitHelper.java b/proposal/sandbox/junit/src/main/org/apache/tools/ant/taskdefs/optional/junit/JUnitHelper.java
index 3d3a5fc56..13d3e82ed 100644
--- a/proposal/sandbox/junit/src/main/org/apache/tools/ant/taskdefs/optional/junit/JUnitHelper.java
+++ b/proposal/sandbox/junit/src/main/org/apache/tools/ant/taskdefs/optional/junit/JUnitHelper.java
@@ -87,6 +87,8 @@ public final class JUnitHelper {
 
     /**
      * Returns the Test corresponding to to the given class name
+     * @param loader classloader to use when loading the class or
+     * <tt>null</tt> for system classloader.
      * @param classname the classname of the test we want to extract.
      * @throws Exception a generic exception
      */
diff --git a/proposal/sandbox/junit/src/main/org/apache/tools/ant/taskdefs/optional/junit/JUnitTask.java b/proposal/sandbox/junit/src/main/org/apache/tools/ant/taskdefs/optional/junit/JUnitTask.java
index 7242a4dc6..24d1fc700 100644
--- a/proposal/sandbox/junit/src/main/org/apache/tools/ant/taskdefs/optional/junit/JUnitTask.java
+++ b/proposal/sandbox/junit/src/main/org/apache/tools/ant/taskdefs/optional/junit/JUnitTask.java
@@ -221,8 +221,8 @@ public class JUnitTask extends Task {
 
     /**
      * Set the maximum memory to be used by the TestRunner
-     * @param   max     the value as defined by <tt>-mx</tt> or <tt>-Xmx</tt>
-     *                  in the java command line options.
+     * @param max the value as defined by <tt>-mx</tt> or <tt>-Xmx</tt>
+     * in the java command line options.
      */
     public void setMaxmemory(String max) {
         if (Project.getJavaVersion().startsWith("1.1")) {
@@ -234,8 +234,8 @@ public class JUnitTask extends Task {
 
     /**
      * Create a new JVM argument. Ignored if no JVM is forked.
-     * @return  create a new JVM argument so that any argument can be passed to the JVM.
-     * @see #setFork(boolean)
+     * @return  create a new JVM argument so that any argument can be
+     * passed to the JVM.
      */
     public Commandline.Argument createJvmarg() {
         return cmd.createVmArgument();
diff --git a/proposal/sandbox/junit/src/main/org/apache/tools/ant/taskdefs/optional/junit/TestRunListener.java b/proposal/sandbox/junit/src/main/org/apache/tools/ant/taskdefs/optional/junit/TestRunListener.java
index b5a59c78c..55e5569cc 100644
--- a/proposal/sandbox/junit/src/main/org/apache/tools/ant/taskdefs/optional/junit/TestRunListener.java
+++ b/proposal/sandbox/junit/src/main/org/apache/tools/ant/taskdefs/optional/junit/TestRunListener.java
@@ -61,8 +61,8 @@ import java.util.Properties;
  *
  * <i>
  * This code is based on the code from Erich Gamma made for the
- * JUnit plugin for Eclipse. {@link http://www.eclipse.org} and is merged
- * with code originating from Ant 1.4.x.
+ * JUnit plugin for <a href="http://www.eclipse.org">Eclipse</a> and is
+ *  merged with code originating from Ant 1.4.x.
  * </i>
  *
  * @author <a href="mailto:sbailliez@apache.org">Stephane Bailliez</a>