From 67a935d30dd36f219f76666c37dc20f5d6eeaaee Mon Sep 17 00:00:00 2001
From: Uwe Schindler <uwe@thetaphi.de>
Date: Sat, 26 Sep 2015 00:15:20 +0200
Subject: [PATCH 1/2] Allow arbitrary Ant resources as signatures

---
 .../thetaphi/forbiddenapis/ant/AntTask.java   | 24 ++++++++++++-------
 src/test/antunit/TestFileSignatures.xml       | 14 +++++++++++
 2 files changed, 29 insertions(+), 9 deletions(-)

diff --git a/src/main/java/de/thetaphi/forbiddenapis/ant/AntTask.java b/src/main/java/de/thetaphi/forbiddenapis/ant/AntTask.java
index 97358099..05cbc2c0 100644
--- a/src/main/java/de/thetaphi/forbiddenapis/ant/AntTask.java
+++ b/src/main/java/de/thetaphi/forbiddenapis/ant/AntTask.java
@@ -34,6 +34,7 @@
 import org.apache.tools.ant.types.resources.FileResource;
 import org.apache.tools.ant.types.resources.Resources;
 import org.apache.tools.ant.types.resources.StringResource;
+import org.apache.tools.ant.types.resources.Union;
 
 import de.thetaphi.forbiddenapis.Checker;
 import de.thetaphi.forbiddenapis.ForbiddenApiException;
@@ -42,10 +43,10 @@
 
 import java.io.IOException;
 import java.io.File;
-import java.util.ArrayList;
+import java.util.Collection;
 import java.util.EnumSet;
 import java.util.Iterator;
-import java.util.List;
+import java.util.LinkedHashSet;
 import java.util.Locale;
 
 /**
@@ -56,10 +57,10 @@
  */
 public class AntTask extends Task {
 
-  private final Resources classFiles = new Resources();
-  private final Resources apiSignatures = new Resources();
-  private final List<BundledSignaturesType> bundledSignatures = new ArrayList<BundledSignaturesType>();
-  private final List<SuppressAnnotationType> suppressAnnotations = new ArrayList<SuppressAnnotationType>();
+  private final Union classFiles = new Union();
+  private final Union apiSignatures = new Union();
+  private final Collection<BundledSignaturesType> bundledSignatures = new LinkedHashSet<BundledSignaturesType>();
+  private final Collection<SuppressAnnotationType> suppressAnnotations = new LinkedHashSet<SuppressAnnotationType>();
   private Path classpath = null;
   
   private boolean failOnUnsupportedJava = false;
@@ -155,7 +156,7 @@ public void info(String msg) {
       }
         
       if (checker.hasNoSignatures()) {
-        throw new BuildException("No API signatures found; use signaturesFile=, <signaturesFileSet/>, <bundledSignatures/> or inner text to define those!");
+        throw new BuildException("No API signatures found; use signaturesFile=, <signatures*/>, <bundledSignatures/> or inner text to define those!");
       }
 
       log.info("Loading classes to check...");
@@ -210,8 +211,8 @@ public void setDir(File dir) {
     classFiles.add(fs);
   }
   
-  private <T extends ResourceCollection> T addSignaturesResource(T res) {
-    ((ProjectComponent) res).setProject(getProject());
+  private <T extends ProjectComponent & ResourceCollection> T addSignaturesResource(T res) {
+    res.setProject(getProject());
     apiSignatures.add(res);
     return res;
   }
@@ -236,6 +237,11 @@ public void setSignaturesFile(File file) {
     createSignaturesFile().setFile(file);
   }
   
+  /** Creates a collection of arbitrary Ant resources */
+  public Resources createSignaturesResources() {
+    return addSignaturesResource(new Resources());
+  }
+
   /** Support for API signatures list as nested text */
   public void addText(String text) {
     addSignaturesResource(new StringResource(text));
diff --git a/src/test/antunit/TestFileSignatures.xml b/src/test/antunit/TestFileSignatures.xml
index b6a5e907..6e1dcc56 100644
--- a/src/test/antunit/TestFileSignatures.xml
+++ b/src/test/antunit/TestFileSignatures.xml
@@ -57,4 +57,18 @@
     <au:assertLogContains level="error" text="java.lang.String#substring(int,int) [You are crazy that you disallow substrings]"/> 
   </target>
   
+  <target name="testGeneralResources">
+    <au:expectfailure expectedMessage="Check for forbidden API calls failed, see log">
+      <forbiddenapis classpathref="path.all">
+        <fileset refid="main.classes"/>
+        <signaturesResources>
+          <file file="signatures1.txt"/>
+          <string>java.util.Locale#ENGLISH @ We are speaking chinese here!</string>
+        </signaturesResources>
+      </forbiddenapis>
+    </au:expectfailure>
+    <au:assertLogContains level="error" text="java.lang.String#substring(int,int) [You are crazy that you disallow substrings]"/> 
+    <au:assertLogContains level="error" text="java.util.Locale#ENGLISH [We are speaking chinese here!]"/> 
+  </target>
+  
 </project>
\ No newline at end of file

From bb7e350c605bb1728b4ee5e2b6ac7b9919deb0a0 Mon Sep 17 00:00:00 2001
From: Uwe Schindler <uwe@thetaphi.de>
Date: Sat, 26 Sep 2015 00:22:22 +0200
Subject: [PATCH 2/2] Add documentation

---
 src/main/docs/ant-task.html | 3 ++-
 1 file changed, 2 insertions(+), 1 deletion(-)

diff --git a/src/main/docs/ant-task.html b/src/main/docs/ant-task.html
index 19900147..24950040 100644
--- a/src/main/docs/ant-task.html
+++ b/src/main/docs/ant-task.html
@@ -165,7 +165,8 @@ <h2>Parameters specified as nested elements</h2>
 
 <ul>
   <li>Use <code>bundledSignatures</code> element to pass a <a href="bundled-signatures.html">built-in signatures</a> file, e.g. <code>&lt;bundledsignatures name=&quot;jdk-unsafe-1.7&quot;/&gt;</code></li>
-  <li>Use <code>signaturesFileSet</code>, <code>signaturesFileList</code>, <code>signaturesFile</code> elements to pass in collections of signatures files. Those elements behave like the corresponding standard Ant types.</li>
+  <li>Use <code>signaturesResources</code> element to wrap any valid <a href="https://ant.apache.org/manual/Types/resources.html">Ant resource</a> type (filesets,..).</li>
+  <li>Alternatively, use <code>signaturesFileSet</code>, <code>signaturesFileList</code>, <code>signaturesFile</code> elements to pass in collections of signatures files. Those elements behave like the corresponding standard Ant types.</li>
   <li>Place signatures as plain text (use CDATA sections) inside the <code>forbiddenapis</code> element.</li>
 </ul>