You are viewing a plain text version of this content. The canonical link for it is here.

Posted to commits@pivot.apache.org by tv...@apache.org on 2009/05/12 07:33:11 UTC

svn commit: r773788 - /incubator/pivot/trunk/wtk/src/pivot/wtk/Container.java

Author: tvolkert
Date: Tue May 12 05:33:10 2009
New Revision: 773788

URL: http://svn.apache.org/viewvc?rev=773788&view=rev
Log:
Added comment to Container

Modified:
    incubator/pivot/trunk/wtk/src/pivot/wtk/Container.java

Modified: incubator/pivot/trunk/wtk/src/pivot/wtk/Container.java
URL: http://svn.apache.org/viewvc/incubator/pivot/trunk/wtk/src/pivot/wtk/Container.java?rev=773788&r1=773787&r2=773788&view=diff
==============================================================================
--- incubator/pivot/trunk/wtk/src/pivot/wtk/Container.java (original)
+++ incubator/pivot/trunk/wtk/src/pivot/wtk/Container.java Tue May 12 05:33:10 2009
@@ -160,6 +160,9 @@
             throw new IllegalArgumentException("Component already has a parent.");
         }
 
+        // TODO Attempting to insert an ancestor will cause a stack overflow.
+        // Do we care to guard against that case?
+
         component.setParent(Container.this);
         components.insert(component, index);