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

Author: bobtarling
Date: 2008-04-29 20:23:43-0700
New Revision: 14528

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

Log:
Tidy up and javadoc

Modified: trunk/src/argouml-core-diagrams-sequence2/src/org/argouml/uml/diagram/sequence2/ui/FigMessage.java
Url: http://argouml.tigris.org/source/browse/argouml/trunk/src/argouml-core-diagrams-sequence2/src/org/argouml/uml/diagram/sequence2/ui/FigMessage.java?view=diff&rev=14528&p1=trunk/src/argouml-core-diagrams-sequence2/src/org/argouml/uml/diagram/sequence2/ui/FigMessage.java&p2=trunk/src/argouml-core-diagrams-sequence2/src/org/argouml/uml/diagram/sequence2/ui/FigMessage.java&r1=14527&r2=14528
==============================================================================
--- trunk/src/argouml-core-diagrams-sequence2/src/org/argouml/uml/diagram/sequence2/ui/FigMessage.java	(original)
+++ trunk/src/argouml-core-diagrams-sequence2/src/org/argouml/uml/diagram/sequence2/ui/FigMessage.java	2008-04-29 20:23:43-0700
@@ -27,7 +27,6 @@
 import java.awt.Color;
 import java.awt.Point;
 
-import org.apache.log4j.Logger;
 import org.argouml.model.Model;
 import org.argouml.uml.diagram.ui.FigEdgeModelElement;
 import org.argouml.uml.diagram.ui.FigTextGroup;
@@ -44,12 +43,6 @@
  */
 public class FigMessage extends FigEdgeModelElement {
 
-    /**
-     * Logger.
-     */
-    private static final Logger LOG =
-        Logger.getLogger(FigMessage.class);
-    
     private static final long serialVersionUID = -2961220746360335159L;
     private FigTextGroup textGroup; 
     
@@ -214,7 +207,10 @@
     /**
      * The default behaviour from FigEdgeModelElement is not correct
      * here. See issue 5005. TODO: We must determine what to do here but for
-     * now doing nothing is better.
+     * now doing nothing is better. I'm not sure why the super method would
+     * not work as I would expect that to do nothing if the ends are already
+     * correct.
+     * @return true at all times for now
      */
     @Override
     protected boolean determineFigNodes() {