svn commit: r16358 - trunk/src/argouml-app/src/org/argouml/uml/diagram/static_structure/ui/ClassDiagramRenderer.java

Author: tfmorris
Date: 2008-12-14 15:37:27-0800
New Revision: 16358

Modified:
   trunk/src/argouml-app/src/org/argouml/uml/diagram/static_structure/ui/ClassDiagramRenderer.java

Log:
Add comment about possible bug

Modified: trunk/src/argouml-app/src/org/argouml/uml/diagram/static_structure/ui/ClassDiagramRenderer.java
Url: http://argouml.tigris.org/source/browse/argouml/trunk/src/argouml-app/src/org/argouml/uml/diagram/static_structure/ui/ClassDiagramRenderer.java?view=diff&pathrev=16358&r1=16357&r2=16358
==============================================================================
--- trunk/src/argouml-app/src/org/argouml/uml/diagram/static_structure/ui/ClassDiagramRenderer.java	(original)
+++ trunk/src/argouml-app/src/org/argouml/uml/diagram/static_structure/ui/ClassDiagramRenderer.java	2008-12-14 15:37:27-0800
@@ -25,7 +25,6 @@
 package org.argouml.uml.diagram.static_structure.ui;
 
 import java.util.Collection;
-import java.util.Iterator;
 import java.util.Map;
 
 import org.apache.log4j.Logger;
@@ -200,17 +199,18 @@
             newEdge = new FigAbstraction(edge, settings);
         } else if (Model.getFacade().isADependency(edge)) {
 
-            Collection c = Model.getFacade().getStereotypes(edge);
-            Iterator i = c.iterator();
             String name = "";
-            while (i.hasNext()) {
-                Object o = i.next();
-                name = Model.getFacade().getName(o);
+            for (Object stereotype : Model.getFacade().getStereotypes(edge)) {
+                name = Model.getFacade().getName(stereotype);
                 if (CoreFactory.REALIZE_STEREOTYPE.equals(name)) {
                     break;
                 }
             }
             if (CoreFactory.REALIZE_STEREOTYPE.equals(name)) {
+                // TODO: This code doesn't look like it will get reached because
+                // any abstraction/realization is going to take the 
+                // isAAbstraction leg of the if before it gets to this more
+                // general case. - tfm 20080508
                 FigAbstraction realFig = new FigAbstraction(edge, settings);
 
                 Object supplier =

------------------------------------------------------
http://argouml.tigris.org/ds/viewMessage.do?dsForumId=5905&dsMessageId=984182

To unsubscribe from this discussion, e-mail: [[email protected]].