You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@mina.apache.org by tr...@apache.org on 2007/11/14 13:03:08 UTC

svn commit: r594846 - /mina/trunk/core/src/main/java/org/apache/mina/common/DefaultIoFilterChainBuilder.java

Author: trustin
Date: Wed Nov 14 04:03:07 2007
New Revision: 594846

URL: http://svn.apache.org/viewvc?rev=594846&view=rev
Log:
Small optimization and more detailed logging

Modified:
    mina/trunk/core/src/main/java/org/apache/mina/common/DefaultIoFilterChainBuilder.java

Modified: mina/trunk/core/src/main/java/org/apache/mina/common/DefaultIoFilterChainBuilder.java
URL: http://svn.apache.org/viewvc/mina/trunk/core/src/main/java/org/apache/mina/common/DefaultIoFilterChainBuilder.java?rev=594846&r1=594845&r2=594846&view=diff
==============================================================================
--- mina/trunk/core/src/main/java/org/apache/mina/common/DefaultIoFilterChainBuilder.java (original)
+++ mina/trunk/core/src/main/java/org/apache/mina/common/DefaultIoFilterChainBuilder.java Wed Nov 14 04:03:07 2007
@@ -424,20 +424,20 @@
             
             newMap.put(filterName, dummyFilter);
             expectedNames.add(filterName);
-        }
-        
-        Iterator<String> i = expectedNames.iterator();
-        for (Object key: newMap.keySet()) {
-            if (!i.next().equals(key)) {
-                if (logger.isDebugEnabled()) {
-                    logger.debug(
-                            "The specified map didn't pass the insertion " +
-                            "order test.");
+
+            Iterator<String> it = expectedNames.iterator();
+            for (Object key: newMap.keySet()) {
+                if (!it.next().equals(key)) {
+                    if (logger.isDebugEnabled()) {
+                        logger.debug(
+                                "The specified map didn't pass the insertion " +
+                                "order test after " + (i + 1) + " tries.");
+                    }
+                    return false;
                 }
-                return false;
             }
         }
-
+        
         if (logger.isDebugEnabled()) {
             logger.debug(
                     "The specified map passed the insertion order test.");