diff --git a/src/main/org/apache/tools/ant/taskdefs/optional/junit/DOMUtil.java b/src/main/org/apache/tools/ant/taskdefs/optional/junit/DOMUtil.java
index 9977b3658..393ef534c 100644
--- a/src/main/org/apache/tools/ant/taskdefs/optional/junit/DOMUtil.java
+++ b/src/main/org/apache/tools/ant/taskdefs/optional/junit/DOMUtil.java
@@ -89,7 +89,7 @@ public final class DOMUtil {
          * @return      <tt>true</tt> if the node is accepted by this filter,
          *                      otherwise <tt>false</tt>
          */
-        public boolean accept(Node node);
+        boolean accept(Node node);
     }
 
     /**
diff --git a/src/main/org/apache/tools/ant/taskdefs/optional/junit/JUnitResultFormatter.java b/src/main/org/apache/tools/ant/taskdefs/optional/junit/JUnitResultFormatter.java
index 8bc0aed3f..8931ffb25 100644
--- a/src/main/org/apache/tools/ant/taskdefs/optional/junit/JUnitResultFormatter.java
+++ b/src/main/org/apache/tools/ant/taskdefs/optional/junit/JUnitResultFormatter.java
@@ -54,6 +54,7 @@
 
 package org.apache.tools.ant.taskdefs.optional.junit;
 
+import java.io.OutputStream;
 import org.apache.tools.ant.BuildException;
 import junit.framework.TestListener;
 
@@ -67,26 +68,25 @@ public interface JUnitResultFormatter extends TestListener {
     /**
      * The whole testsuite started.
      */
-    public void startTestSuite(JUnitTest suite) throws BuildException;
-
+    void startTestSuite(JUnitTest suite) throws BuildException;
+    
     /**
      * The whole testsuite ended.
      */
-    public void endTestSuite(JUnitTest suite) throws BuildException;
+    void endTestSuite(JUnitTest suite) throws BuildException;
 
     /**
      * Sets the stream the formatter is supposed to write its results to.
      */
-    public void setOutput(java.io.OutputStream out);
+    void setOutput( OutputStream out );
 
     /**
      * This is what the test has written to System.out
      */
-    public void setSystemOutput( String out );
+    void setSystemOutput( String out );
 
     /**
      * This is what the test has written to System.err
      */
-    public void setSystemError( String err );
-
+    void setSystemError( String err );
 }
diff --git a/src/main/org/apache/tools/ant/taskdefs/optional/junit/XMLConstants.java b/src/main/org/apache/tools/ant/taskdefs/optional/junit/XMLConstants.java
index ede45026c..4e62e912c 100644
--- a/src/main/org/apache/tools/ant/taskdefs/optional/junit/XMLConstants.java
+++ b/src/main/org/apache/tools/ant/taskdefs/optional/junit/XMLConstants.java
@@ -68,57 +68,57 @@ package org.apache.tools.ant.taskdefs.optional.junit;
  */
 public interface XMLConstants {
     /** the testsuites element for the aggregate document */
-    public final static String TESTSUITES = "testsuites";
+    String TESTSUITES = "testsuites";
 
     /** the testsuite element */
-    public final static String TESTSUITE = "testsuite";
+    String TESTSUITE = "testsuite";
 
     /** the testcase element */
-    public final static String TESTCASE = "testcase";
+    String TESTCASE = "testcase";
 
     /** the error element */
-    public final static String ERROR = "error";
+    String ERROR = "error";
 
     /** the failure element */
-    public final static String FAILURE = "failure";
+    String FAILURE = "failure";
 
     /** the system-err element */
-    public final static String SYSTEM_ERR = "system-err";
+    String SYSTEM_ERR = "system-err";
 
     /** the system-out element */
-    public final static String SYSTEM_OUT = "system-out";
+    String SYSTEM_OUT = "system-out";
 
     /** package attribute for the aggregate document */
-    public final static String ATTR_PACKAGE = "package";
+    String ATTR_PACKAGE = "package";
 
     /** name attribute for property, testcase and testsuite elements */
-    public final static String ATTR_NAME = "name";
+    String ATTR_NAME = "name";
 
     /** time attribute for testcase and testsuite elements */
-    public final static String ATTR_TIME = "time";
+    String ATTR_TIME = "time";
 
     /** errors attribute for testsuite elements */
-    public final static String ATTR_ERRORS = "errors";
+    String ATTR_ERRORS = "errors";
 
     /** failures attribute for testsuite elements */
-    public final static String ATTR_FAILURES = "failures";
+    String ATTR_FAILURES = "failures";
 
     /** tests attribute for testsuite elements */
-    public final static String ATTR_TESTS = "tests";
+    String ATTR_TESTS = "tests";
 
     /** type attribute for failure and error elements */
-    public final static String ATTR_TYPE = "type";
+    String ATTR_TYPE = "type";
 
     /** message attribute for failure elements */
-    public final static String ATTR_MESSAGE = "message";
+    String ATTR_MESSAGE = "message";
 
     /** the properties element */
-    public final static String PROPERTIES = "properties";
+    String PROPERTIES = "properties";
 
     /** the property element */
-    public final static String PROPERTY = "property";
+    String PROPERTY = "property";
 
     /** value attribute for property elements */
-    public final static String ATTR_VALUE = "value";
+    String ATTR_VALUE = "value";
 
 }