Made some "unset" checks explicit.

diff --git a/src/HttpReply.cc b/src/HttpReply.cc
index 92d61ebc48dbe80b813a9e50b17385167f118c75..6ff6694561961878bad40b07dca4e203cf31914f 100644 (file)
--- a/src/HttpReply.cc
+++ b/src/HttpReply.cc
@@ -334,7 +334,7 @@ HttpReply::hdrExpirationTime()
             if (cache_control->getSMaxAge() != HttpHdrCc::S_MAXAGE_UNSET)
                 return date + cache_control->getSMaxAge();
 
-            if (cache_control->getMaxAge() >= 0)
+            if (cache_control->getMaxAge() != HttpHdrCc::MAX_AGE_UNSET)
                 return date + cache_control->getMaxAge();
         } else {
             /*
@@ -342,10 +342,10 @@ HttpReply::hdrExpirationTime()
              * header, but no Date for reference?
              */
 
-            if (cache_control->getSMaxAge()!=HttpHdrCc::S_MAXAGE_UNSET)
+            if (cache_control->getSMaxAge() != HttpHdrCc::S_MAXAGE_UNSET)
                 return squid_curtime;
 
-            if (cache_control->getMaxAge() >= 0)
+            if (cache_control->getMaxAge() != HttpHdrCc::MAX_AGE_UNSET)
                 return squid_curtime;
         }
     }

diff --git a/src/refresh.cc b/src/refresh.cc
index abb79c36ce2ba4ab1c24808555e9770cc8aaff4c..fa603a914438792cc5e08da1b1bfa4e71ab46ad4 100644 (file)
--- a/src/refresh.cc
+++ b/src/refresh.cc
@@ -336,7 +336,7 @@ refreshCheck(const StoreEntry * entry, HttpRequest * request, time_t delta)
 
 #endif
         if (NULL != cc) {
-            if (cc->getMaxAge() >= 0) {
+            if (cc->getMaxAge() != HttpHdrCc::MAX_AGE_UNSET) {
 #if USE_HTTP_VIOLATIONS
                 if (R->flags.ignore_reload && cc->getMaxAge() == 0) {
                     debugs(22, 3, "refreshCheck: MAYBE: client-max-age = 0 and ignore-reload");