svn commit: r16500 - trunk/src/argouml-app/src/org/argouml/notation/providers/uml/AbstractMessageNotationUml.java

Author: penyaskito
Date: 2009-01-04 06:52:16-0800
New Revision: 16500

Modified:
   trunk/src/argouml-app/src/org/argouml/notation/providers/uml/AbstractMessageNotationUml.java

Log:
Style: Foreach loop.

Modified: trunk/src/argouml-app/src/org/argouml/notation/providers/uml/AbstractMessageNotationUml.java
Url: http://argouml.tigris.org/source/browse/argouml/trunk/src/argouml-app/src/org/argouml/notation/providers/uml/AbstractMessageNotationUml.java?view=diff&pathrev=16500&r1=16499&r2=16500
==============================================================================
--- trunk/src/argouml-app/src/org/argouml/notation/providers/uml/AbstractMessageNotationUml.java	(original)
+++ trunk/src/argouml-app/src/org/argouml/notation/providers/uml/AbstractMessageNotationUml.java	2009-01-04 06:52:16-0800
@@ -174,13 +174,11 @@
 
     protected int countSuccessors(Object message) {
         int count = 0;
-        Object activator = Model.getFacade().getActivator(message);
-        Collection successors = Model.getFacade().getSuccessors(message);
+        final Object activator = Model.getFacade().getActivator(message);
+        final Collection successors = Model.getFacade().getSuccessors(message);
         if (successors != null) {
-            Iterator it = successors.iterator();
-            while (it.hasNext()) {
-                Object msg = it.next();
-                if (Model.getFacade().getActivator(msg) != activator) {
+            for (Object suc : successors) {
+                if (Model.getFacade().getActivator(suc) != activator) {
                     continue;
                 }
                 count++;
@@ -1267,13 +1265,11 @@
      * Finds the messages in Collection c that has message a as activator.
      */
     private Collection filterWithActivator(Collection c, Object/*MMessage*/a) {
-        Iterator it = c.iterator();
         List v = new ArrayList();
-        while (it.hasNext()) {
-            Object m = /* (MMessage) */it.next();
-            if (Model.getFacade().getActivator(m) == a) {
-                v.add(m);
-            }
+        for (Object msg : c) {
+            if (Model.getFacade().getActivator(msg) == a) {
+                v.add(msg);
+            }            
         }
         return v;
     }

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

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