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

[email protected]
Newsgroups gmane.comp.lang.uml.argouml.cvs
Message-ID <[email protected]>
Author: bobtarling
Date: 2008-04-14 16:58:58-0700
New Revision: 14351

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

Log:
Cut down on one needless rebuild of classifierrole

Modified: trunk/src/argouml-core-diagrams-sequence2/src/org/argouml/uml/diagram/sequence2/ui/ModeCreateMessage.java
Url: http://argouml.tigris.org/source/browse/argouml/trunk/src/argouml-core-diagrams-sequence2/src/org/argouml/uml/diagram/sequence2/ui/ModeCreateMessage.java?view=diff&rev=14351&p1=trunk/src/argouml-core-diagrams-sequence2/src/org/argouml/uml/diagram/sequence2/ui/ModeCreateMessage.java&p2=trunk/src/argouml-core-diagrams-sequence2/src/org/argouml/uml/diagram/sequence2/ui/ModeCreateMessage.java&r1=14350&r2=14351
==============================================================================
--- trunk/src/argouml-core-diagrams-sequence2/src/org/argouml/uml/diagram/sequence2/ui/ModeCreateMessage.java	(original)
+++ trunk/src/argouml-core-diagrams-sequence2/src/org/argouml/uml/diagram/sequence2/ui/ModeCreateMessage.java	2008-04-14 16:58:58-0700
@@ -105,8 +105,6 @@
                     Model.getMetaTypes().getMessage(),
                     Model.getMetaTypes().getReturnAction());
             
-            final Layer layer = editor.getLayerManager().getActiveLayer();
-            
             final FigMessage returnEdge = new FigMessage(returnMessage);
 
             returnEdge.setSourcePortFig(fe.getDestPortFig());
@@ -131,11 +129,14 @@
             // it doesn't need to be added again. 
             // The problem here is that the diagram
             // isn't damaged, and btw the edge ends' X are wrong.
+            final Layer layer = editor.getLayerManager().getActiveLayer();
             layer.add(returnEdge);
         }
         dcr.createActivations();
         dcr.renderingChanged();
-        scr.createActivations();
-        scr.renderingChanged();
+        if (dcr != scr) {
+            scr.createActivations();
+            scr.renderingChanged();
+        }
     }
 }
lmpx.com only provides a reader for public news (NNTP) servers. It is not affiliated with the servers or forums shown here and is not responsible for the content of articles, which is written by their respective authors.