You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@commons.apache.org by oh...@apache.org on 2008/03/22 19:41:28 UTC

svn commit: r640044 - /commons/proper/configuration/branches/configuration2_experimental/src/main/java/org/apache/commons/configuration2/tree/DefaultConfigurationNode.java

Author: oheger
Date: Sat Mar 22 11:41:27 2008
New Revision: 640044

URL: http://svn.apache.org/viewvc?rev=640044&view=rev
Log:
Changed order of statements in detachNode() so that the parent is set to null only after the reference is removed.

Modified:
    commons/proper/configuration/branches/configuration2_experimental/src/main/java/org/apache/commons/configuration2/tree/DefaultConfigurationNode.java

Modified: commons/proper/configuration/branches/configuration2_experimental/src/main/java/org/apache/commons/configuration2/tree/DefaultConfigurationNode.java
URL: http://svn.apache.org/viewvc/commons/proper/configuration/branches/configuration2_experimental/src/main/java/org/apache/commons/configuration2/tree/DefaultConfigurationNode.java?rev=640044&r1=640043&r2=640044&view=diff
==============================================================================
--- commons/proper/configuration/branches/configuration2_experimental/src/main/java/org/apache/commons/configuration2/tree/DefaultConfigurationNode.java (original)
+++ commons/proper/configuration/branches/configuration2_experimental/src/main/java/org/apache/commons/configuration2/tree/DefaultConfigurationNode.java Sat Mar 22 11:41:27 2008
@@ -685,11 +685,11 @@
          */
         protected void detachNode(ConfigurationNode subNode)
         {
-            subNode.setParentNode(null);
             if (subNode instanceof DefaultConfigurationNode)
             {
                 ((DefaultConfigurationNode) subNode).removeReference();
             }
+            subNode.setParentNode(null);
         }
 
         /**