svn commit: r12804 - trunk/src_new/org/argouml/uml/diagram/ui/FigEdgeModelElement.java

Author: mvw
Date: 2007-06-09 13:47:33-0700
New Revision: 12804

Modified:
   trunk/src_new/org/argouml/uml/diagram/ui/FigEdgeModelElement.java

Log:
Added TODO statements. IMHO these cleanups should be done in GEF.

Modified: trunk/src_new/org/argouml/uml/diagram/ui/FigEdgeModelElement.java
Url: http://argouml.tigris.org/source/browse/argouml/trunk/src_new/org/argouml/uml/diagram/ui/FigEdgeModelElement.java?view=diff&rev=12804&p1=trunk/src_new/org/argouml/uml/diagram/ui/FigEdgeModelElement.java&p2=trunk/src_new/org/argouml/uml/diagram/ui/FigEdgeModelElement.java&r1=12803&r2=12804
==============================================================================
--- trunk/src_new/org/argouml/uml/diagram/ui/FigEdgeModelElement.java	(original)
+++ trunk/src_new/org/argouml/uml/diagram/ui/FigEdgeModelElement.java	2007-06-09 13:47:33-0700
@@ -962,6 +962,8 @@
         if (own != null) {
             ProjectManager.getManager().getCurrentProject().moveToTrash(own);
         }
+
+        /* TODO: MVW: Why is this not done in GEF? */
         Iterator it = getPathItemFigs().iterator();
         while (it.hasNext()) {
             ((Fig) it.next()).deleteFromModel();
@@ -1057,6 +1059,7 @@
             notationProviderName.cleanListener(this, getOwner());
         }
 
+        /* TODO: MVW: Why is this not done in GEF? */
         Iterator it = getPathItemFigs().iterator();
         while (it.hasNext()) {
             Fig fig = (Fig) it.next();