You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@myfaces.apache.org by im...@apache.org on 2007/03/01 08:52:39 UTC

svn commit: r513217 - /myfaces/fusion/trunk/core/src/main/java/org/apache/myfaces/fusion/conversation/Conversation.java

Author: imario
Date: Wed Feb 28 23:52:38 2007
New Revision: 513217

URL: http://svn.apache.org/viewvc?view=rev&rev=513217
Log:
fixed concurModEx

Modified:
    myfaces/fusion/trunk/core/src/main/java/org/apache/myfaces/fusion/conversation/Conversation.java

Modified: myfaces/fusion/trunk/core/src/main/java/org/apache/myfaces/fusion/conversation/Conversation.java
URL: http://svn.apache.org/viewvc/myfaces/fusion/trunk/core/src/main/java/org/apache/myfaces/fusion/conversation/Conversation.java?view=diff&rev=513217&r1=513216&r2=513217
==============================================================================
--- myfaces/fusion/trunk/core/src/main/java/org/apache/myfaces/fusion/conversation/Conversation.java (original)
+++ myfaces/fusion/trunk/core/src/main/java/org/apache/myfaces/fusion/conversation/Conversation.java Wed Feb 28 23:52:38 2007
@@ -22,7 +22,6 @@
 import org.apache.commons.logging.Log;
 import org.apache.commons.logging.LogFactory;
 
-import java.util.Iterator;
 import java.util.Map;
 import java.util.TreeMap;
 
@@ -222,13 +221,11 @@
 
 		synchronized(this)
 		{
-			Iterator iterBeans = beans.keySet().iterator();
-			while (iterBeans.hasNext())
+			String[] beanNames = (String[]) beans.keySet().toArray(new String[beans.size()]);
+			for (int i = 0; i< beanNames.length; i++)
 			{
-				String name = (String) iterBeans.next();
-				removeAttribute(name);
+				removeAttribute(beanNames[i]);
 			}
-			beans.clear();
 		}
 
 		conversationContext.removeConversation(getName());