svn commit: r12599 - trunk/src_new/org/argouml/uml/diagram/sequence/ui/UMLSequenceDiagram.java

Author: tfmorris
Date: 2007-05-11 10:42:28-0700
New Revision: 12599

Modified:
   trunk/src_new/org/argouml/uml/diagram/sequence/ui/UMLSequenceDiagram.java

Log:
Remove unused import and clean up warnings

Modified: trunk/src_new/org/argouml/uml/diagram/sequence/ui/UMLSequenceDiagram.java
Url: http://argouml.tigris.org/source/browse/argouml/trunk/src_new/org/argouml/uml/diagram/sequence/ui/UMLSequenceDiagram.java?view=diff&rev=12599&p1=trunk/src_new/org/argouml/uml/diagram/sequence/ui/UMLSequenceDiagram.java&p2=trunk/src_new/org/argouml/uml/diagram/sequence/ui/UMLSequenceDiagram.java&r1=12598&r2=12599
==============================================================================
--- trunk/src_new/org/argouml/uml/diagram/sequence/ui/UMLSequenceDiagram.java	(original)
+++ trunk/src_new/org/argouml/uml/diagram/sequence/ui/UMLSequenceDiagram.java	2007-05-11 10:42:28-0700
@@ -33,7 +33,6 @@
 import org.argouml.model.Model;
 import org.argouml.ui.ActionSetMode;
 import org.argouml.uml.diagram.sequence.SequenceDiagramGraphModel;
-import org.argouml.uml.diagram.static_structure.ClassDiagramGraphModel;
 import org.argouml.uml.diagram.ui.RadioAction;
 import org.argouml.uml.diagram.ui.UMLDiagram;
 
@@ -55,11 +54,11 @@
 
     /**
      * Constructs a new sequence diagram with a default name and NO namespace.
-     * namespaces are used to determine the 'owner' of the diagram for diagrams
+     * Namespaces are used to determine the 'owner' of the diagram for diagrams
      * but that's plain misuse.
      */
     public UMLSequenceDiagram() {
-        // TODO: All super constrcutors should take a GraphModel
+        // TODO: All super constructors should take a GraphModel
         super();
         // Dirty hack to remove the trash the Diagram constructor leaves
         SequenceDiagramGraphModel gm =
@@ -89,9 +88,8 @@
         setNamespace(collaboration); //See issue 3373.
     }
 
-    /*
-     * @see org.argouml.uml.diagram.ui.UMLDiagram#getOwner()
-     */
+
+    @Override
     public Object getOwner() {
         return getNamespace();
     }
@@ -110,9 +108,8 @@
         return name;
     }
 
-    /*
-     * @see org.argouml.uml.diagram.ui.UMLDiagram#getLabelName()
-     */
+
+    @Override
     public String getLabelName() {
         return Translator.localize("label.sequence-diagram");
     }
@@ -153,8 +150,9 @@
                     "button.new-destroyaction", },
             };
 
+            Hashtable<String, Object> args = new Hashtable<String, Object>();
 	    for (int i = 0; i < actionList.length; i++) {
-		Hashtable args = new Hashtable();
+                args.clear();
                 args.put("edgeClass", Model.getMetaTypes().getMessage());
 		args.put("action", actionList[i][0]);
                 args.put("actionName",
@@ -165,13 +163,14 @@
                         ModeCreateMessage.class, args,
                         (String) actionList[i][1]));
 	    }
-            Hashtable args = new Hashtable();
+
+            args.clear();
             args.put("name", SEQUENCE_EXPAND_BUTTON);
             actions[5] =
 		new RadioAction(new ActionSetMode(ModeChangeHeight.class,
 					       args,
 					       SEQUENCE_EXPAND_BUTTON));
-            args = new Hashtable();
+            args.clear();
             args.put("name", SEQUENCE_CONTRACT_BUTTON);
             actions[6] =
 		new RadioAction(new ActionSetMode(ModeChangeHeight.class,
@@ -181,16 +180,14 @@
         return actions;
     }
 
-    /*
-     * @see org.argouml.uml.diagram.ui.UMLDiagram#getNamespace()
-     */
+
+    @Override
     public Object getNamespace() {
         return ((SequenceDiagramGraphModel) getGraphModel()).getCollaboration();
     }
 
-    /*
-     * @see org.argouml.uml.diagram.ui.UMLDiagram#setNamespace(java.lang.Object)
-     */
+
+    @Override
     public void setNamespace(Object ns) {
         ((SequenceDiagramGraphModel) getGraphModel()).setCollaboration(ns);
         super.setNamespace(ns);
@@ -206,10 +203,8 @@
 */
     }
 
-    /*
-     * @see org.argouml.uml.diagram.ui.UMLDiagram#isRelocationAllowed(
-     *         java.lang.Object)
-     */
+    
+    @Override
     public boolean isRelocationAllowed(Object base)  {
     	return false;
 	/* TODO: We may return the following when the
@@ -219,11 +214,10 @@
 //        	|| Model.getFacade().isAOperation(base);
     }
 
-    /*
-     * @see org.argouml.uml.diagram.ui.UMLDiagram#relocate(java.lang.Object)
-     */
+
+    @Override
     public boolean relocate(Object base) {
         return false;
     }
 
-} /* end class UMLSequenceDiagram */
+}