]> git.ipfire.org Git - thirdparty/squid.git/commitdiff
Remove Vector::operator +=. Removed some useless comments
authorFrancesco Chemolli <kinkie@squid-cache.org>
Tue, 4 Feb 2014 19:47:14 +0000 (20:47 +0100)
committerFrancesco Chemolli <kinkie@squid-cache.org>
Tue, 4 Feb 2014 19:47:14 +0000 (20:47 +0100)
src/DiskIO/DiskIOModule.cc
src/HttpHeader.cc
src/StoreFileSystem.cc
src/adaptation/Service.cc
src/adaptation/icap/ServiceRep.cc
src/base/Vector.h
src/errorpage.cc

index c72d290b2423cda3c1f4121b16849221107ec1b3..2992f2237dd0f0acebebc6563fd052481c110a97 100644 (file)
@@ -92,7 +92,6 @@ DiskIOModule::GetModules()
 void
 DiskIOModule::FreeAllModules()
 {
-    // TODO fix antipattern: walk the vector and then clear in one go
     while (!GetModules().empty()) {
         DiskIOModule *fs = GetModules().back();
         GetModules().pop_back();
index ff441802c47c14e35f0aaa199bcc5dcd9616687c..f0f3e454ea8d25bbaa1dd7fdc72ec88be583b838 100644 (file)
@@ -748,11 +748,11 @@ HttpHeaderEntry *
 HttpHeader::getEntry(HttpHeaderPos * pos) const
 {
     assert(pos);
-    assert(*pos >= HttpHeaderInitPos && *pos < (ssize_t)entries.size());
+    assert(*pos >= HttpHeaderInitPos && *pos < static_cast<ssize_t>(entries.size()));
 
-    for (++(*pos); *pos < (ssize_t)entries.size(); ++(*pos)) {
+    for (++(*pos); *pos < static_cast<ssize_t>(entries.size()); ++(*pos)) {
         if (entries[*pos])
-            return (HttpHeaderEntry*)entries[*pos];
+            return static_cast<HttpHeaderEntry*>(entries[*pos]);
     }
 
     return NULL;
@@ -871,8 +871,8 @@ void
 HttpHeader::delAt(HttpHeaderPos pos, int &headers_deleted)
 {
     HttpHeaderEntry *e;
-    assert(pos >= HttpHeaderInitPos && pos < (ssize_t)entries.size());
-    e = (HttpHeaderEntry*)entries[pos];
+    assert(pos >= HttpHeaderInitPos && pos < static_cast<ssize_t>(entries.size()));
+    e = static_cast<HttpHeaderEntry*>(entries[pos]);
     entries[pos] = NULL;
     /* decrement header length, allow for ": " and crlf */
     len -= e->name.size() + 2 + e->value.size() + 2;
index ad91f70802a3539c7b2f48f277c7e60d4448274c..293c37f4e20108fc934e2e450734c0fa7efbf8d0 100644 (file)
@@ -87,7 +87,6 @@ StoreFileSystem::GetFileSystems()
 void
 StoreFileSystem::FreeAllFs()
 {
-    //TODO fix antipattern: call for all filesystems and clear in one go.
     while (!GetFileSystems().empty()) {
         StoreFileSystem *fs = GetFileSystems().back();
         GetFileSystems().pop_back();
index 8a54e85456782518e1410ec7e843ce1b7129754c..10f1e642e81c54e9d14a8e00fbcc73eb29057c81 100644 (file)
@@ -71,7 +71,6 @@ Adaptation::FindService(const Service::Id& key)
 
 void Adaptation::DetachServices()
 {
-    // TODO fix antipattern: walk the vector and then clear in one go
     while (!AllServices().empty()) {
         AllServices().back()->detach();
         AllServices().pop_back();
index 95d2d68a6b57b41da5980d3c10ff69d43ac3be19..ac295a767e916a0191652b98d7fdd2b725dd060f 100644 (file)
@@ -374,7 +374,6 @@ void Adaptation::Icap::ServiceRep::noteTimeToNotify()
 
     Pointer us = NULL;
 
-    // TODO fix antipattern: walk the vector and then clear in one go
     while (!theClients.empty()) {
         Client i = theClients.back();
         theClients.pop_back();
index d76b5791f7f7154fe52f5c605a44d37479eac069..3e9207017b541b64592f4dbc2398c62dd9abca5e 100644 (file)
@@ -100,7 +100,6 @@ public:
     void clear();
     void reserve (size_t capacity);
     void push_back (E);
-    Vector &operator += (E item) {push_back(item); return *this;};
 
     void insert (E);
     const E &front() const;
index 92ab11c91e369e52ed94cad083467b1da1fe5fe8..cf74c3a449aab35fcd0d62cd6dcadaa561581473 100644 (file)
@@ -60,6 +60,8 @@
 #include "ssl/ErrorDetailManager.h"
 #endif
 
+#include <algorithm>
+
 /**
  \defgroup ErrorPageInternal Error Page Internals
  \ingroup ErrorPageAPI
@@ -247,7 +249,6 @@ errorClean(void)
         safe_free(error_text);
     }
 
-    //TODO fix entipattern: destroy pages and then clear in one go
     while (!ErrorDynamicPages.empty()) {
         errorDynamicPageInfoDestroy(ErrorDynamicPages.back());
         ErrorDynamicPages.pop_back();