svn commit: r16853 - trunk/src/argouml-core-model-mdr/src/org/argouml/model/mdr/FacadeMDRImpl.java

Author: bobtarling
Date: 2009-03-04 02:01:20-0800
New Revision: 16853

Modified:
   trunk/src/argouml-core-model-mdr/src/org/argouml/model/mdr/FacadeMDRImpl.java

Log:
Issue 5712: Limit the gap for a possible thread clash accessing and changing predecessors. Not a full resolution for this problem.

Modified: trunk/src/argouml-core-model-mdr/src/org/argouml/model/mdr/FacadeMDRImpl.java
Url: http://argouml.tigris.org/source/browse/argouml/trunk/src/argouml-core-model-mdr/src/org/argouml/model/mdr/FacadeMDRImpl.java?view=diff&pathrev=16853&r1=16852&r2=16853
==============================================================================
--- trunk/src/argouml-core-model-mdr/src/org/argouml/model/mdr/FacadeMDRImpl.java	(original)
+++ trunk/src/argouml-core-model-mdr/src/org/argouml/model/mdr/FacadeMDRImpl.java	2009-03-04 02:01:20-0800
@@ -2851,7 +2851,9 @@
     public Collection<Message> getPredecessors(Object handle) {
         try {
             if (handle instanceof Message) {
-                return ((Message) handle).getPredecessor();
+                return Collections.unmodifiableCollection(
+                        new ArrayList<Message>(
+                                ((Message) handle).getPredecessor()));
             }
         } catch (InvalidObjectException e) {
             throw new InvalidElementException(e);

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

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