Delete the copy constructor on the Request object
This code was in the codebase previously, and at some point got removed
to make copies. Considering the previous commits to this one, making
copies of the request object is a bad idea, and should be avoided.
Therefore, this commit deletes the copy constructor for the Request
object, making request copies fail to compile.
It should be noted, it does leave the move constructor, which I don't
think it's really used, but as a rule isn't an anti-pattern.
Tested:
Code compiles. No functional change.
Signed-off-by: Ed Tanous <edtanous@google.com>
Change-Id: Ib38ed79a7c60340fb00f922f29265a3c3c7beca8
diff --git a/http/http_request.hpp b/http/http_request.hpp
index be84f18..4567314 100644
--- a/http/http_request.hpp
+++ b/http/http_request.hpp
@@ -45,6 +45,9 @@
}
}
+ Request(const Request&) = delete;
+ Request& operator=(const Request&) = delete;
+
boost::beast::http::verb method() const
{
return req.method();