]> git.ipfire.org Git - thirdparty/squid.git/commitdiff
SourceFormat Enforcement
authorAutomatic source maintenance <squidadm@squid-cache.org>
Sun, 17 Jun 2012 01:19:07 +0000 (19:19 -0600)
committerAutomatic source maintenance <squidadm@squid-cache.org>
Sun, 17 Jun 2012 01:19:07 +0000 (19:19 -0600)
src/acl/Acl.cc
src/acl/Checklist.cc
src/acl/Checklist.h
src/external_acl.cc
src/ident/AclIdent.cc

index 3172e7ac4438572b7321d9ace178ed8afd1cb275..36d7db7d88b51c575dec84f9df753490d9e65203 100644 (file)
@@ -323,7 +323,7 @@ ACLList::matches (ACLChecklist *checklist) const
 {
     assert (_acl);
     // XXX: AclMatchedName does not contain a matched ACL name when the acl
-    // does not match (or contains stale name if no ACLs are checked). In 
+    // does not match (or contains stale name if no ACLs are checked). In
     // either case, we get misleading debugging and possibly incorrect error
     // messages. Unfortunately, deny_info's "when none http_access
     // lines match" exception essentially requires this mess.
index f1a45be51727b8b7ed6ede5af6dbef561fedb655..fe6997d3fb7bc0126b5c3c481801e08e29fcd1b1 100644 (file)
@@ -218,7 +218,7 @@ ACLChecklist::matchNodes(const ACLList * head, bool const fast)
 
         if (resultBeforeAsync == nmrMatch)
             continue;
-        
+
         if (resultBeforeAsync == nmrMismatch || resultBeforeAsync == nmrFinished)
             return false;
 
@@ -236,7 +236,7 @@ ACLChecklist::matchNodes(const ACLList * head, bool const fast)
         // This is inefficient and ugly, but fixing all match() code, including
         // the code it calls, such as ipcache_nbgethostbyname(), takes time.
         if (!asyncInProgress()) { // failed to start an async operation
-            
+
             if (finished()) {
                 debugs(28, 3, HERE << this << " finished after failing to go async: " << currentAnswer());
                 return false; // an exceptional case
@@ -294,7 +294,7 @@ ACLChecklist::matchNode(const ACLList &node, bool const fast)
         assert(!needsAsync);
         debugs(28, 3, HERE << this << " exception: " << currentAnswer());
         return nmrFinished;
-     }
+    }
 
     if (!needsAsync) {
         debugs(28, 3, HERE << this << " simple mismatch");
@@ -395,8 +395,7 @@ ACLChecklist::fastCheck(const ACLList * list)
     // assume DENY/ALLOW on mis/matches due to not having acl_access object
     if (matchAclList(list, true))
         markFinished(ACCESS_ALLOWED, "all ACLs matched");
-    else
-    if (!finished())
+    else if (!finished())
         markFinished(ACCESS_DENIED, "ACL mismatched");
     PROF_stop(aclCheckFast);
     return currentAnswer();
@@ -442,7 +441,7 @@ ACLChecklist::fastCheck()
 }
 
 /// When no rules matched, the answer is the inversion of the last seen rule
-/// action (or ACCESS_DUNNO if the reversal is not possible). The caller 
+/// action (or ACCESS_DUNNO if the reversal is not possible). The caller
 /// should set lastSeenAction to ACCESS_DUNNO if there were no rules to see.
 void
 ACLChecklist::calcImplicitAnswer(const allow_t &lastSeenAction)
index e7e4c5e382deb052fb938b70bb2c52311c13706e..51a2d4ae9a7e4cce16f3b0ee0ded03ed7f3ef26c 100644 (file)
@@ -205,7 +205,7 @@ public:
 private: /* internal methods */
     /// possible outcomes when trying to match a single ACL node in a list
     typedef enum { nmrMatch, nmrMismatch, nmrFinished, nmrNeedsAsync }
-        NodeMatchingResult;
+    NodeMatchingResult;
 
     /// prepare for checking ACLs; called once per check
     void preCheck(const char *what);
index fd160d9566a822ba1ed964fb1216449c27d4cc96..3c51377ca455e446f23ea211c1fa2ba5403a04c7 100644 (file)
@@ -1398,7 +1398,7 @@ void
 ExternalACLLookup::Start(ACLChecklist *checklist, external_acl_data *acl, bool inBackground)
 {
     external_acl *def = acl->def;
+
     ACLFilledChecklist *ch = Filled(checklist);
     const char *key = makeExternalAclKey(ch, acl);
     assert(key);
index bc8f95fe6f97696e1cc7291186edd597f56ee52d..7b4a0d29ff20c10058cca57c8950e0c1cb7956ea 100644 (file)
@@ -134,9 +134,9 @@ IdentLookup::checkForAsync(ACLChecklist *cl)const
     const ConnStateData *conn = checklist->conn();
     // check that ACLIdent::match() tested this lookup precondition
     assert(conn && Comm::IsConnOpen(conn->clientConnection));
-        debugs(28, 3, HERE << "Doing ident lookup" );
-        checklist->asyncInProgress(true);
-        Ident::Start(checklist->conn()->clientConnection, LookupDone, checklist);
+    debugs(28, 3, HERE << "Doing ident lookup" );
+    checklist->asyncInProgress(true);
+    Ident::Start(checklist->conn()->clientConnection, LookupDone, checklist);
 }
 
 void