I was just doing a quick lookthrough of the now committed sleep task when I
noticed that the internal validate() method (which validates parameters)
didn't declare that it threw a BuildException, even though it could.

This patch fixes that.

-steve
Index: Sleep.java
===================================================================
RCS file: 
/home/cvspublic/jakarta-ant/src/main/org/apache/tools/ant/taskdefs/Sleep.java,v
retrieving revision 1.1
diff -u -r1.1 Sleep.java
--- Sleep.java  2001/06/21 10:19:11     1.1
+++ Sleep.java  2001/06/24 20:31:18
@@ -177,7 +177,8 @@
      *
      * @throws BuildException if something is invalid
      */
-    public void validate() {
+    public void validate() 
+        throws BuildException {
         long sleepTime = getSleepTime();
         if (getSleepTime() < 0) {
             throw new BuildException("Negative sleep periods are not 
supported");

Reply via email to