diff --git a/src/main/org/apache/tools/ant/AntClassLoader.java b/src/main/org/apache/tools/ant/AntClassLoader.java
index 91fe7f12b..c6589bd14 100644
--- a/src/main/org/apache/tools/ant/AntClassLoader.java
+++ b/src/main/org/apache/tools/ant/AntClassLoader.java
@@ -54,24 +54,25 @@
 
 package org.apache.tools.ant;
 
+import java.io.ByteArrayOutputStream;
+import java.io.File;
+import java.io.FileInputStream;
+import java.io.IOException;
+import java.io.InputStream;
 import java.lang.reflect.Constructor;
-import java.lang.reflect.Method;
 import java.lang.reflect.InvocationTargetException;
+import java.lang.reflect.Method;
+import java.net.MalformedURLException;
+import java.net.URL;
 import java.util.Enumeration;
-import java.util.Vector;
 import java.util.Hashtable;
-import java.util.zip.ZipFile;
+import java.util.Vector;
 import java.util.zip.ZipEntry;
-import java.io.File;
-import java.io.InputStream;
-import java.io.FileInputStream;
-import java.io.IOException;
-import java.io.ByteArrayOutputStream;
-import java.net.URL;
-import java.net.MalformedURLException;
+import java.util.zip.ZipFile;
+
 import org.apache.tools.ant.types.Path;
-import org.apache.tools.ant.util.LoaderUtils;
 import org.apache.tools.ant.util.JavaEnvUtils;
+import org.apache.tools.ant.util.LoaderUtils;
 
 /**
  * Used to load classes within ant with a different claspath from
diff --git a/src/main/org/apache/tools/ant/DynamicConfigurator.java b/src/main/org/apache/tools/ant/DynamicConfigurator.java
index d22011bc3..92604eb89 100644
--- a/src/main/org/apache/tools/ant/DynamicConfigurator.java
+++ b/src/main/org/apache/tools/ant/DynamicConfigurator.java
@@ -61,8 +61,23 @@ package org.apache.tools.ant;
  * @since Ant 1.5
  */
 public interface DynamicConfigurator {
+    
+    /**
+     * Set a named attribute to the given value
+     * 
+     * @param name the name of the attribute
+     * @param value the new value of the attribute
+     * @throws BuildException when any error occurs
+     */    
     public void setDynamicAttribute(String name, String value)
             throws BuildException;
 
+    /**
+     * Create an element with the given name
+     *
+     * @param name the element nbame
+     * @throws BuildException when any error occurs
+     * @return the element created
+     */    
     public Object createDynamicElement(String name) throws BuildException;
 }
diff --git a/src/main/org/apache/tools/ant/ExitException.java b/src/main/org/apache/tools/ant/ExitException.java
index 053baa6af..d904d9fc7 100644
--- a/src/main/org/apache/tools/ant/ExitException.java
+++ b/src/main/org/apache/tools/ant/ExitException.java
@@ -75,7 +75,9 @@ public class ExitException extends SecurityException {
     }
 
     /**
-     * @return the status code return via System.exit()
+     * The status code returned by System.exit()
+     *
+     * @return the status code returned by System.exit()
      */
     public int getStatus() {
         return status;
diff --git a/src/main/org/apache/tools/ant/IntrospectionHelper.java b/src/main/org/apache/tools/ant/IntrospectionHelper.java
index ab0b38bf7..3e62537f0 100644
--- a/src/main/org/apache/tools/ant/IntrospectionHelper.java
+++ b/src/main/org/apache/tools/ant/IntrospectionHelper.java
@@ -55,13 +55,13 @@
 package org.apache.tools.ant;
 
 import org.apache.tools.ant.DynamicConfigurator;
-import org.apache.tools.ant.types.Path;
 import org.apache.tools.ant.types.EnumeratedAttribute;
+import org.apache.tools.ant.types.Path;
 
-import java.lang.reflect.Method;
-import java.lang.reflect.InvocationTargetException;
-import java.lang.reflect.Constructor;
 import java.io.File;
+import java.lang.reflect.Constructor;
+import java.lang.reflect.InvocationTargetException;
+import java.lang.reflect.Method;
 import java.util.Enumeration;
 import java.util.Hashtable;
 import java.util.Locale;
@@ -890,16 +890,44 @@ public class IntrospectionHelper implements BuildListener {
         helpers.clear();
     }
 
-    /** Empty implementation to satisfy the BuildListener interface. */
+    /**
+     * Empty implementation to satisfy the BuildListener interface.
+     * @param event Ignored in this implementation.
+     */
     public void buildStarted(BuildEvent event) {}
-    /** Empty implementation to satisfy the BuildListener interface. */
+    
+    /**
+     * Empty implementation to satisfy the BuildListener interface.
+     *
+     * @param event Ignored in this implementation.
+     */
     public void targetStarted(BuildEvent event) {}
-    /** Empty implementation to satisfy the BuildListener interface. */
+    
+    /**
+     * Empty implementation to satisfy the BuildListener interface.
+     *
+     * @param event Ignored in this implementation.
+     */
     public void targetFinished(BuildEvent event) {}
-    /** Empty implementation to satisfy the BuildListener interface. */
+    
+    /**
+     * Empty implementation to satisfy the BuildListener interface.
+     *
+     * @param event Ignored in this implementation.
+     */
     public void taskStarted(BuildEvent event) {}
-    /** Empty implementation to satisfy the BuildListener interface. */
+    
+    /**
+     * Empty implementation to satisfy the BuildListener interface.
+     *
+     * @param event Ignored in this implementation.
+     */
     public void taskFinished(BuildEvent event) {}
-    /** Empty implementation to satisfy the BuildListener interface. */
+    
+    /**
+     * Empty implementation to satisfy the BuildListener interface.
+     *
+     * @param event Ignored in this implementation.
+     */
     public void messageLogged(BuildEvent event) {}
 }