-public abstract class DataType extends ProjectComponent {
+public abstract class DataType extends ProjectComponent implements Cloneable {
+    public Object clone() throws CloneNotSupportedException {
+        DataType dt = (DataType) super.clone();
+        dt.setDescription(getDescription());
+        if (getRefid() != null) {
+           dt.setRefid(getRefid());
+       }
+        dt.setChecked(isChecked());
+        return dt;
+    }
 }

--- ant/core/trunk/src/main/org/apache/tools/ant/types/PatternSet.java 
(original)
+++ ant/core/trunk/src/main/org/apache/tools/ant/types/PatternSet.java Tue Oct 
>     public Object clone() {
-        if (isReference()) {
-            return getRef(getProject()).clone();
-        } else {
...
+        try {
+            PatternSet ps = (PatternSet) super.clone();

You've removed the isReference check Antoine. Did you really intend to
clone the reference'r rather than the reference'd? I prefer to ask,
since I don't understand the reason for these changes (which is not to
say I context the changes). --DD

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

Reply via email to