svn commit: r16247 - trunk/src/argouml-core-diagrams-sequence2/src/org/argouml/uml/diagram/sequence2/ui/FigClassifierRole.java

Author: bobtarling
Date: 2008-12-04 15:06:23-0800
New Revision: 16247

Modified:
   trunk/src/argouml-core-diagrams-sequence2/src/org/argouml/uml/diagram/sequence2/ui/FigClassifierRole.java

Log:
Make it clear where we have a FigEdge rather than Fig.

A TODO about a dubious piece of code

Modified: trunk/src/argouml-core-diagrams-sequence2/src/org/argouml/uml/diagram/sequence2/ui/FigClassifierRole.java
Url: http://argouml.tigris.org/source/browse/argouml/trunk/src/argouml-core-diagrams-sequence2/src/org/argouml/uml/diagram/sequence2/ui/FigClassifierRole.java?view=diff&pathrev=16247&r1=16246&r2=16247
==============================================================================
--- trunk/src/argouml-core-diagrams-sequence2/src/org/argouml/uml/diagram/sequence2/ui/FigClassifierRole.java	(original)
+++ trunk/src/argouml-core-diagrams-sequence2/src/org/argouml/uml/diagram/sequence2/ui/FigClassifierRole.java	2008-12-04 15:06:23-0800
@@ -229,13 +229,15 @@
      */
     private void updateMinimumHeight() {
         int yMax = getY();
-        List<Fig> figs = getEdges();
+        List<FigEdge> figsEdges = getEdges();
         FigMessage createMessage = getFirstCreateFigMessage();
         
-        if (figs.size() == 1 && createMessage != null) {
+        // TODO: Is this next line safe? What happens if there is just one
+        // comment edge or a comment edge and a single message?
+        if (figsEdges.size() == 1 && createMessage != null) {
             minimumHeight = headFig.getMinimumSize().height + offset + 10;
         } else {
-            for (Fig fig : figs) {
+            for (Fig fig : figsEdges) {
                 if ( fig instanceof FigMessage
                         // we need the edge to be complete
                         && ((FigMessage) fig).getDestFigNode() != null

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

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