[PATCH] bugzilla 40238 - nested macrodef

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

CONTRIBUTORS

@@ -147,6 +147,7 @@ Martijn Kruithof
 Martin Landers
 Martin Poeschl
 Martin van den Bemt
+Mathieu Champlon
 Mathieu Peltier
 Matt Albrecht
 Matt Benson

WHATSNEW

@@ -270,6 +270,9 @@ Fixed bugs:
 * <rmic> has a new adapter, xnew, to use the -XNew back end on java1.5+.
   By forking rmic, this works on java1.6+. Bugzilla report 38732.
 
+* Copy of UnknownElement in macroinstance was not recursive.
+  Bugzilla report 40238.
+
 Other changes:
 --------------
 

src/main/org/apache/tools/ant/taskdefs/MacroInstance.java

@@ -291,7 +291,7 @@ public class MacroInstance extends Task implements DynamicAttribute, TaskContain
                 }
                 for (Iterator i = unknownElements.iterator();
                      i.hasNext();) {
-                    UnknownElement child = (UnknownElement) i.next();
+                    UnknownElement child = copy((UnknownElement) i.next());
                     rc.addChild(child.getWrapper());
                     ret.addChild(child);
                 }
@@ -307,7 +307,7 @@ public class MacroInstance extends Task implements DynamicAttribute, TaskContain
                 }
                 for (Iterator i = list.iterator();
                      i.hasNext();) {
-                    UnknownElement child = (UnknownElement) i.next();
+                    UnknownElement child = copy((UnknownElement) i.next());
                     rc.addChild(child.getWrapper());
                     ret.addChild(child);
                 }