Author: mbenson
Date: Mon May 14 08:31:24 2007
New Revision: 537880

URL: http://svn.apache.org/viewvc?view=rev&rev=537880
Log:
early exit for empty substring with optional verbose log message

Modified:
    
ant/core/trunk/src/main/org/apache/tools/ant/taskdefs/condition/ResourceContains.java

Modified: 
ant/core/trunk/src/main/org/apache/tools/ant/taskdefs/condition/ResourceContains.java
URL: 
http://svn.apache.org/viewvc/ant/core/trunk/src/main/org/apache/tools/ant/taskdefs/condition/ResourceContains.java?view=diff&rev=537880&r1=537879&r2=537880
==============================================================================
--- 
ant/core/trunk/src/main/org/apache/tools/ant/taskdefs/condition/ResourceContains.java
 (original)
+++ 
ant/core/trunk/src/main/org/apache/tools/ant/taskdefs/condition/ResourceContains.java
 Mon May 14 08:31:24 2007
@@ -23,6 +23,7 @@
 import java.io.InputStreamReader;
 
 import org.apache.tools.ant.BuildException;
+import org.apache.tools.ant.Project;
 import org.apache.tools.ant.types.Resource;
 import org.apache.tools.ant.types.resources.FileResource;
 import org.apache.tools.ant.util.FileUtils;
@@ -34,11 +35,28 @@
  */
 public class ResourceContains implements Condition {
 
+    private Project project;
     private String substring;
     private Resource resource;
     private boolean casesensitive = true;
 
     /**
+     * Set this condition's Project.
+     * @param project Project
+     */
+    public void setProject(Project project) {
+        this.project = project;
+    }
+
+    /**
+     * Get this condition's Project.
+     * @return Project
+     */
+    public Project getProject() {
+        return project;
+    }
+
+    /**
      * Sets the resource to search
      * @param r
      */
@@ -70,6 +88,13 @@
         if (resource == null || substring == null) {
             throw new BuildException("both resource and substring are required 
"
                                      + "in <resourcecontains>");
+        }
+
+        if (substring.length() == 0) {
+            if (getProject() != null) {
+                getProject().log("ResourceContains: substring is empty; 
returning true", Project.MSG_VERBOSE);
+            }
+            return true;
         }
 
         if (resource.getSize() == 0) {



---------------------------------------------------------------------
To unsubscribe, e-mail: [EMAIL PROTECTED]
For additional commands, e-mail: [EMAIL PROTECTED]

Reply via email to