From: Alex Rousskov <rousskov@measurement-factory.com>
Date: Fri, 4 May 2012 22:20:07 +0000 (-0600)
Subject: ACL Optimization: inline currentAnswer() methods to reduce copying
X-Git-Tag: BumpSslServerFirst.take08~9
X-Git-Url: http://git.ipfire.org/cgi-bin/gitweb.cgi?a=commitdiff_plain;h=0229d5802b4aaccb70ecf56fdcc9195aeb64bb2a;p=thirdparty%2Fsquid.git

ACL Optimization: inline currentAnswer() methods to reduce copying
of answer objects which now contain two integers instead of just one.
---

diff --git a/src/acl/Checklist.cc b/src/acl/Checklist.cc
index 496dbf9228..aef2bae626 100644
--- a/src/acl/Checklist.cc
+++ b/src/acl/Checklist.cc
@@ -36,18 +36,6 @@
 #include "squid-old.h"
 #include "acl/Checklist.h"
 
-allow_t const &
-ACLChecklist::currentAnswer() const
-{
-    return allow_;
-}
-
-void
-ACLChecklist::currentAnswer(allow_t const newAnswer)
-{
-    allow_ = newAnswer;
-}
-
 void
 ACLChecklist::matchNonBlocking()
 {
diff --git a/src/acl/Checklist.h b/src/acl/Checklist.h
index 33b81e0141..98a79bb031 100644
--- a/src/acl/Checklist.h
+++ b/src/acl/Checklist.h
@@ -131,8 +131,8 @@ public:
     bool finished() const;
     void markFinished();
 
-    allow_t const & currentAnswer() const;
-    void currentAnswer(allow_t const);
+    const allow_t &currentAnswer() const { return allow_; }
+    void currentAnswer(const allow_t &newAnswer) { allow_ = newAnswer; }
 
     void changeState(AsyncState *);
     AsyncState *asyncState() const;