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);
         }