Add comment explaining why there is 'break' in the loop that looks for

candidate classes during ACL checks.

diff --git a/src/ICAP/ICAPConfig.cc b/src/ICAP/ICAPConfig.cc
index f98f864160fd1c9d129e80b7210e47f79b8e6616..6925f02975c91c4f669d1d9342519e3ff31ee027 100644 (file)
--- a/src/ICAP/ICAPConfig.cc
+++ b/src/ICAP/ICAPConfig.cc
@@ -1,6 +1,6 @@
 
 /*
- * $Id: ICAPConfig.cc,v 1.2 2005/11/21 23:46:27 wessels Exp $
+ * $Id: ICAPConfig.cc,v 1.3 2005/12/01 22:39:46 wessels Exp $
  *
  * SQUID Web Proxy Cache          http://www.squid-cache.org/
  * ----------------------------------------------------------
@@ -159,7 +159,14 @@ ICAPAccessCheck::check()
 
             candidateClasses += theClass->key;
 
+            /*
+             * Break here because we only need one matching service
+             * to justify ACL-checking a class.  We might use other
+             * services belonging to the class if the first service
+             * is unavailable, etc.
+             */
             break;
+
         }
     }