Tidying:

fix #85

* Remove unnecessary boost/system includes
* Remove unused headers_type from write_preparation
* Use braced-init style construction
* handler_alloc doc update

CHANGELOG.md

@@ -1,6 +1,7 @@
 1.0.0-b17
 
 * Change implicit to default value in example
+* Tidy up some declarations
 
 --------------------------------------------------------------------------------
 

include/beast/core/buffer_cat.hpp

@@ -45,7 +45,7 @@ buffer_cat(B1 const& b1, B2 const& b2, Bn const&... bn)
 {
     return detail::buffer_cat_helper<
         boost::asio::const_buffer,
-            B1, B2, Bn...>(b1, b2, bn...);
+            B1, B2, Bn...>{b1, b2, bn...};
 }
 
 } // beast

include/beast/core/detail/stream_concepts.hpp

@@ -11,7 +11,6 @@
 #include <beast/core/buffer_concepts.hpp>
 #include <beast/core/error.hpp>
 #include <boost/asio/io_service.hpp>
-#include <boost/system/error_code.hpp>
 #include <type_traits>
 #include <utility>
 

include/beast/core/dynabuf_readstream.hpp

@@ -16,7 +16,6 @@
 #include <beast/core/detail/get_lowest_layer.hpp>
 #include <boost/asio/buffer.hpp>
 #include <boost/asio/io_service.hpp>
-#include <boost/system/error_code.hpp>
 #include <cstdint>
 #include <utility>
 

include/beast/core/handler_alloc.hpp

@@ -23,12 +23,13 @@ namespace beast {
 
     This allocator uses the handler customizations `asio_handler_allocate`
     and `asio_handler_deallocate` to manage memory. It meets the requirements
-    of `Allocator` and can be used anywhere a `std::allocator` is
+    of @b Allocator and can be used anywhere a `std::allocator` is
     accepted.
 
     @tparam T The type of objects allocated by the allocator.
 
-    @tparam CompletionHandler The type of handler.
+    @tparam CompletionHandler The type of handler. Copies will be made
+    of the handler as needed.
 
     @note Allocated memory is only valid until the handler is called. The
     caller is still responsible for freeing memory.

include/beast/core/impl/dynabuf_readstream.ipp

@@ -9,10 +9,9 @@
 #define BEAST_IMPL_DYNABUF_READSTREAM_HPP
 
 #include <beast/core/bind_handler.hpp>
+#include <beast/core/error.hpp>
 #include <beast/core/handler_concepts.hpp>
 #include <beast/core/handler_alloc.hpp>
-#include <boost/system/error_code.hpp>
-#include <boost/system/system_error.hpp>
 
 namespace beast {
 

include/beast/http/basic_parser_v1.hpp

@@ -513,9 +513,6 @@ private:
     bool
     needs_eof(std::false_type) const;
 
-    template<class...>
-    using void_t = void;
-
     template<class C>
     class check_on_start_t
     {

include/beast/http/detail/basic_parser_v1.hpp

@@ -8,9 +8,6 @@
 #ifndef BEAST_HTTP_DETAIL_BASIC_PARSER_V1_HPP
 #define BEAST_HTTP_DETAIL_BASIC_PARSER_V1_HPP
 
-#include <boost/system/error_code.hpp>
-#include <boost/utility/string_ref.hpp>
-#include <array>
 #include <cstdint>
 
 namespace beast {

include/beast/http/impl/write.ipp

@@ -109,9 +109,6 @@ write_fields(DynamicBuffer& dynabuf, FieldSequence const& fields)
 template<bool isRequest, class Body, class Headers>
 struct write_preparation
 {
-    using headers_type =
-        basic_headers<std::allocator<char>>;
-
     message_v1<isRequest, Body, Headers> const& msg;
     typename Body::writer w;
     streambuf sb;