From dd693007fe284cc61dfd2cfe02fbc1e1affa25fe Mon Sep 17 00:00:00 2001 From: Robert Dailey Date: Thu, 6 Aug 2015 17:17:32 -0500 Subject: [PATCH] Remove user-declared constructors for response class Removed the following defaulted constructors: * Default Constructor * Copy Constructor * Move Constructor The defaulted move constructor prevents implicit declaration of the copy assignment operator, which results in compiler errors since std::promise requires it. All 3 user-declared methods removed here are implicitly declared by the compiler anyway, so no need to default them in the first place. Relates to #547 --- http/src/network/http/v2/client/response.hpp | 17 ----------------- 1 file changed, 17 deletions(-) diff --git a/http/src/network/http/v2/client/response.hpp b/http/src/network/http/v2/client/response.hpp index 21cf37c0c..37fa348f9 100644 --- a/http/src/network/http/v2/client/response.hpp +++ b/http/src/network/http/v2/client/response.hpp @@ -60,23 +60,6 @@ class response { */ typedef headers_type::const_iterator const_headers_iterator; - /** - * \brief Constructor. - */ - response() = default; - - /** - * \brief Copy constructor. - * \param other The other response object. - */ - response(const response &other) = default; - - /** - * \brief Move constructor. - * \param other The other response object. - */ - response(response &&other) noexcept = default; - /** * \brief Swap function. * \param other The other response object.