Return const reference from get functions

demozon · demozon · commit 7099700af7de · 2011-05-09T00:04:34.000-07:00
The const versions of destination(), source(), body()
and headers() returned either a temporary or a non const
reference. Changed the to return a const references.
This makes them work with boost::iterator_ranges.
diff --git a/boost/network/message.hpp b/boost/network/message.hpp
@@ -83,7 +83,7 @@ namespace boost { namespace network {
             _headers.erase(key);
         }
 
-        headers_container_type & headers() const {
+        headers_container_type const & headers() const {
             return _headers;
         }
 
@@ -95,7 +95,7 @@ namespace boost { namespace network {
             _body = body_;
         }
 
-        string_type body() const {
+        string_type const & body() const {
             return _body;
         }
         
@@ -107,7 +107,7 @@ namespace boost { namespace network {
             _source = source_;
         }
 
-        string_type source() const {
+        string_type const & source() const {
             return _source;
         }
 
@@ -119,7 +119,7 @@ namespace boost { namespace network {
             _destination = destination_;
         }
 
-        string_type destination() const {
+        string_type const & destination() const {
             return _destination;
         }
 

Original file line number	Diff line number	Diff line change
`@@ -83,7 +83,7 @@ namespace boost { namespace network {`
`83`	`83`	`_headers.erase(key);`
`84`	`84`	`}`
`85`	`85`
`86`		`- headers_container_type & headers() const {`
	`86`	`+ headers_container_type const & headers() const {`
`87`	`87`	`return _headers;`
`88`	`88`	`}`
`89`	`89`
`@@ -95,7 +95,7 @@ namespace boost { namespace network {`
`95`	`95`	`_body = body_;`
`96`	`96`	`}`
`97`	`97`
`98`		`- string_type body() const {`
	`98`	`+ string_type const & body() const {`
`99`	`99`	`return _body;`
`100`	`100`	`}`
`101`	`101`
`@@ -107,7 +107,7 @@ namespace boost { namespace network {`
`107`	`107`	`_source = source_;`
`108`	`108`	`}`
`109`	`109`
`110`		`- string_type source() const {`
	`110`	`+ string_type const & source() const {`
`111`	`111`	`return _source;`
`112`	`112`	`}`
`113`	`113`
`@@ -119,7 +119,7 @@ namespace boost { namespace network {`
`119`	`119`	`_destination = destination_;`
`120`	`120`	`}`
`121`	`121`
`122`		`- string_type destination() const {`
	`122`	`+ string_type const & destination() const {`
`123`	`123`	`return _destination;`
`124`	`124`	`}`
`125`	`125`