sometimes an infinite recursion of filters is flagged when there is none. PR 44226. Submitted by Adam Bryzak.

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

CONTRIBUTORS

@@ -1,6 +1,7 @@
 Amongst other, the following people contributed to ant:
 
 Adam Blinkinsop
+Adam Bryzak
 Aleksandr Ishutin
 Alexey Panchenko
 Alexey Solofnenko

WHATSNEW

@@ -154,6 +154,10 @@ Fixed bugs:
    was not set.
    Bugzilla Report 45245.
 
+ * <filterset> sometimes incorrectly flagged infinite recursions of
+   filter tokens
+   Bugzilla Report 44226.
+
 Other changes:
 --------------
 

contributors.xml

@@ -34,6 +34,10 @@
     <first>Adam</first>
     <last>Blinkinsop</last>
   </name>
+  <name>
+    <first>Adam</first>
+    <last>Bryzak</last>
+  </name>
   <name>
     <first>Aleksandr</first>
     <last>Ishutin</last>

src/main/org/apache/tools/ant/types/FilterSet.java

@@ -593,6 +593,9 @@ public class FilterSet extends DataType implements Cloneable {
                     duplicateToken = false;
                 }
             }
+        } else if (passedTokens.size() > 0) {
+            // remove last seen token when crawling out of recursion 
+            passedTokens.remove(passedTokens.size() - 1);
         }
         recurseDepth--;
         return value;

src/tests/antunit/types/filterset-test.xml

@@ -15,27 +15,29 @@
   See the License for the specific language governing permissions and
   limitations under the License.
 -->
-<project xmlns:au="antlib:org.apache.ant.antunit">
+<project xmlns:au="antlib:org.apache.ant.antunit" default="antunit">
+
+  <import file="../antunit-base.xml" />
 
   <property name="br" value="${line.separator}" />
 
   <target name="testRecursionRegression">
     <copy todir="${basedir}">
-      <string value="@a@${br}@b@${br}@c@${br}" />
+      <string value="@a@${br}@b@${br}@c@${br}@d@${br}" />
       <mergemapper to="filterset-output.txt" />
       <filterset>
         <filter token="a" value="aaa" />
         <filter token="b" value="bbb" />
         <filter token="c" value="@a@:@b@" />
+        <filter token="d" value="@c@:@c@" />
       </filterset>
     </copy>
 
-    <au:assertTrue>
-      <resourcesmatch astext="true">
-        <file file="filterset-output.txt" />
-        <string value="aaa${br}bbb${br}aaa:bbb${br}" />
-      </resourcesmatch>
-    </au:assertTrue>
+    <loadfile property="afterfiltering" srcFile="filterset-output.txt"/>
+
+    <au:assertEquals
+       expected="aaa${br}bbb${br}aaa:bbb${br}aaa:bbb:aaa:bbb${br}"
+       actual="${afterfiltering}"/>
   </target>
 
   <target name="tearDown">