svn commit: r16223 - trunk/src/argouml-app/src/org/argouml/uml/diagram/ui/ActionCompartmentDisplay.java

Author: tfmorris
Date: 2008-12-02 11:23:32-0800
New Revision: 16223

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

Log:
Add generic types

Modified: trunk/src/argouml-app/src/org/argouml/uml/diagram/ui/ActionCompartmentDisplay.java
Url: http://argouml.tigris.org/source/browse/argouml/trunk/src/argouml-app/src/org/argouml/uml/diagram/ui/ActionCompartmentDisplay.java?view=diff&pathrev=16223&r1=16222&r2=16223
==============================================================================
--- trunk/src/argouml-app/src/org/argouml/uml/diagram/ui/ActionCompartmentDisplay.java	(original)
+++ trunk/src/argouml-app/src/org/argouml/uml/diagram/ui/ActionCompartmentDisplay.java	2008-12-02 11:23:32-0800
@@ -30,6 +30,8 @@
 import java.util.Iterator;
 import java.util.List;
 
+import javax.swing.Action;
+
 import org.argouml.i18n.Translator;
 import org.argouml.uml.diagram.AttributesCompartmentContainer;
 import org.argouml.uml.diagram.ExtensionsCompartmentContainer;
@@ -180,8 +182,8 @@
      * @return Only returns the actions for the menu-items that make sense for
      *         the current selection.
      */
-    public static Collection getActions() {
-        Collection actions = new ArrayList();
+    public static Collection<Action> getActions() {
+        Collection<Action> actions = new ArrayList<Action>();
         Editor ce = Globals.curEditor();
 
         int present = 0;
@@ -298,6 +300,7 @@
      *
      * @param ae  The event that triggered us.
      */
+    @Override
     public void actionPerformed(ActionEvent ae) {
 	Iterator i =
             Globals.curEditor().getSelectionManager().selections().iterator();
@@ -307,14 +310,16 @@
 
             // Perform the action
             if ((cType & COMPARTMENT_ATTRIBUTE) != 0) {
-		if (f instanceof AttributesCompartmentContainer)
+		if (f instanceof AttributesCompartmentContainer) {
 		    ((AttributesCompartmentContainer) f)
                         .setAttributesVisible(display);
+		}
             }
             if ((cType & COMPARTMENT_OPERATION) != 0) {
-		if (f instanceof OperationsCompartmentContainer)
+		if (f instanceof OperationsCompartmentContainer) {
 		    ((OperationsCompartmentContainer) f)
                         .setOperationsVisible(display);
+		}
             }
 
             if ((cType & COMPARTMENT_EXTENSIONPOINT) != 0) {

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

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