Cleanup code a little

git-svn-id: https://svn.apache.org/repos/asf/ant/core/trunk@270562 13f79535-47bb-0310-9956-ffa450edef68

proposal/myrmidon/src/main/org/apache/tools/ant/taskdefs/unix/Chmod.java

@@ -107,10 +107,10 @@ public class Chmod extends ExecuteOn
         havePerm = true;
     }
 
-    public void setSkipEmptyFilesets( boolean skip )
-        throws TaskException
+    public void setSkipEmptyFilesets( final boolean skip )
     {
-        throw new TaskException( getName() + " doesn\'t support the skipemptyfileset attribute" );
+        final String message = getName() + " doesn\'t support the skipemptyfileset attribute";
+        throw new IllegalArgumentException( message );
     }
 
     /**
@@ -168,7 +168,8 @@ public class Chmod extends ExecuteOn
             }
             catch( IOException e )
             {
-                throw new TaskException( "Execute failed: " + e, e );
+                final String message = "Execute failed: " + e;
+                throw new TaskException( message, e );
             }
             finally
             {

proposal/myrmidon/src/todo/org/apache/tools/ant/taskdefs/unix/Chmod.java

@@ -107,10 +107,10 @@ public class Chmod extends ExecuteOn
         havePerm = true;
     }
 
-    public void setSkipEmptyFilesets( boolean skip )
-        throws TaskException
+    public void setSkipEmptyFilesets( final boolean skip )
     {
-        throw new TaskException( getName() + " doesn\'t support the skipemptyfileset attribute" );
+        final String message = getName() + " doesn\'t support the skipemptyfileset attribute";
+        throw new IllegalArgumentException( message );
     }
 
     /**
@@ -168,7 +168,8 @@ public class Chmod extends ExecuteOn
             }
             catch( IOException e )
             {
-                throw new TaskException( "Execute failed: " + e, e );
+                final String message = "Execute failed: " + e;
+                throw new TaskException( message, e );
             }
             finally
             {