svn commit: r14693 - trunk/src/argouml-app/src/org/argouml/uml/diagram/ui/CompartmentFigText.java

Author: bobtarling
Date: 2008-05-11 11:48:44-0700
New Revision: 14693

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

Log:
Remove some leftover code from debugging.

Update the javadoc

Modified: trunk/src/argouml-app/src/org/argouml/uml/diagram/ui/CompartmentFigText.java
Url: http://argouml.tigris.org/source/browse/argouml/trunk/src/argouml-app/src/org/argouml/uml/diagram/ui/CompartmentFigText.java?view=diff&rev=14693&p1=trunk/src/argouml-app/src/org/argouml/uml/diagram/ui/CompartmentFigText.java&p2=trunk/src/argouml-app/src/org/argouml/uml/diagram/ui/CompartmentFigText.java&r1=14692&r2=14693
==============================================================================
--- trunk/src/argouml-app/src/org/argouml/uml/diagram/ui/CompartmentFigText.java	(original)
+++ trunk/src/argouml-app/src/org/argouml/uml/diagram/ui/CompartmentFigText.java	2008-05-11 11:48:44-0700
@@ -213,20 +213,16 @@
     }
 
     /**
-     * Mark whether this item is to be highlighted.<p>
-     *
-     * If it is highlighted, make the superclass line width 1 rather
-     * than 0 and set the associated component fig as the target in
-     * the browser.<p>
-     *
+     * This is actually used to mark this Fig as selected, however setSelected
+     * is set final in GEF.
+     * TODO: Can setSelected be used without side-effect if GEF is adjusted?
+     * Otherwise consider renamaing as setSelectedChild and try to make
+     * protected.
      * @param flag  <code>true</code> if the entry is to be highlighted,
      *              <code>false</code> otherwise.
      */
     public void setHighlighted(boolean flag) {
         highlighted = flag;
-        if (flag == false) {
-            flag = false;
-        }
     }
     
     /**