Cleaning up warnings related to cpp-netlib code.

deanberris · deanberris · commit 0f692fed514c · 2010-10-12T21:35:42.000+08:00
diff --git a/boost/network/protocol/http/client/facade.hpp b/boost/network/protocol/http/client/facade.hpp
@@ -13,7 +13,7 @@
 namespace boost { namespace network { namespace http {
 
     template <class Tag>
-    class basic_request;
+    struct basic_request;
 
     template <class Tag>
     struct basic_response;
diff --git a/boost/network/protocol/http/impl/request.hpp b/boost/network/protocol/http/impl/request.hpp
@@ -42,7 +42,7 @@ namespace boost { namespace network { namespace http {
     {};
 
     template <class Tag>
-    class basic_request : public request_base<Tag>::type
+    struct basic_request : public request_base<Tag>::type
     {
 
         mutable boost::network::uri::http::uri uri_;
diff --git a/boost/network/protocol/http/message/async_message.hpp b/boost/network/protocol/http/message/async_message.hpp
@@ -24,20 +24,20 @@ namespace boost { namespace network { namespace http {
 
         async_message()
             : status_message_(),
-            status_(),
             version_(),
             source_(),
             destination_(),
+            status_(),
             headers_(),
             body_()
         {}
 
         async_message(async_message const & other)
             : status_message_(other.status_message_),
-            status_(other.status_),
             version_(other.version_),
             source_(other.source_),
             destination_(other.destination_),
+            status_(other.status_),
             headers_(other.headers_),
             body_(other.body_)
         {}
@@ -129,11 +129,12 @@ namespace boost { namespace network { namespace http {
     private:
 
         mutable boost::shared_future<string_type> status_message_,
-            version_, source_, destination_, body_;
+            version_, source_, destination_;
         mutable boost::shared_future<boost::uint16_t> status_;
         mutable boost::shared_future<headers_container_type> headers_;
         mutable headers_container_type added_headers;
         mutable std::set<string_type> removed_headers;
+        mutable boost::shared_future<string_type> body_;
     };
 
     template <class Tag>
diff --git a/boost/network/protocol/http/message/modifiers/uri.hpp b/boost/network/protocol/http/message/modifiers/uri.hpp
@@ -13,7 +13,7 @@
 namespace boost { namespace network { namespace http {
 
     template <class Tag>
-    class basic_request;
+    struct basic_request;
 
     template <class Tag, class T>
     void uri(basic_request<Tag> & request, T const & value) {
diff --git a/boost/network/protocol/http/message/wrappers/anchor.hpp b/boost/network/protocol/http/message/wrappers/anchor.hpp
@@ -13,7 +13,7 @@
 namespace boost { namespace network { namespace http {
 
     template <class Tag>
-    class basic_request;
+    struct basic_request;
 
     namespace impl {
         template <class Tag>
diff --git a/boost/network/protocol/http/message/wrappers/body.hpp b/boost/network/protocol/http/message/wrappers/body.hpp
@@ -16,15 +16,15 @@ namespace boost { namespace network { namespace http {
     struct basic_response;
 
     template <class Tag>
-    class basic_request;
+    struct basic_request;
 
     namespace impl {
 
         template <class Message>
         struct body_wrapper {
             typedef typename string<typename Message::tag>::type string_type;
             Message const & message_;
-            body_wrapper(Message const & message)
+            explicit body_wrapper(Message const & message)
                 : message_(message) {}
             body_wrapper(body_wrapper const & other)
                 : message_(other.message_) {}
diff --git a/boost/network/protocol/http/message/wrappers/destination.hpp b/boost/network/protocol/http/message/wrappers/destination.hpp
@@ -16,7 +16,7 @@ namespace boost { namespace network { namespace http {
     struct basic_response;
 
     template <class Tag>
-    class basic_request;
+    struct basic_request;
 
     namespace impl {
 
diff --git a/boost/network/protocol/http/message/wrappers/headers.hpp b/boost/network/protocol/http/message/wrappers/headers.hpp
@@ -19,7 +19,7 @@ namespace boost { namespace network { namespace http {
     };
 
     template <class Tag>
-    class basic_request;
+    struct basic_request;
 
     template <class Tag>
     struct basic_response;
diff --git a/boost/network/protocol/http/message/wrappers/host.hpp b/boost/network/protocol/http/message/wrappers/host.hpp
@@ -13,7 +13,7 @@
 namespace boost { namespace network { namespace http {
 
     template <class Tag>
-    class basic_request;
+    struct basic_request;
 
     namespace impl {
 
diff --git a/boost/network/protocol/http/message/wrappers/path.hpp b/boost/network/protocol/http/message/wrappers/path.hpp
@@ -13,7 +13,7 @@
 namespace boost { namespace network { namespace http {
 
     template <class Tag>
-    class basic_request;
+    struct basic_request;
 
     namespace impl {
 
diff --git a/boost/network/protocol/http/message/wrappers/port.hpp b/boost/network/protocol/http/message/wrappers/port.hpp
@@ -13,7 +13,7 @@
 namespace boost { namespace network { namespace http {
 
     template <class Tag>
-    class basic_request;
+    struct basic_request;
 
     namespace impl {
 
diff --git a/boost/network/protocol/http/message/wrappers/protocol.hpp b/boost/network/protocol/http/message/wrappers/protocol.hpp
@@ -13,7 +13,7 @@
 namespace boost { namespace network { namespace http {
 
     template <class Tag>
-    class basic_request;
+    struct basic_request;
 
     namespace impl {
         template <class Tag>
diff --git a/boost/network/protocol/http/message/wrappers/query.hpp b/boost/network/protocol/http/message/wrappers/query.hpp
@@ -13,7 +13,7 @@
 namespace boost { namespace network { namespace http {
 
     template <class Tag>
-    class basic_request;
+    struct basic_request;
 
     namespace impl {
 
diff --git a/boost/network/protocol/http/message/wrappers/source.hpp b/boost/network/protocol/http/message/wrappers/source.hpp
@@ -16,7 +16,7 @@ namespace boost { namespace network { namespace http {
     struct basic_response;
 
     template <class Tag>
-    class basic_request;
+    struct basic_request;
 
     namespace impl {
 
diff --git a/boost/network/protocol/http/message/wrappers/uri.hpp b/boost/network/protocol/http/message/wrappers/uri.hpp
@@ -13,7 +13,7 @@
 namespace boost { namespace network { namespace http {
 
     template <class Tag>
-    class basic_request;
+    struct basic_request;
 
     namespace impl {
         template <class Tag>
diff --git a/boost/network/protocol/http/policies/async_connection.hpp b/boost/network/protocol/http/policies/async_connection.hpp
@@ -54,7 +54,6 @@ namespace boost { namespace network { namespace http {
         typedef boost::shared_ptr<connection_impl> connection_ptr;
         connection_ptr get_connection(boost::shared_ptr<resolver_type> resolver, basic_request<typename sync_only<Tag>::type> const & request_) {
             string_type protocol_ = protocol(request_);
-            boost::uint16_t port_ = port(request_);
             connection_ptr connection_(
                 new connection_impl(
                     follow_redirect_
diff --git a/boost/network/protocol/http/request.hpp b/boost/network/protocol/http/request.hpp
@@ -34,7 +34,7 @@
 namespace boost { namespace network { namespace http {
 
     template <class Tag>
-    class basic_request;
+    struct basic_request;
 
 } // namespace http
 
diff --git a/libs/network/test/CMakeLists.txt b/libs/network/test/CMakeLists.txt
@@ -15,6 +15,14 @@ set(Boost_USE_STATIC_LIBS ON)
 set(Boost_USE_MULTITHREADED ON)
 
 if (Boost_FOUND)
+    set_source_files_properties(
+        http_incremental_parser.cpp hello_world.cpp
+        http_1_0_test.cpp http_1_1_test.cpp
+        http_localhost_tests.cpp https_localhost_tests.cpp
+        message_test.cpp http_message_twst.cpp
+        message_transform_test.cpp url_test.cpp
+        PROPERTIES COMPILE_FLAGS "-Wall -Werror"
+        )
     add_executable(cpp-netlib-http_incremental_parser http_incremental_parser.cpp)
     add_executable(cpp-netlib-hello_world hello_world.cpp)
     add_executable(cpp-netlib-http_1_0_test http_1_0_test.cpp)

Original file line number	Diff line number	Diff line change
`@@ -42,7 +42,7 @@ namespace boost { namespace network { namespace http {`
`42`	`42`	`{};`
`43`	`43`
`44`	`44`	`template <class Tag>`
`45`		`- class basic_request : public request_base<Tag>::type`
	`45`	`+ struct basic_request : public request_base<Tag>::type`
`46`	`46`	`{`
`47`	`47`
`48`	`48`	`mutable boost::network::uri::http::uri uri_;`