Enforced constness of headers getters in http_response

Sebastiano Merlino · Sebastiano Merlino · commit 4625f6568bba · 2014-02-22T09:39:21.000Z
diff --git a/src/http_response.cpp b/src/http_response.cpp
@@ -39,19 +39,19 @@ http_response::~http_response()
         webserver::unlock_cache_entry(ce);
 }
 
-size_t http_response::get_headers(std::map<std::string, std::string, header_comparator>& result)
+size_t http_response::get_headers(std::map<std::string, std::string, header_comparator>& result) const
 {
     result = this->headers;
     return result.size();
 }
 
-size_t http_response::get_footers(std::map<std::string, std::string, header_comparator>& result)
+size_t http_response::get_footers(std::map<std::string, std::string, header_comparator>& result) const
 {
     result = this->footers;
     return result.size();
 }
 
-size_t http_response::get_cookies(std::map<std::string, std::string, header_comparator>& result)
+size_t http_response::get_cookies(std::map<std::string, std::string, header_comparator>& result) const
 {
     result = this->cookies;
     return result.size();
diff --git a/src/httpserver/http_response.hpp b/src/httpserver/http_response.hpp
@@ -324,17 +324,17 @@ class http_response
         **/
         size_t get_headers(
                 std::map<std::string, std::string, header_comparator>& result
-        );
+        ) const;
         /**
          * Method used to get all footers passed with the request.
          * @return a map<string,string> containing all footers.
         **/
         size_t get_footers(
                 std::map<std::string, std::string, header_comparator>& result
-        );
+        ) const;
         size_t get_cookies(
                 std::map<std::string, std::string, header_comparator>& result
-        );
+        ) const;
         /**
          * Method used to set all headers of the response.
          * @param headers The headers key-value map to set for the response.

Original file line number	Diff line number	Diff line change
`@@ -39,19 +39,19 @@ http_response::~http_response()`
`39`	`39`	`webserver::unlock_cache_entry(ce);`
`40`	`40`	`}`
`41`	`41`
`42`		`-size_t http_response::get_headers(std::map<std::string, std::string, header_comparator>& result)`
	`42`	`+size_t http_response::get_headers(std::map<std::string, std::string, header_comparator>& result) const`
`43`	`43`	`{`
`44`	`44`	`result = this->headers;`
`45`	`45`	`return result.size();`
`46`	`46`	`}`
`47`	`47`
`48`		`-size_t http_response::get_footers(std::map<std::string, std::string, header_comparator>& result)`
	`48`	`+size_t http_response::get_footers(std::map<std::string, std::string, header_comparator>& result) const`
`49`	`49`	`{`
`50`	`50`	`result = this->footers;`
`51`	`51`	`return result.size();`
`52`	`52`	`}`
`53`	`53`
`54`		`-size_t http_response::get_cookies(std::map<std::string, std::string, header_comparator>& result)`
	`54`	`+size_t http_response::get_cookies(std::map<std::string, std::string, header_comparator>& result) const`
`55`	`55`	`{`
`56`	`56`	`result = this->cookies;`
`57`	`57`	`return result.size();`