Author: eric
Date: Mon Apr  2 09:59:02 2012
New Revision: 1308278

URL: http://svn.apache.org/viewvc?rev=1308278&view=rev
Log:
Nest level counting for matcher mbean name - Patch provided by TzeKai Lee 
(JAMES-1397)

Modified:
    
james/server/trunk/mailetcontainer-camel/src/main/java/org/apache/james/mailetcontainer/impl/jmx/JMXStateMailetProcessorListener.java

Modified: 
james/server/trunk/mailetcontainer-camel/src/main/java/org/apache/james/mailetcontainer/impl/jmx/JMXStateMailetProcessorListener.java
URL: 
http://svn.apache.org/viewvc/james/server/trunk/mailetcontainer-camel/src/main/java/org/apache/james/mailetcontainer/impl/jmx/JMXStateMailetProcessorListener.java?rev=1308278&r1=1308277&r2=1308278&view=diff
==============================================================================
--- 
james/server/trunk/mailetcontainer-camel/src/main/java/org/apache/james/mailetcontainer/impl/jmx/JMXStateMailetProcessorListener.java
 (original)
+++ 
james/server/trunk/mailetcontainer-camel/src/main/java/org/apache/james/mailetcontainer/impl/jmx/JMXStateMailetProcessorListener.java
 Mon Apr  2 09:59:02 2012
@@ -138,22 +138,20 @@ public class JMXStateMailetProcessorList
      */
     @SuppressWarnings("unchecked")
     private void registerMatchers(String parentMBeanName, Iterator<Matcher> 
matchers, int nestingLevel) throws MalformedObjectNameException, JMException {
-        // current level
-        int currentLevel = nestingLevel;
         int i = 0;
 
         while (matchers.hasNext()) {
             Matcher matcher = matchers.next();
             MatcherManagement matcherManagement = new 
MatcherManagement(matcher.getMatcherConfig());
 
-            String matcherMBeanName = parentMBeanName + ",subtype" + 
currentLevel + "=matcher,index" + currentLevel + "=" + (i++) + ",matchername" + 
currentLevel + "=" + matcherManagement.getMatcherName();
+            String matcherMBeanName = parentMBeanName + ",subtype" + 
nestingLevel + "=matcher,index" + nestingLevel + "=" + (i++) + ",matchername" + 
nestingLevel + "=" + matcherManagement.getMatcherName();
             registerMBean(matcherMBeanName, matcherManagement);
             matcherMap.put(matcher, matcherManagement);
             // Handle CompositeMatcher which were added by JAMES-948
             if (matcher instanceof CompositeMatcher) {
                 // we increment the nesting as we have one more child level and
                 // register the child matchers
-                registerMatchers(matcherMBeanName, ((CompositeMatcher) 
matcher).iterator(), ++nestingLevel);
+                registerMatchers(matcherMBeanName, ((CompositeMatcher) 
matcher).iterator(), nestingLevel + 1);
             }
 
         }



---------------------------------------------------------------------
To unsubscribe, e-mail: server-dev-unsubscr...@james.apache.org
For additional commands, e-mail: server-dev-h...@james.apache.org

Reply via email to