You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@tapestry.apache.org by jo...@apache.org on 2011/02/03 02:46:30 UTC
svn commit: r1066720 -
/tapestry/tapestry5/trunk/tapestry-jmx/src/main/java/org/apache/tapestry5/internal/jmx/MBeanSupportImpl.java
Author: joshcanfield
Date: Thu Feb 3 01:46:29 2011
New Revision: 1066720
URL: http://svn.apache.org/viewvc?rev=1066720&view=rev
Log:
Fix for TAP5-1422 - pulled the registeredBeans into another data structure to iterate over so we can remove them from the set.
Modified:
tapestry/tapestry5/trunk/tapestry-jmx/src/main/java/org/apache/tapestry5/internal/jmx/MBeanSupportImpl.java
Modified: tapestry/tapestry5/trunk/tapestry-jmx/src/main/java/org/apache/tapestry5/internal/jmx/MBeanSupportImpl.java
URL: http://svn.apache.org/viewvc/tapestry/tapestry5/trunk/tapestry-jmx/src/main/java/org/apache/tapestry5/internal/jmx/MBeanSupportImpl.java?rev=1066720&r1=1066719&r2=1066720&view=diff
==============================================================================
--- tapestry/tapestry5/trunk/tapestry-jmx/src/main/java/org/apache/tapestry5/internal/jmx/MBeanSupportImpl.java (original)
+++ tapestry/tapestry5/trunk/tapestry-jmx/src/main/java/org/apache/tapestry5/internal/jmx/MBeanSupportImpl.java Thu Feb 3 01:46:29 2011
@@ -109,8 +109,9 @@ public class MBeanSupportImpl implements
public void registryDidShutdown()
{
lock.lock();
-
- for (final ObjectName name : this.registeredBeans)
+ // store into new data structure so we can remove them from registered beans
+ ObjectName[] objects = registeredBeans.toArray(new ObjectName[registeredBeans.size()]);
+ for (final ObjectName name : objects)
{
doUnregister(name);
}