fix for bug 20306 - regex handling of $ in replace string

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

src/etc/testcases/filters/tokenfilter.xml

@@ -181,6 +181,15 @@
     </concat>
   </target>
 
+  <target name="dollermatch">
+    <concat>
+      @hello@
+      <filterchain>
+        <replaceregex pattern="@([^@]*)@" replace="${\1}"/>
+      </filterchain>
+    </concat>
+  </target>
+
   <!-- need to check for existance of regex -->
   <target name="containsregex">
     <concat destfile="result/input">

src/main/org/apache/tools/ant/util/regexp/JakartaOroRegexp.java

@@ -76,7 +76,10 @@ public class JakartaOroRegexp extends JakartaOroMatcher implements Regexp {
         StringBuffer subst = new StringBuffer();
         for (int i = 0; i < argument.length(); i++) {
             char c = argument.charAt(i);
-            if (c == '\\') {
+            if (c == '$') {
+                subst.append('\\');
+                subst.append('$');
+            } else if (c == '\\') {
                 if (++i < argument.length()) {
                     c = argument.charAt(i);
                     int value = Character.digit(c, 10);

src/main/org/apache/tools/ant/util/regexp/Jdk14RegexpRegexp.java

@@ -83,7 +83,10 @@ public class Jdk14RegexpRegexp extends Jdk14RegexpMatcher implements Regexp {
         StringBuffer subst = new StringBuffer();
         for (int i = 0; i < argument.length(); i++) {
             char c = argument.charAt(i);
-            if (c == '\\') {
+            if (c == '$') {
+                subst.append('\\');
+                subst.append('$');
+            } else if (c == '\\') {
                 if (++i < argument.length()) {
                     c = argument.charAt(i);
                     int value = Character.digit(c, 10);

src/testcases/org/apache/tools/ant/filters/TokenFilterTest.java

@@ -77,7 +77,7 @@ public class TokenFilterTest extends BuildFileTest {
     }
 
     public void tearDown() {
-        //executeTarget("cleanup");
+        executeTarget("cleanup");
     }
 
     /** make sure tokenfilter exists */
@@ -147,6 +147,12 @@ public class TokenFilterTest extends BuildFileTest {
         assertStringContains(contents, "world world world world");
     }
 
+    public void testHandleDollerMatch() throws IOException {
+        if (! hasRegex("testFilterReplaceRegex"))
+            return;
+        executeTarget("dollermatch");
+    }
+    
     public void testTrimFile() throws IOException {
         String contents = getFileString(
             "trimfile", "result/trimfile");