You are viewing a plain text version of this content. The canonical link for it is here.
Posted to notifications@ant.apache.org by ke...@apache.org on 2008/04/27 04:46:11 UTC

svn commit: r651879 - /ant/core/branches/ANT_17_BRANCH/src/main/org/apache/tools/ant/UnknownElement.java

Author: kevj
Date: Sat Apr 26 19:46:09 2008
New Revision: 651879

URL: http://svn.apache.org/viewvc?rev=651879&view=rev
Log:
-merge 641903 back into 1.7.1

Modified:
    ant/core/branches/ANT_17_BRANCH/src/main/org/apache/tools/ant/UnknownElement.java

Modified: ant/core/branches/ANT_17_BRANCH/src/main/org/apache/tools/ant/UnknownElement.java
URL: http://svn.apache.org/viewvc/ant/core/branches/ANT_17_BRANCH/src/main/org/apache/tools/ant/UnknownElement.java?rev=651879&r1=651878&r2=651879&view=diff
==============================================================================
--- ant/core/branches/ANT_17_BRANCH/src/main/org/apache/tools/ant/UnknownElement.java (original)
+++ ant/core/branches/ANT_17_BRANCH/src/main/org/apache/tools/ant/UnknownElement.java Sat Apr 26 19:46:09 2008
@@ -288,11 +288,14 @@
                 ((Task) realThing).execute();
             }
         } finally {
-            // Finished executing the task, null it to allow
+        	// Finished executing the task
+            // null it (unless it has an ID) to allow
             // GC do its job
             // If this UE is used again, a new "realthing" will be made
-            realThing = null;
-            getWrapper().setProxy(null);
+            if (getWrapper().getId() == null) {
+                realThing = null;
+                getWrapper().setProxy(null);
+            }
         }
     }