Merge pull request #8 from glynos/http-client-refactoring

deanberris · deanberris · commit 477c253e6481 · 2011-10-23T16:50:08.000-07:00
Http client refactoring
diff --git a/boost/network/message/message.ipp b/boost/network/message/message.ipp
@@ -135,6 +135,7 @@ message::message(message const & other)
 
 message& message::operator=(message other) {
   *pimpl = *other.pimpl;
+  return *this;
 }
 
 message::~message() {
diff --git a/libs/network/src/CMakeLists.txt b/libs/network/src/CMakeLists.txt
@@ -25,5 +25,5 @@ add_library(cppnetlib-message-directives ${CPP-NETLIB_MESSAGE_DIRECTIVES_SRCS})
 set(CPP-NETLIB_MESSAGE_WRAPPERS_SRCS message/wrappers.cpp)
 add_library(cppnetlib-message-wrappers ${CPP-NETLIB_MESSAGE_WRAPPERS_SRCS})
 
-set(CPP-NETLIB_HTTP_MESSAGE_SRCS http/request.cpp http/response.cpp)
-add_library(cppnetlib-http-message ${CPP-NETLIB_HTTP_MESSAGE_SRCS})
+#set(CPP-NETLIB_HTTP_MESSAGE_SRCS http/request.cpp http/response.cpp)
+#add_library(cppnetlib-http-message ${CPP-NETLIB_HTTP_MESSAGE_SRCS})
diff --git a/libs/network/test/message_test.cpp b/libs/network/test/message_test.cpp
@@ -67,6 +67,6 @@ BOOST_AUTO_TEST_CASE(remove_header_directive_test) {
     message instance;
     instance << header("name", "value")
         << remove_header("name");
-    message::headers_range range = headers(instance);
+    message::headers_range range = headers(instance)["name"];
     BOOST_CHECK ( boost::begin(range) == boost::end(range) );
 }

Original file line number	Diff line number	Diff line change
`@@ -135,6 +135,7 @@ message::message(message const & other)`
`135`	`135`
`136`	`136`	`message& message::operator=(message other) {`
`137`	`137`	`pimpl = other.pimpl;`
	`138`	`+ return *this;`
`138`	`139`	`}`
`139`	`140`
`140`	`141`	`message::~message() {`
Original file line number	Diff line number	Diff line change
`@@ -67,6 +67,6 @@ BOOST_AUTO_TEST_CASE(remove_header_directive_test) {`
`67`	`67`	`message instance;`
`68`	`68`	`instance << header("name", "value")`
`69`	`69`	`<< remove_header("name");`
`70`		`- message::headers_range range = headers(instance);`
	`70`	`+ message::headers_range range = headers(instance)["name"];`
`71`	`71`	`BOOST_CHECK ( boost::begin(range) == boost::end(range) );`
`72`	`72`	`}`